[dpdk-test-report] |WARNING| pw64806 [PATCH V4 10/10] net/bnxt: fix to cap max rings to minimum of compl rings and stat contexts

checkpatch at dpdk.org checkpatch at dpdk.org
Thu Jan 16 13:50:29 CET 2020


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

_coding style issues_


CHECK:MACRO_ARG_REUSE: Macro argument reuse 'bp' - possible side-effects?
#85: FILE: drivers/net/bnxt/bnxt.h:626:
+#define BNXT_MAX_RX_RINGS(bp) \
 	(BNXT_STINGRAY(bp) ? RTE_MIN(RTE_MIN(bp->max_rx_rings, \
 					     MAX_STINGRAY_RINGS), \
+				     bp->max_stat_ctx / 2U) : \
+				RTE_MIN(bp->max_rx_rings, \
+					bp->max_stat_ctx / 2U))

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'bp' - possible side-effects?
#93: FILE: drivers/net/bnxt/bnxt.h:632:
+#define BNXT_MAX_TX_RINGS(bp) \
+	(RTE_MIN((bp)->max_tx_rings, BNXT_MAX_RX_RINGS(bp)))

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'bp' - possible side-effects?
#96: FILE: drivers/net/bnxt/bnxt.h:635:
+#define BNXT_MAX_RINGS(bp) \
+	(RTE_MIN((((bp)->max_cp_rings - BNXT_NUM_ASYNC_CPR(bp)) / 2U), \
+		 BNXT_MAX_TX_RINGS(bp)))

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


More information about the test-report mailing list