* |WARNING| pw143879 [PATCH 21/33] net/cnxk: support Rx burst scalar for cn20k
[not found] <20240910085909.1514457-22-ndabilpuram@marvell.com>
@ 2024-09-10 9:36 ` checkpatch
0 siblings, 0 replies; only message in thread
From: checkpatch @ 2024-09-10 9:36 UTC (permalink / raw)
To: test-report; +Cc: Nithin Dabilpuram
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/143879
_coding style issues_
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'b' - possible side-effects?
#290: FILE: drivers/net/cnxk/cn20k_rx.h:37:
+#define CQE_PTR_OFF(b, i, o, f) \
+ (((f) & NIX_RX_VWQE_F) ? (uint64_t *)(((uintptr_t)((uint64_t *)(b))[i]) + (o)) : \
+ (uint64_t *)(((uintptr_t)(b)) + CQE_SZ(i) + (o)))
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i' - possible side-effects?
#290: FILE: drivers/net/cnxk/cn20k_rx.h:37:
+#define CQE_PTR_OFF(b, i, o, f) \
+ (((f) & NIX_RX_VWQE_F) ? (uint64_t *)(((uintptr_t)((uint64_t *)(b))[i]) + (o)) : \
+ (uint64_t *)(((uintptr_t)(b)) + CQE_SZ(i) + (o)))
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'o' - possible side-effects?
#290: FILE: drivers/net/cnxk/cn20k_rx.h:37:
+#define CQE_PTR_OFF(b, i, o, f) \
+ (((f) & NIX_RX_VWQE_F) ? (uint64_t *)(((uintptr_t)((uint64_t *)(b))[i]) + (o)) : \
+ (uint64_t *)(((uintptr_t)(b)) + CQE_SZ(i) + (o)))
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'b' - possible side-effects?
#293: FILE: drivers/net/cnxk/cn20k_rx.h:40:
+#define CQE_PTR_DIFF(b, i, o, f) \
+ (((f) & NIX_RX_VWQE_F) ? (uint64_t *)(((uintptr_t)((uint64_t *)(b))[i]) - (o)) : \
+ (uint64_t *)(((uintptr_t)(b)) + CQE_SZ(i) - (o)))
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i' - possible side-effects?
#293: FILE: drivers/net/cnxk/cn20k_rx.h:40:
+#define CQE_PTR_DIFF(b, i, o, f) \
+ (((f) & NIX_RX_VWQE_F) ? (uint64_t *)(((uintptr_t)((uint64_t *)(b))[i]) - (o)) : \
+ (uint64_t *)(((uintptr_t)(b)) + CQE_SZ(i) - (o)))
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'o' - possible side-effects?
#293: FILE: drivers/net/cnxk/cn20k_rx.h:40:
+#define CQE_PTR_DIFF(b, i, o, f) \
+ (((f) & NIX_RX_VWQE_F) ? (uint64_t *)(((uintptr_t)((uint64_t *)(b))[i]) - (o)) : \
+ (uint64_t *)(((uintptr_t)(b)) + CQE_SZ(i) - (o)))
WARNING:BRACES: braces {} are not necessary for single statement blocks
#308: FILE: drivers/net/cnxk/cn20k_rx.h:55:
+ if (m->nb_segs == 1 && m->next) {
+ rte_panic("mbuf->next[%p] valid when mbuf->nb_segs is %d", m->next, m->nb_segs);
+ }
total: 0 errors, 1 warnings, 6 checks, 756 lines checked
Warning in drivers/net/cnxk/cn20k_rx.h:
Using rte_panic/rte_exit
^ permalink raw reply [flat|nested] only message in thread