* [dpdk-test-report] |WARNING| pw64524 [PATCH V2 10/10] net/bnxt: fix to cap max rings to minimum of compl rings and stat contexts
[not found] <20200113050118.28719-11-kalesh-anakkur.purayil@broadcom.com>
@ 2020-01-13 4:46 ` checkpatch
2020-01-13 8:58 ` [dpdk-test-report] |SUCCESS| pw64524 net/bnxt: fix to use correct IOVA mapping 0-day Robot
1 sibling, 0 replies; 2+ messages in thread
From: checkpatch @ 2020-01-13 4:46 UTC (permalink / raw)
To: test-report; +Cc: Kalesh A P
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/64524
_coding style issues_
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'bp' - possible side-effects?
#84: 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?
#92: 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?
#95: 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