diff mbox series

[v3,14/37] net/qede: replace RTE_LOG_DP with rte_log_dp

Message ID 20231213163820.894987-15-stephen@networkplumber.org (mailing list archive)
State Superseded, archived
Delegated to: Thomas Monjalon
Headers
Series Replace uso of RTE_LOGTYPE_PMD |

Checks

Context Check Description
ci/checkpatch warning coding style issues

Commit Message

Stephen Hemminger Dec. 13, 2023, 4:35 p.m. UTC 
  Want datapath logs to use own logtype.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/qede/qede_logs.h | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

Comments

Devendra Singh Rawat Dec. 14, 2023, 12:34 a.m. UTC | #1 
>-----Original Message-----
>From: Stephen Hemminger <stephen@networkplumber.org>
>Sent: Wednesday, December 13, 2023 10:06 PM
>To: dev@dpdk.org
>Cc: Stephen Hemminger <stephen@networkplumber.org>; Devendra Singh
>Rawat <dsinghrawat@marvell.com>; Alok Prasad <palok@marvell.com>
>Subject: [EXT] [PATCH v3 14/37] net/qede: replace RTE_LOG_DP with
>rte_log_dp
>
>External Email
>
>----------------------------------------------------------------------
>Want datapath logs to use own logtype.
>
>Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
>---
> drivers/net/qede/qede_logs.h | 10 ++++++----
> 1 file changed, 6 insertions(+), 4 deletions(-)
>
>diff --git a/drivers/net/qede/qede_logs.h b/drivers/net/qede/qede_logs.h
>index 3187d97bb7b0..f7aa14df7e3b 100644
>--- a/drivers/net/qede/qede_logs.h
>+++ b/drivers/net/qede/qede_logs.h
>@@ -58,16 +58,18 @@ extern int qede_logtype_init;  #define
>PMD_INIT_FUNC_TRACE(edev) PMD_INIT_LOG(DEBUG, edev, " >>")
>
> #ifdef RTE_LIBRTE_QEDE_DEBUG_TX
>-#define PMD_TX_LOG(level, q, fmt, args...) \
>-	RTE_LOG(level, PMD, "%s(): port=%u queue=%u " fmt "\n", \
>+#define PMD_TX_LOG(level, q, fmt, args...)			\
>+	rte_log(RTE_LOG_ ## level, qede_logtype_driver,		\
>+		"%s(): port=%u queue=%u " fmt "\n",		\
> 		__func__, q->port_id, q->queue_id, ## args)  #else  #define
>PMD_TX_LOG(level, fmt, args...) do { } while (0)  #endif
>
> #ifdef RTE_LIBRTE_QEDE_DEBUG_RX
>-#define PMD_RX_LOG(level, q, fmt, args...) \
>-	RTE_LOG(level, PMD, "%s(): port=%u queue=%u " fmt "\n",	\
>+#define PMD_RX_LOG(level, q, fmt, args...)			\
>+	rte_log(RTE_LOG_ ## level, qede_logtype_driver,		\
>+		"%s(): port=%u queue=%u " fmt "\n",		\
> 		__func__, q->port_id, q->queue_id, ## args)  #else  #define
>PMD_RX_LOG(level, q, fmt, args...) do { } while (0)
>--
>2.43.0

Acked.
Acked-by: Devendra Singh Rawat <dsinghrawat@marvell.com>
diff mbox series

Patch

diff --git a/drivers/net/qede/qede_logs.h b/drivers/net/qede/qede_logs.h
index 3187d97bb7b0..f7aa14df7e3b 100644
--- a/drivers/net/qede/qede_logs.h
+++ b/drivers/net/qede/qede_logs.h
@@ -58,16 +58,18 @@  extern int qede_logtype_init;
 #define PMD_INIT_FUNC_TRACE(edev) PMD_INIT_LOG(DEBUG, edev, " >>")
 
 #ifdef RTE_LIBRTE_QEDE_DEBUG_TX
-#define PMD_TX_LOG(level, q, fmt, args...) \
-	RTE_LOG(level, PMD, "%s(): port=%u queue=%u " fmt "\n", \
+#define PMD_TX_LOG(level, q, fmt, args...)			\
+	rte_log(RTE_LOG_ ## level, qede_logtype_driver,		\
+		"%s(): port=%u queue=%u " fmt "\n",		\
 		__func__, q->port_id, q->queue_id, ## args)
 #else
 #define PMD_TX_LOG(level, fmt, args...) do { } while (0)
 #endif
 
 #ifdef RTE_LIBRTE_QEDE_DEBUG_RX
-#define PMD_RX_LOG(level, q, fmt, args...) \
-	RTE_LOG(level, PMD, "%s(): port=%u queue=%u " fmt "\n",	\
+#define PMD_RX_LOG(level, q, fmt, args...)			\
+	rte_log(RTE_LOG_ ## level, qede_logtype_driver,		\
+		"%s(): port=%u queue=%u " fmt "\n",		\
 		__func__, q->port_id, q->queue_id, ## args)
 #else
 #define PMD_RX_LOG(level, q, fmt, args...) do { } while (0)