* [dpdk-test-report] |WARNING| pw64612 [PATCH v3 10/10] net/bnxt: fix calculation of max rings
[not found] <20200114051435.46093-11-ajit.khaparde@broadcom.com>
@ 2020-01-14 5:19 ` checkpatch
2020-01-14 11:59 ` [dpdk-test-report] |SUCCESS| pw64612 net/bnxt: fix to use correct IOVA mapping 0-day Robot
1 sibling, 0 replies; 2+ messages in thread
From: checkpatch @ 2020-01-14 5:19 UTC (permalink / raw)
To: test-report; +Cc: Ajit Khaparde
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/64612
_coding style issues_
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'bp' - possible side-effects?
#88: 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?
#96: 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?
#99: 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
^ permalink raw reply [flat|nested] 2+ messages in thread