[PATCH v1 27/30] net/i40e/base: change time variables from 64 bit to 32 bit

[Ilgiewicz, Jaroslaw]

We will send the fix once we sync internally hopefully tomorrow.
Sorry for the delay.
Jaroslaw Ilgiewicz
________________________________
From: David Marchand <david.marchand at redhat.com>
Sent: Friday, October 25, 2024 1:45 PM
To: Burakov, Anatoly <anatoly.burakov at intel.com>; Richardson, Bruce <bruce.richardson at intel.com>; Ilgiewicz, Jaroslaw <jaroslaw.ilgiewicz at intel.com>
Cc: dev at dpdk.org <dev at dpdk.org>; Thomas Monjalon <thomas at monjalon.net>; Luca Boccassi <bluca at debian.org>
Subject: Re: [PATCH v1 27/30] net/i40e/base: change time variables from 64 bit to 32 bit

On Thu, Oct 17, 2024 at 1:43 PM David Marchand
<david.marchand at redhat.com> wrote:
>
> On Mon, Sep 2, 2024 at 11:58 AM Anatoly Burakov
> <anatoly.burakov at intel.com> wrote:
> >
> > From: Jaroslaw Ilgiewicz <jaroslaw.ilgiewicz at intel.com>
> >
> > Time variables were designed for 32 bit and 64 bit variables are not
> > necessary. Changed all to 32 bit.
> >
> > Signed-off-by: Jaroslaw Ilgiewicz <jaroslaw.ilgiewicz at intel.com>
> > Signed-off-by: Anatoly Burakov <anatoly.burakov at intel.com>
> > ---
> >  drivers/net/i40e/base/i40e_common.c    | 2 +-
> >  drivers/net/i40e/base/i40e_nvm.c       | 6 +++---
> >  drivers/net/i40e/base/i40e_prototype.h | 2 +-
> >  3 files changed, 5 insertions(+), 5 deletions(-)
> >
> > diff --git a/drivers/net/i40e/base/i40e_common.c b/drivers/net/i40e/base/i40e_common.c
> > index 07e18deaea..a2cfafeda9 100644
> > --- a/drivers/net/i40e/base/i40e_common.c
> > +++ b/drivers/net/i40e/base/i40e_common.c
> > @@ -3596,7 +3596,7 @@ enum i40e_status_code i40e_aq_debug_write_register(struct i40e_hw *hw,
> >  enum i40e_status_code i40e_aq_request_resource(struct i40e_hw *hw,
> >                                 enum i40e_aq_resources_ids resource,
> >                                 enum i40e_aq_resource_access_type access,
> > -                               u8 sdp_number, u64 *timeout,
> > +                               u8 sdp_number, u32 *timeout,
> >                                 struct i40e_asq_cmd_details *cmd_details)
> >  {
> >         struct i40e_aq_desc desc;
> > diff --git a/drivers/net/i40e/base/i40e_nvm.c b/drivers/net/i40e/base/i40e_nvm.c
> > index 2f6cd9eda5..185af67817 100644
> > --- a/drivers/net/i40e/base/i40e_nvm.c
> > +++ b/drivers/net/i40e/base/i40e_nvm.c
> > @@ -62,7 +62,7 @@ enum i40e_status_code i40e_acquire_nvm(struct i40e_hw *hw,
> >  {
> >         enum i40e_status_code ret_code = I40E_SUCCESS;
> >         u32 gtime, timeout;
> > -       u64 time_left = 0;
> > +       u32 time_left = 0;
>
> Some logs (a few lines below) needs updating:
>
>                 i40e_debug(hw, I40E_DEBUG_NVM,
>                            "NVM acquire type %d failed time_left=%"
> PRIu64 " ret=%d aq_err=%d\n",
>                            access, time_left, ret_code,
> hw->aq.asq_last_status);
>
> And
>                         i40e_debug(hw, I40E_DEBUG_NVM,
>                                    "NVM acquire timed out, wait %"
> PRIu64 " ms before trying again. status=%d aq_err=%d\n",
>                                    time_left, ret_code,
> hw->aq.asq_last_status);
>
> Afaiu, it should be PRIu32.
>
> This was raised by OBS CI:
> https://build.opensuse.org/package/live_build_log/home:bluca:dpdk/dpdk/Debian_12/x86_64

Reminder, this was mentionned during the maintainers call yesterday.
Please send a fix.


Thanks.

--
David Marchand

---------------------------------------------------------------------
Intel Technology Poland sp. z o.o.
ul. Slowackiego 173 | 80-298 Gdansk | Sad Rejonowy Gdansk Polnoc | VII Wydzial Gospodarczy Krajowego Rejestru Sadowego - KRS 101882 | NIP 957-07-52-316 | Kapital zakladowy 200.000 PLN.
Spolka oswiadcza, ze posiada status duzego przedsiebiorcy w rozumieniu ustawy z dnia 8 marca 2013 r. o przeciwdzialaniu nadmiernym opoznieniom w transakcjach handlowych.

Ta wiadomosc wraz z zalacznikami jest przeznaczona dla okreslonego adresata i moze zawierac informacje poufne. W razie przypadkowego otrzymania tej wiadomosci, prosimy o powiadomienie nadawcy oraz trwale jej usuniecie; jakiekolwiek przegladanie lub rozpowszechnianie jest zabronione.
This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). If you are not the intended recipient, please contact the sender and delete all copies; any review or distribution by others is strictly prohibited.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mails.dpdk.org/archives/dev/attachments/20241028/c345fd3b/attachment-0001.htm>