|WARNING| pw135149 [PATCH v3 14/37] net/qede: replace RTE_LOG_DP with rte_log_dp
checkpatch at dpdk.org
checkpatch at dpdk.org
Wed Dec 13 17:41:00 CET 2023
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/135149
_coding style issues_
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'q' - possible side-effects?
#103: FILE: drivers/net/qede/qede_logs.h:61:
+#define PMD_TX_LOG(level, q, fmt, args...) \
+ rte_log(RTE_LOG_ ## level, qede_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
#103: FILE: drivers/net/qede/qede_logs.h:61:
+#define PMD_TX_LOG(level, q, fmt, args...) \
+ rte_log(RTE_LOG_ ## level, qede_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?
#114: FILE: drivers/net/qede/qede_logs.h:70:
+#define PMD_RX_LOG(level, q, fmt, args...) \
+ rte_log(RTE_LOG_ ## level, qede_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
#114: FILE: drivers/net/qede/qede_logs.h:70:
+#define PMD_RX_LOG(level, q, fmt, args...) \
+ rte_log(RTE_LOG_ ## level, qede_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