* [dpdk-test-report] |WARNING| pw94434 [PATCH v3 25/62] net/cnxk: add Rx support for cn10k
[not found] <20210618103741.26526-26-ndabilpuram@marvell.com>
@ 2021-06-18 10:42 ` checkpatch
0 siblings, 0 replies; only message in thread
From: checkpatch @ 2021-06-18 10:42 UTC (permalink / raw)
To: test-report; +Cc: Nithin Dabilpuram
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/94434
_coding style issues_
CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#349: FILE: drivers/net/cnxk/cn10k_rx.h:193:
+ rte_prefetch_non_temporal(
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#378: FILE: drivers/net/cnxk/cn10k_rx.h:222:
+#define NIX_RX_FASTPATH_MODES \
+R(no_offload, 0, 0, 0, 0, NIX_RX_OFFLOAD_NONE) \
+R(rss, 0, 0, 0, 1, RSS_F) \
+R(ptype, 0, 0, 1, 0, PTYPE_F) \
+R(ptype_rss, 0, 0, 1, 1, PTYPE_F | RSS_F) \
+R(cksum, 0, 1, 0, 0, CKSUM_F) \
+R(cksum_rss, 0, 1, 0, 1, CKSUM_F | RSS_F) \
+R(cksum_ptype, 0, 1, 1, 0, CKSUM_F | PTYPE_F) \
+R(cksum_ptype_rss, 0, 1, 1, 1, CKSUM_F | PTYPE_F | RSS_F) \
+R(mark, 1, 0, 0, 0, MARK_F) \
+R(mark_rss, 1, 0, 0, 1, MARK_F | RSS_F) \
+R(mark_ptype, 1, 0, 1, 0, MARK_F | PTYPE_F) \
+R(mark_ptype_rss, 1, 0, 1, 1, MARK_F | PTYPE_F | RSS_F) \
+R(mark_cksum, 1, 1, 0, 0, MARK_F | CKSUM_F) \
+R(mark_cksum_rss, 1, 1, 0, 1, MARK_F | CKSUM_F | RSS_F) \
+R(mark_cksum_ptype, 1, 1, 1, 0, MARK_F | CKSUM_F | PTYPE_F)\
+R(mark_cksum_ptype_rss, 1, 1, 1, 1, MARK_F | CKSUM_F | PTYPE_F | RSS_F)
WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon
#396: FILE: drivers/net/cnxk/cn10k_rx.h:240:
+#define R(name, f3, f2, f1, f0, flags) \
+ uint16_t __rte_noinline __rte_hot cn10k_nix_recv_pkts_##name( \
+ void *rx_queue, struct rte_mbuf **rx_pkts, uint16_t pkts);
total: 1 errors, 1 warnings, 1 checks, 303 lines checked
^ permalink raw reply [flat|nested] only message in thread