* [dpdk-test-report] |WARNING| pw64806 [PATCH V4 10/10] net/bnxt: fix to cap max rings to minimum of compl rings and stat contexts
[not found] <20200116130455.30193-11-kalesh-anakkur.purayil@broadcom.com>
@ 2020-01-16 12:50 ` checkpatch
2020-01-16 20:00 ` [dpdk-test-report] |SUCCESS| pw64806 net/bnxt: fix to use correct IOVA mapping 0-day Robot
1 sibling, 0 replies; 2+ messages in thread
From: checkpatch @ 2020-01-16 12:50 UTC (permalink / raw)
To: test-report; +Cc: Kalesh A P
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
^ permalink raw reply [flat|nested] 2+ messages in thread