|WARNING| pw135089 [PATCH 15/26] net/qede: replace RTE_LOG_DP with rte_log_dp

checkpatch at dpdk.org checkpatch at dpdk.org
Wed Dec 13 02:48:43 CET 2023


Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/135089

_coding style issues_


CHECK:MACRO_ARG_REUSE: Macro argument reuse 'q' - possible side-effects?
#101: FILE: drivers/net/qede/qede_logs.h:61:
+#define PMD_TX_LOG(level, q, fmt, args...)			\
+	rte_log(RTE_LOG_ ## level, qded_logtype_driver,		\
+		"%s(): port=%u queue=%u " fmt "
",		\
 		__func__, q->port_id, q->queue_id, ## args)

CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'q' may be better as '(q)' to avoid precedence issues
#101: FILE: drivers/net/qede/qede_logs.h:61:
+#define PMD_TX_LOG(level, q, fmt, args...)			\
+	rte_log(RTE_LOG_ ## level, qded_logtype_driver,		\
+		"%s(): port=%u queue=%u " fmt "
",		\
 		__func__, q->port_id, q->queue_id, ## args)

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'q' - possible side-effects?
#112: FILE: drivers/net/qede/qede_logs.h:70:
+#define PMD_RX_LOG(level, q, fmt, args...)			\
+	rte_log(RTE_LOG_ ## level, qded_logtype_driver,		\
+		"%s(): port=%u queue=%u " fmt "
",		\
 		__func__, q->port_id, q->queue_id, ## args)

CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'q' may be better as '(q)' to avoid precedence issues
#112: FILE: drivers/net/qede/qede_logs.h:70:
+#define PMD_RX_LOG(level, q, fmt, args...)			\
+	rte_log(RTE_LOG_ ## level, qded_logtype_driver,		\
+		"%s(): port=%u queue=%u " fmt "
",		\
 		__func__, q->port_id, q->queue_id, ## args)

total: 0 errors, 0 warnings, 4 checks, 22 lines checked


More information about the test-report mailing list