[PATCH v2] net/i40e/base: fix the debug print format
Bruce Richardson
bruce.richardson at intel.com
Tue Jan 21 10:05:34 CET 2025
On Wed, Jan 15, 2025 at 03:44:17PM +0800, Jaroslaw Ilgiewicz wrote:
> This patch modifies format specifier in debug prints to match to the
> change of time variables from 64 bit to 32 bit.
>
> Fixes: d980a401b137 ("net/i40e/base: add NVM acquire with custom timeout")
> Cc: stable at dpdk.org
>
> Signed-off-by: Zhichao Zeng <zhichaox.zeng at intel.com>
> Signed-off-by: Jaroslaw Ilgiewicz <jaroslaw.ilgiewicz at intel.com>
> ---
> drivers/net/i40e/base/README | 2 +-
> drivers/net/i40e/base/i40e_nvm.c | 10 ++++------
> 2 files changed, 5 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/net/i40e/base/README b/drivers/net/i40e/base/README
> index c5305ffae0..188633dde3 100644
> --- a/drivers/net/i40e/base/README
> +++ b/drivers/net/i40e/base/README
> @@ -6,7 +6,7 @@ Intel® I40E driver
> ==================
>
> This directory contains source code of i40e base driver generated on
> -2024-08-30 released by the team which develops
> +2025-01-15 released by the team which develops
> basic drivers for any i40e NIC. The directory of base/ contains the
> original source package.
> This driver is valid for the product(s) listed below
> diff --git a/drivers/net/i40e/base/i40e_nvm.c b/drivers/net/i40e/base/i40e_nvm.c
> index 3e16a0d997..890c1dfc8a 100644
> --- a/drivers/net/i40e/base/i40e_nvm.c
> +++ b/drivers/net/i40e/base/i40e_nvm.c
> @@ -145,9 +145,8 @@ enum i40e_status_code i40e_acquire_nvm_ex(struct i40e_hw *hw,
>
> if (ret_code)
> i40e_debug(hw, I40E_DEBUG_NVM,
> - "NVM acquire type %d failed time_left=%llu ret=%d aq_err=%d\n",
> - access, (unsigned long long)time_left, ret_code,
> - hw->aq.asq_last_status);
> + "NVM acquire type %d failed time_left=%" PRIu32 " ret=%d aq_err=%d\n",
> + access, time_left, ret_code, hw->aq.asq_last_status);
>
Since this is not actually fixing a bug, but just aligning the DPDK code
with the internal base code, I think we can defer this patch. Let's look at
it again, if it's worth aligning these simple fmt strings, when we next have
a base code update with some more significant changes in it.
Thanks,
/Bruce
More information about the dev
mailing list