diff mbox series

[v3,16/37] net/vmxnet3: do not use PMD logtype

Message ID	20231213163820.894987-17-stephen@networkplumber.org (mailing list archive)
State	Superseded, archived
Delegated to:	Thomas Monjalon
Headers	From: Stephen Hemminger <stephen@networkplumber.org> To: dev@dpdk.org Cc: Stephen Hemminger <stephen@networkplumber.org>, Jochen Behrens <jbehrens@vmware.com> Subject: [PATCH v3 16/37] net/vmxnet3: do not use PMD logtype Date: Wed, 13 Dec 2023 08:35:50 -0800 Message-ID: <20231213163820.894987-17-stephen@networkplumber.org> In-Reply-To: <20231213163820.894987-1-stephen@networkplumber.org> References: <20231213014408.612051-1-stephen@networkplumber.org> <20231213163820.894987-1-stephen@networkplumber.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: list Errors-To: dev-bounces@dpdk.org
Series	Replace uso of RTE_LOGTYPE_PMD \| [v3,00/37] Replace uso of RTE_LOGTYPE_PMD [v3,01/37] log: fix doc comment for RTE_LOG_DP() [v3,02/37] log: add rte_log_dp() [v3,03/37] net/atlantic: replace RTE_LOG_DP with rte_log_dp [v3,04/37] net/avp: replace RTE_LOG_DP with rte_log_dp [v3,05/37] net/bnxt: replace RTE_LOG_DP with rte_log_dp [v3,06/37] net/dpaa: replace RTE_LOG_DP with rte_log_dp [v3,07/37] net/dpaa2: replace RTE_LOG_DP with rte_log_dp [v3,08/37] net/enetc, net/enetfec: replace RTE_LOG_DP with rte_log_dp [v3,09/37] net/igc: replace RTE_LOG_DP with rte_log_dp [v3,10/37] net/mana: replace RTE_LOG_DP with rte_log_dp [v3,11/37] net/mvpp2: do not use PMD logtype [v3,12/37] net/octeon_ep: replace RTE_LOG_DP with rte_log_dp [v3,13/37] net/pfe: replace RTE_LOG_DP with rte_log_dp [v3,14/37] net/qede: replace RTE_LOG_DP with rte_log_dp [v3,15/37] net/virtio: replace RTE_LOG_DP with rte_log_dp [v3,16/37] net/vmxnet3: do not use PMD logtype [v3,17/37] common/cnxk: replace RTE_LOG_DP with rte_log_dp [v3,18/37] common/cpt: replace RTE_LOG_DP with rte_log_dp [v3,19/37] common/sfc_efx: remove use of PMD logtype [v3,20/37] common/dpaax: do not use PMD logtype [v3,21/37] basband/la12xx: replace RTE_LOG_DP with rte_log_dp [v3,22/37] bus/cdx: replace RTE_LOG_DP with rte_log_dp [v3,23/37] bus/fslmc: replace RTE_LOG_DP with rte_log_dp [v3,24/37] dma/dpaa, dma/dpaa2: replace RTE_LOG_DP with rte_log_dp [v3,25/37] mempool/dpaa, mempool/dpaa2: do not use logtype PMD [v3,26/37] vdpa/ifc: replace use of RTE_LOGTYPE_PMD [v3,27/37] raw/cnxk_bphy: replace PMD logtype with dynamic type [v3,28/37] raw/cnxk_gpio: replace PMD logtype with dynamic type [v3,29/37] raw/dpaa2_cmdif: replace PMD logtype with rte_log_dp [v3,30/37] raw/ifpga: replace PMD logtype with dynamic type [v3,31/37] event/dlb2: replace logtype PMD with rte_log_dp [v3,32/37] event/dpaa, event/dpaa2: use local logtype [v3,33/37] event/skeleton: replace logtype PMD with dynamic type [v3,34/37] crypto/caam_jr: replace logtype PMD [v3,35/37] crypto/ccp: do not use PMD logtype [v3,36/37] crypto/scheduler: replace use of logtype PMD [v3,37/37] crypto/dpaa_sec, crypto/dpaa2_sec: replace use of PMD logtype

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK

Commit Message

Stephen Hemminger Dec. 13, 2023, 4:35 p.m. UTC

  The driver used a slightly different (and older) format
for datapath logs. Remove PMD logtype here.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/vmxnet3/vmxnet3_ethdev.c |  2 +-
 drivers/net/vmxnet3/vmxnet3_logs.h   | 12 ++++++++----
 2 files changed, 9 insertions(+), 5 deletions(-)

diff mbox series

Patch

diff --git a/drivers/net/vmxnet3/vmxnet3_ethdev.c b/drivers/net/vmxnet3/vmxnet3_ethdev.c
index e49191718aea..4fd704045fc4 100644
--- a/drivers/net/vmxnet3/vmxnet3_ethdev.c
+++ b/drivers/net/vmxnet3/vmxnet3_ethdev.c
@@ -1919,7 +1919,7 @@  vmxnet3_interrupt_handler(void *param)
 	if (events == 0)
 		goto done;
 
-	RTE_LOG(DEBUG, PMD, "Reading events: 0x%X", events);
+	PMD_DRV_LOG(DEBUG, "Reading events: 0x%X", events);
 	vmxnet3_process_events(dev);
 done:
 	vmxnet3_enable_intr(hw, *eventIntrIdx);
diff --git a/drivers/net/vmxnet3/vmxnet3_logs.h b/drivers/net/vmxnet3/vmxnet3_logs.h
index 74154e3a1a6d..058d704f82c9 100644
--- a/drivers/net/vmxnet3/vmxnet3_logs.h
+++ b/drivers/net/vmxnet3/vmxnet3_logs.h
@@ -11,28 +11,32 @@  extern int vmxnet3_logtype_init;
 		"%s(): " fmt "\n", __func__, ## args)
 #define PMD_INIT_FUNC_TRACE() PMD_INIT_LOG(DEBUG, " >>")
 
+extern int vmxnet3_logtype_driver;
+
 #ifdef RTE_LIBRTE_VMXNET3_DEBUG_RX
 #define PMD_RX_LOG(level, fmt, args...) \
-	RTE_LOG(level, PMD, "%s(): " fmt "\n", __func__, ## args)
+	rte_log(RTE_LOG_ ## level, vmxnet3_logtype_driver, \
+		 "%s(): " fmt "\n", __func__, ## args)
 #else
 #define PMD_RX_LOG(level, fmt, args...) do { } while(0)
 #endif
 
 #ifdef RTE_LIBRTE_VMXNET3_DEBUG_TX
 #define PMD_TX_LOG(level, fmt, args...) \
-	RTE_LOG(level, PMD, "%s(): " fmt "\n", __func__, ## args)
+	rte_log(RTE_LOG_ ## level, vmxnet3_logtype_driver, \
+		 "%s(): " fmt "\n", __func__, ## args)
 #else
 #define PMD_TX_LOG(level, fmt, args...) do { } while(0)
 #endif
 
 #ifdef RTE_LIBRTE_VMXNET3_DEBUG_TX_FREE
 #define PMD_TX_FREE_LOG(level, fmt, args...) \
-	RTE_LOG(level, PMD, "%s(): " fmt "\n", __func__, ## args)
+	rte_log(RTE_LOG_ ## level, vmxnet3_logtype_driver, \
+		 "%s(): " fmt "\n", __func__, ## args)
 #else
 #define PMD_TX_FREE_LOG(level, fmt, args...) do { } while(0)
 #endif
 
-extern int vmxnet3_logtype_driver;
 #define PMD_DRV_LOG(level, fmt, args...) \
 	rte_log(RTE_LOG_ ## level, vmxnet3_logtype_driver, \
 		"%s(): " fmt "\n", __func__, ## args)