diff mbox series

[net,v3] iavf: Fix missing state logs

Message ID 20220615175720.4058112-1-xuejun.zhang@intel.com
State Accepted
Delegated to: Anthony Nguyen
Headers show
Series [net,v3] iavf: Fix missing state logs | expand

Commit Message

Zhang, Xuejun June 15, 2022, 5:57 p.m. UTC
From: Przemyslaw Patynowski <przemyslawx.patynowski@intel.com>

Fix debug prints, by adding missing state prints.

Extend iavf_state_str by strings for __IAVF_INIT_EXTENDED_CAPS and
__IAVF_INIT_CONFIG_ADAPTER.

Without this patch, when enabling debug prints for iavf.h, user will
see:
iavf 0000:06:0e.0: state transition from:__IAVF_INIT_GET_RESOURCES to:__IAVF_UNKNOWN_STATE
iavf 0000:06:0e.0: state transition from:__IAVF_UNKNOWN_STATE to:__IAVF_UNKNOWN_STATE

Fixes: 605ca7c5c670 ("iavf: Fix kernel BUG in free_msi_irqs")
Signed-off-by: Przemyslaw Patynowski <przemyslawx.patynowski@intel.com>
Signed-off-by: Jun Zhang <xuejun.zhang@intel.com>
---
v2: add Fixes Tag and Blank lines @ commit message
v3: change code author
---
 drivers/net/ethernet/intel/iavf/iavf.h | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Jankowski, Konrad0 July 5, 2022, 8:51 a.m. UTC | #1
> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf Of
> Jun Zhang
> Sent: Wednesday, June 15, 2022 7:57 PM
> To: intel-wired-lan@lists.osuosl.org
> Cc: Patynowski, PrzemyslawX <przemyslawx.patynowski@intel.com>
> Subject: [Intel-wired-lan] [PATCH net v3] iavf: Fix missing state logs
> 
> From: Przemyslaw Patynowski <przemyslawx.patynowski@intel.com>
> 
> Fix debug prints, by adding missing state prints.
> 
> Extend iavf_state_str by strings for __IAVF_INIT_EXTENDED_CAPS and
> __IAVF_INIT_CONFIG_ADAPTER.
> 
> Without this patch, when enabling debug prints for iavf.h, user will
> see:
> iavf 0000:06:0e.0: state transition from:__IAVF_INIT_GET_RESOURCES
> to:__IAVF_UNKNOWN_STATE iavf 0000:06:0e.0: state transition
> from:__IAVF_UNKNOWN_STATE to:__IAVF_UNKNOWN_STATE
> 
> Fixes: 605ca7c5c670 ("iavf: Fix kernel BUG in free_msi_irqs")
> Signed-off-by: Przemyslaw Patynowski
> <przemyslawx.patynowski@intel.com>
> Signed-off-by: Jun Zhang <xuejun.zhang@intel.com>
> ---
> v2: add Fixes Tag and Blank lines @ commit message
> v3: change code author
> ---
>  drivers/net/ethernet/intel/iavf/iavf.h | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/drivers/net/ethernet/intel/iavf/iavf.h
> b/drivers/net/ethernet/intel/iavf/iavf.h
> index fda1198d2c00..cbcf0c710072 100644
> --- a/drivers/net/ethernet/intel/iavf/iavf.h
> +++ b/drivers/net/ethernet/intel/iavf/iavf.h

Tested-by: Konrad Jankowski <konrad0.jankowski@intel.com>
diff mbox series

Patch

diff --git a/drivers/net/ethernet/intel/iavf/iavf.h b/drivers/net/ethernet/intel/iavf/iavf.h
index fda1198d2c00..cbcf0c710072 100644
--- a/drivers/net/ethernet/intel/iavf/iavf.h
+++ b/drivers/net/ethernet/intel/iavf/iavf.h
@@ -464,6 +464,10 @@  static inline const char *iavf_state_str(enum iavf_state_t state)
 		return "__IAVF_INIT_VERSION_CHECK";
 	case __IAVF_INIT_GET_RESOURCES:
 		return "__IAVF_INIT_GET_RESOURCES";
+	case __IAVF_INIT_EXTENDED_CAPS:
+		return "__IAVF_INIT_EXTENDED_CAPS";
+	case __IAVF_INIT_CONFIG_ADAPTER:
+		return "__IAVF_INIT_CONFIG_ADAPTER";
 	case __IAVF_INIT_SW:
 		return "__IAVF_INIT_SW";
 	case __IAVF_INIT_FAILED: