From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 08BFAA00BE for ; Mon, 13 Dec 2021 09:24:58 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id F268A410F5; Mon, 13 Dec 2021 09:24:57 +0100 (CET) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id 390A040140 for ; Mon, 13 Dec 2021 09:24:57 +0100 (CET) Received: by dpdk.org (Postfix, from userid 65534) id 3514112353B; Mon, 13 Dec 2021 09:24:57 +0100 (CET) Subject: |WARNING| pw105095 [PATCH 2/8] net/cnxk: add CN9K template Rx functions to build In-Reply-To: <20211213082226.3646-2-pbhagavatula@marvell.com> References: <20211213082226.3646-2-pbhagavatula@marvell.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: Message-Id: <20211213082457.3514112353B@dpdk.org> Date: Mon, 13 Dec 2021 09:24:57 +0100 (CET) X-BeenThere: test-report@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: automatic DPDK test reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: test-report-bounces@dpdk.org Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/105095 _coding style issues_ ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #145: FILE: drivers/event/cnxk/cn9k_eventdev.c:314: +#define R(name, flags) [flags] = cn9k_sso_hws_deq_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #154: FILE: drivers/event/cnxk/cn9k_eventdev.c:320: +#define R(name, flags) [flags] = cn9k_sso_hws_deq_burst_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #163: FILE: drivers/event/cnxk/cn9k_eventdev.c:326: +#define R(name, flags) [flags] = cn9k_sso_hws_deq_tmo_##name, ERROR:OPEN_BRACE: that open brace { should be on the previous line #174: FILE: drivers/event/cnxk/cn9k_eventdev.c:332: + const event_dequeue_burst_t sso_hws_deq_tmo_burst[NIX_RX_OFFLOAD_MAX] = + { ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #175: FILE: drivers/event/cnxk/cn9k_eventdev.c:333: +#define R(name, flags) [flags] = cn9k_sso_hws_deq_tmo_burst_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #185: FILE: drivers/event/cnxk/cn9k_eventdev.c:339: +#define R(name, flags) [flags] = cn9k_sso_hws_deq_ca_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #195: FILE: drivers/event/cnxk/cn9k_eventdev.c:345: +#define R(name, flags) [flags] = cn9k_sso_hws_deq_ca_burst_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #204: FILE: drivers/event/cnxk/cn9k_eventdev.c:351: +#define R(name, flags) [flags] = cn9k_sso_hws_deq_tmo_ca_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #214: FILE: drivers/event/cnxk/cn9k_eventdev.c:358: +#define R(name, flags) [flags] = cn9k_sso_hws_deq_tmo_ca_burst_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #220: FILE: drivers/event/cnxk/cn9k_eventdev.c:364: +#define R(name, flags) [flags] = cn9k_sso_hws_deq_seg_##name, ERROR:OPEN_BRACE: that open brace { should be on the previous line #229: FILE: drivers/event/cnxk/cn9k_eventdev.c:370: + const event_dequeue_burst_t sso_hws_deq_seg_burst[NIX_RX_OFFLOAD_MAX] = + { ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #230: FILE: drivers/event/cnxk/cn9k_eventdev.c:371: +#define R(name, flags) [flags] = cn9k_sso_hws_deq_seg_burst_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #236: FILE: drivers/event/cnxk/cn9k_eventdev.c:377: +#define R(name, flags) [flags] = cn9k_sso_hws_deq_tmo_seg_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #246: FILE: drivers/event/cnxk/cn9k_eventdev.c:384: +#define R(name, flags) [flags] = cn9k_sso_hws_deq_tmo_seg_burst_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #256: FILE: drivers/event/cnxk/cn9k_eventdev.c:390: +#define R(name, flags) [flags] = cn9k_sso_hws_deq_ca_seg_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #266: FILE: drivers/event/cnxk/cn9k_eventdev.c:397: +#define R(name, flags) [flags] = cn9k_sso_hws_deq_ca_seg_burst_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #277: FILE: drivers/event/cnxk/cn9k_eventdev.c:403: +#define R(name, flags) [flags] = cn9k_sso_hws_deq_tmo_ca_seg_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #287: FILE: drivers/event/cnxk/cn9k_eventdev.c:410: +#define R(name, flags) [flags] = cn9k_sso_hws_deq_tmo_ca_seg_burst_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #294: FILE: drivers/event/cnxk/cn9k_eventdev.c:417: +#define R(name, flags) [flags] = cn9k_sso_hws_dual_deq_##name, ERROR:OPEN_BRACE: that open brace { should be on the previous line #303: FILE: drivers/event/cnxk/cn9k_eventdev.c:423: + const event_dequeue_burst_t sso_hws_dual_deq_burst[NIX_RX_OFFLOAD_MAX] = + { ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #304: FILE: drivers/event/cnxk/cn9k_eventdev.c:424: +#define R(name, flags) [flags] = cn9k_sso_hws_dual_deq_burst_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #310: FILE: drivers/event/cnxk/cn9k_eventdev.c:430: +#define R(name, flags) [flags] = cn9k_sso_hws_dual_deq_tmo_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #320: FILE: drivers/event/cnxk/cn9k_eventdev.c:437: +#define R(name, flags) [flags] = cn9k_sso_hws_dual_deq_tmo_burst_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #330: FILE: drivers/event/cnxk/cn9k_eventdev.c:443: +#define R(name, flags) [flags] = cn9k_sso_hws_dual_deq_ca_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #340: FILE: drivers/event/cnxk/cn9k_eventdev.c:450: +#define R(name, flags) [flags] = cn9k_sso_hws_dual_deq_ca_burst_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #350: FILE: drivers/event/cnxk/cn9k_eventdev.c:456: +#define R(name, flags) [flags] = cn9k_sso_hws_dual_deq_tmo_ca_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #360: FILE: drivers/event/cnxk/cn9k_eventdev.c:463: +#define R(name, flags) [flags] = cn9k_sso_hws_dual_deq_tmo_ca_burst_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #369: FILE: drivers/event/cnxk/cn9k_eventdev.c:469: +#define R(name, flags) [flags] = cn9k_sso_hws_dual_deq_seg_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #380: FILE: drivers/event/cnxk/cn9k_eventdev.c:476: +#define R(name, flags) [flags] = cn9k_sso_hws_dual_deq_seg_burst_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #389: FILE: drivers/event/cnxk/cn9k_eventdev.c:482: +#define R(name, flags) [flags] = cn9k_sso_hws_dual_deq_tmo_seg_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #400: FILE: drivers/event/cnxk/cn9k_eventdev.c:489: +#define R(name, flags) [flags] = cn9k_sso_hws_dual_deq_tmo_seg_burst_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #406: FILE: drivers/event/cnxk/cn9k_eventdev.c:495: +#define R(name, flags) [flags] = cn9k_sso_hws_dual_deq_ca_seg_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #413: FILE: drivers/event/cnxk/cn9k_eventdev.c:502: +#define R(name, flags) [flags] = cn9k_sso_hws_dual_deq_ca_seg_burst_##name, ERROR:OPEN_BRACE: that open brace { should be on the previous line #419: FILE: drivers/event/cnxk/cn9k_eventdev.c:508: + const event_dequeue_t sso_hws_dual_deq_tmo_ca_seg[NIX_RX_OFFLOAD_MAX] = + { ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #420: FILE: drivers/event/cnxk/cn9k_eventdev.c:509: +#define R(name, flags) [flags] = cn9k_sso_hws_dual_deq_tmo_ca_seg_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #427: FILE: drivers/event/cnxk/cn9k_eventdev.c:516: +#define R(name, flags) [flags] = cn9k_sso_hws_dual_deq_tmo_ca_seg_burst_##name, WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided #533: FILE: drivers/event/cnxk/cn9k_worker.h:433: +#define SSO_DEQ(fn, flags) \ + uint16_t __rte_hot fn(void *port, struct rte_event *ev, \ + uint64_t timeout_ticks) \ + { \ + struct cn9k_sso_hws *ws = port; \ + \ + RTE_SET_USED(timeout_ticks); \ + if (ws->swtag_req) { \ + ws->swtag_req = 0; \ + cnxk_sso_hws_swtag_wait(ws->base + SSOW_LF_GWS_TAG); \ + return 1; \ + } \ + return cn9k_sso_hws_get_work(ws, ev, flags, ws->lookup_mem); \ + } ERROR:CODE_INDENT: code indent should use tabs where possible #538: FILE: drivers/event/cnxk/cn9k_worker.h:438: + \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #538: FILE: drivers/event/cnxk/cn9k_worker.h:438: + \$ WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided #552: FILE: drivers/event/cnxk/cn9k_worker.h:452: +#define SSO_DEQ_TMO(fn, flags) \ + uint16_t __rte_hot fn(void *port, struct rte_event *ev, \ + uint64_t timeout_ticks) \ + { \ + struct cn9k_sso_hws *ws = port; \ + uint16_t ret = 1; \ + uint64_t iter; \ + \ + if (ws->swtag_req) { \ + ws->swtag_req = 0; \ + cnxk_sso_hws_swtag_wait(ws->base + SSOW_LF_GWS_TAG); \ + return ret; \ + } \ + ret = cn9k_sso_hws_get_work(ws, ev, flags, ws->lookup_mem); \ + for (iter = 1; iter < timeout_ticks && (ret == 0); iter++) \ + ret = cn9k_sso_hws_get_work(ws, ev, flags, \ + ws->lookup_mem); \ + return ret; \ + } ERROR:CODE_INDENT: code indent should use tabs where possible #559: FILE: drivers/event/cnxk/cn9k_worker.h:459: + \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #559: FILE: drivers/event/cnxk/cn9k_worker.h:459: + \$ WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided #607: FILE: drivers/event/cnxk/cn9k_worker.h:521: +#define SSO_DUAL_DEQ(fn, flags) \ + uint16_t __rte_hot fn(void *port, struct rte_event *ev, \ + uint64_t timeout_ticks) \ + { \ + struct cn9k_sso_hws_dual *dws = port; \ + uint16_t gw; \ + \ + RTE_SET_USED(timeout_ticks); \ + if (dws->swtag_req) { \ + dws->swtag_req = 0; \ + cnxk_sso_hws_swtag_wait(dws->base[!dws->vws] + \ + SSOW_LF_GWS_TAG); \ + return 1; \ + } \ + gw = cn9k_sso_hws_dual_get_work( \ + dws->base[dws->vws], dws->base[!dws->vws], ev, flags, \ + dws->lookup_mem, dws->tstamp); \ + dws->vws = !dws->vws; \ + return gw; \ + } ERROR:CODE_INDENT: code indent should use tabs where possible #613: FILE: drivers/event/cnxk/cn9k_worker.h:527: + \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #613: FILE: drivers/event/cnxk/cn9k_worker.h:527: + \$ WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided #633: FILE: drivers/event/cnxk/cn9k_worker.h:547: +#define SSO_DUAL_DEQ_TMO(fn, flags) \ + uint16_t __rte_hot fn(void *port, struct rte_event *ev, \ + uint64_t timeout_ticks) \ + { \ + struct cn9k_sso_hws_dual *dws = port; \ + uint16_t ret = 1; \ + uint64_t iter; \ + \ + if (dws->swtag_req) { \ + dws->swtag_req = 0; \ + cnxk_sso_hws_swtag_wait(dws->base[!dws->vws] + \ + SSOW_LF_GWS_TAG); \ + return ret; \ + } \ + ret = cn9k_sso_hws_dual_get_work( \ + dws->base[dws->vws], dws->base[!dws->vws], ev, flags, \ + dws->lookup_mem, dws->tstamp); \ + dws->vws = !dws->vws; \ + for (iter = 1; iter < timeout_ticks && (ret == 0); iter++) { \ + ret = cn9k_sso_hws_dual_get_work( \ + dws->base[dws->vws], dws->base[!dws->vws], ev, \ + flags, dws->lookup_mem, dws->tstamp); \ + dws->vws = !dws->vws; \ + } \ + return ret; \ + } ERROR:CODE_INDENT: code indent should use tabs where possible #640: FILE: drivers/event/cnxk/cn9k_worker.h:554: + \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #640: FILE: drivers/event/cnxk/cn9k_worker.h:554: + \$ WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided #667: FILE: drivers/event/cnxk/cn9k_worker.h:581: +#define SSO_CMN_DEQ_BURST(fnb, fn, flags) \ + uint16_t __rte_hot fnb(void *port, struct rte_event ev[], \ + uint16_t nb_events, uint64_t timeout_ticks) \ + { \ + RTE_SET_USED(nb_events); \ + return fn(port, ev, timeout_ticks); \ + } WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided #675: FILE: drivers/event/cnxk/cn9k_worker.h:589: +#define SSO_CMN_DEQ_SEG_BURST(fnb, fn, flags) \ + uint16_t __rte_hot fnb(void *port, struct rte_event ev[], \ + uint16_t nb_events, uint64_t timeout_ticks) \ + { \ + RTE_SET_USED(nb_events); \ + return fn(port, ev, timeout_ticks); \ + } ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #1751: FILE: drivers/net/cnxk/cn9k_rx.h:811: +#define NIX_RX_FASTPATH_MODES_0_15 \ + R(no_offload, NIX_RX_OFFLOAD_NONE) \ + R(rss, RSS_F) \ + R(ptype, PTYPE_F) \ + R(ptype_rss, PTYPE_F | RSS_F) \ + R(cksum, CKSUM_F) \ + R(cksum_rss, CKSUM_F | RSS_F) \ + R(cksum_ptype, CKSUM_F | PTYPE_F) \ + R(cksum_ptype_rss, CKSUM_F | PTYPE_F | RSS_F) \ + R(mark, MARK_F) \ + R(mark_rss, MARK_F | RSS_F) \ + R(mark_ptype, MARK_F | PTYPE_F) \ + R(mark_ptype_rss, MARK_F | PTYPE_F | RSS_F) \ + R(mark_cksum, MARK_F | CKSUM_F) \ + R(mark_cksum_rss, MARK_F | CKSUM_F | RSS_F) \ + R(mark_cksum_ptype, MARK_F | CKSUM_F | PTYPE_F) \ + R(mark_cksum_ptype_rss, MARK_F | CKSUM_F | PTYPE_F | RSS_F) ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #1769: FILE: drivers/net/cnxk/cn9k_rx.h:829: +#define NIX_RX_FASTPATH_MODES_16_31 \ + R(ts, TS_F) \ + R(ts_rss, TS_F | RSS_F) \ + R(ts_ptype, TS_F | PTYPE_F) \ + R(ts_ptype_rss, TS_F | PTYPE_F | RSS_F) \ + R(ts_cksum, TS_F | CKSUM_F) \ + R(ts_cksum_rss, TS_F | CKSUM_F | RSS_F) \ + R(ts_cksum_ptype, TS_F | CKSUM_F | PTYPE_F) \ + R(ts_cksum_ptype_rss, TS_F | CKSUM_F | PTYPE_F | RSS_F) \ + R(ts_mark, TS_F | MARK_F) \ + R(ts_mark_rss, TS_F | MARK_F | RSS_F) \ + R(ts_mark_ptype, TS_F | MARK_F | PTYPE_F) \ + R(ts_mark_ptype_rss, TS_F | MARK_F | PTYPE_F | RSS_F) \ + R(ts_mark_cksum, TS_F | MARK_F | CKSUM_F) \ + R(ts_mark_cksum_rss, TS_F | MARK_F | CKSUM_F | RSS_F) \ + R(ts_mark_cksum_ptype, TS_F | MARK_F | CKSUM_F | PTYPE_F) \ + R(ts_mark_cksum_ptype_rss, TS_F | MARK_F | CKSUM_F | PTYPE_F | RSS_F) ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #1787: FILE: drivers/net/cnxk/cn9k_rx.h:847: +#define NIX_RX_FASTPATH_MODES_32_47 \ + R(vlan, RX_VLAN_F) \ + R(vlan_rss, RX_VLAN_F | RSS_F) \ + R(vlan_ptype, RX_VLAN_F | PTYPE_F) \ + R(vlan_ptype_rss, RX_VLAN_F | PTYPE_F | RSS_F) \ + R(vlan_cksum, RX_VLAN_F | CKSUM_F) \ + R(vlan_cksum_rss, RX_VLAN_F | CKSUM_F | RSS_F) \ + R(vlan_cksum_ptype, RX_VLAN_F | CKSUM_F | PTYPE_F) \ + R(vlan_cksum_ptype_rss, RX_VLAN_F | CKSUM_F | PTYPE_F | RSS_F) \ + R(vlan_mark, RX_VLAN_F | MARK_F) \ + R(vlan_mark_rss, RX_VLAN_F | MARK_F | RSS_F) \ + R(vlan_mark_ptype, RX_VLAN_F | MARK_F | PTYPE_F) \ + R(vlan_mark_ptype_rss, RX_VLAN_F | MARK_F | PTYPE_F | RSS_F) \ + R(vlan_mark_cksum, RX_VLAN_F | MARK_F | CKSUM_F) \ + R(vlan_mark_cksum_rss, RX_VLAN_F | MARK_F | CKSUM_F | RSS_F) \ + R(vlan_mark_cksum_ptype, RX_VLAN_F | MARK_F | CKSUM_F | PTYPE_F) \ + R(vlan_mark_cksum_ptype_rss, \ + RX_VLAN_F | MARK_F | CKSUM_F | PTYPE_F | RSS_F) ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #1806: FILE: drivers/net/cnxk/cn9k_rx.h:866: +#define NIX_RX_FASTPATH_MODES_48_63 \ + R(vlan_ts, RX_VLAN_F | TS_F) \ + R(vlan_ts_rss, RX_VLAN_F | TS_F | RSS_F) \ + R(vlan_ts_ptype, RX_VLAN_F | TS_F | PTYPE_F) \ + R(vlan_ts_ptype_rss, RX_VLAN_F | TS_F | PTYPE_F | RSS_F) \ + R(vlan_ts_cksum, RX_VLAN_F | TS_F | CKSUM_F) \ + R(vlan_ts_cksum_rss, RX_VLAN_F | TS_F | CKSUM_F | RSS_F) \ + R(vlan_ts_cksum_ptype, RX_VLAN_F | TS_F | CKSUM_F | PTYPE_F) \ + R(vlan_ts_cksum_ptype_rss, \ + RX_VLAN_F | TS_F | CKSUM_F | PTYPE_F | RSS_F) \ + R(vlan_ts_mark, RX_VLAN_F | TS_F | MARK_F) \ + R(vlan_ts_mark_rss, RX_VLAN_F | TS_F | MARK_F | RSS_F) \ + R(vlan_ts_mark_ptype, RX_VLAN_F | TS_F | MARK_F | PTYPE_F) \ + R(vlan_ts_mark_ptype_rss, RX_VLAN_F | TS_F | MARK_F | PTYPE_F | RSS_F) \ + R(vlan_ts_mark_cksum, RX_VLAN_F | TS_F | MARK_F | CKSUM_F) \ + R(vlan_ts_mark_cksum_rss, RX_VLAN_F | TS_F | MARK_F | CKSUM_F | RSS_F) \ + R(vlan_ts_mark_cksum_ptype, \ + RX_VLAN_F | TS_F | MARK_F | CKSUM_F | PTYPE_F) \ + R(vlan_ts_mark_cksum_ptype_rss, \ + RX_VLAN_F | TS_F | MARK_F | CKSUM_F | PTYPE_F | RSS_F) ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #1827: FILE: drivers/net/cnxk/cn9k_rx.h:887: +#define NIX_RX_FASTPATH_MODES_64_79 \ + R(sec, R_SEC_F) \ + R(sec_rss, R_SEC_F | RSS_F) \ + R(sec_ptype, R_SEC_F | PTYPE_F) \ + R(sec_ptype_rss, R_SEC_F | PTYPE_F | RSS_F) \ + R(sec_cksum, R_SEC_F | CKSUM_F) \ + R(sec_cksum_rss, R_SEC_F | CKSUM_F | RSS_F) \ + R(sec_cksum_ptype, R_SEC_F | CKSUM_F | PTYPE_F) \ + R(sec_cksum_ptype_rss, R_SEC_F | CKSUM_F | PTYPE_F | RSS_F) \ + R(sec_mark, R_SEC_F | MARK_F) \ + R(sec_mark_rss, R_SEC_F | MARK_F | RSS_F) \ + R(sec_mark_ptype, R_SEC_F | MARK_F | PTYPE_F) \ + R(sec_mark_ptype_rss, R_SEC_F | MARK_F | PTYPE_F | RSS_F) \ + R(sec_mark_cksum, R_SEC_F | MARK_F | CKSUM_F) \ + R(sec_mark_cksum_rss, R_SEC_F | MARK_F | CKSUM_F | RSS_F) \ + R(sec_mark_cksum_ptype, R_SEC_F | MARK_F | CKSUM_F | PTYPE_F) \ + R(sec_mark_cksum_ptype_rss, \ + R_SEC_F | MARK_F | CKSUM_F | PTYPE_F | RSS_F) ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #1846: FILE: drivers/net/cnxk/cn9k_rx.h:906: +#define NIX_RX_FASTPATH_MODES_80_95 \ + R(sec_ts, R_SEC_F | TS_F) \ + R(sec_ts_rss, R_SEC_F | TS_F | RSS_F) \ + R(sec_ts_ptype, R_SEC_F | TS_F | PTYPE_F) \ + R(sec_ts_ptype_rss, R_SEC_F | TS_F | PTYPE_F | RSS_F) \ + R(sec_ts_cksum, R_SEC_F | TS_F | CKSUM_F) \ + R(sec_ts_cksum_rss, R_SEC_F | TS_F | CKSUM_F | RSS_F) \ + R(sec_ts_cksum_ptype, R_SEC_F | TS_F | CKSUM_F | PTYPE_F) \ + R(sec_ts_cksum_ptype_rss, R_SEC_F | TS_F | CKSUM_F | PTYPE_F | RSS_F) \ + R(sec_ts_mark, R_SEC_F | TS_F | MARK_F) \ + R(sec_ts_mark_rss, R_SEC_F | TS_F | MARK_F | RSS_F) \ + R(sec_ts_mark_ptype, R_SEC_F | TS_F | MARK_F | PTYPE_F) \ + R(sec_ts_mark_ptype_rss, R_SEC_F | TS_F | MARK_F | PTYPE_F | RSS_F) \ + R(sec_ts_mark_cksum, R_SEC_F | TS_F | MARK_F | CKSUM_F) \ + R(sec_ts_mark_cksum_rss, R_SEC_F | TS_F | MARK_F | CKSUM_F | RSS_F) \ + R(sec_ts_mark_cksum_ptype, \ + R_SEC_F | TS_F | MARK_F | CKSUM_F | PTYPE_F) \ + R(sec_ts_mark_cksum_ptype_rss, \ + R_SEC_F | TS_F | MARK_F | CKSUM_F | PTYPE_F | RSS_F) ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #1866: FILE: drivers/net/cnxk/cn9k_rx.h:926: +#define NIX_RX_FASTPATH_MODES_96_111 \ + R(sec_vlan, R_SEC_F | RX_VLAN_F) \ + R(sec_vlan_rss, R_SEC_F | RX_VLAN_F | RSS_F) \ + R(sec_vlan_ptype, R_SEC_F | RX_VLAN_F | PTYPE_F) \ + R(sec_vlan_ptype_rss, R_SEC_F | RX_VLAN_F | PTYPE_F | RSS_F) \ + R(sec_vlan_cksum, R_SEC_F | RX_VLAN_F | CKSUM_F) \ + R(sec_vlan_cksum_rss, R_SEC_F | RX_VLAN_F | CKSUM_F | RSS_F) \ + R(sec_vlan_cksum_ptype, R_SEC_F | RX_VLAN_F | CKSUM_F | PTYPE_F) \ + R(sec_vlan_cksum_ptype_rss, \ + R_SEC_F | RX_VLAN_F | CKSUM_F | PTYPE_F | RSS_F) \ + R(sec_vlan_mark, R_SEC_F | RX_VLAN_F | MARK_F) \ + R(sec_vlan_mark_rss, R_SEC_F | RX_VLAN_F | MARK_F | RSS_F) \ + R(sec_vlan_mark_ptype, R_SEC_F | RX_VLAN_F | MARK_F | PTYPE_F) \ + R(sec_vlan_mark_ptype_rss, \ + R_SEC_F | RX_VLAN_F | MARK_F | PTYPE_F | RSS_F) \ + R(sec_vlan_mark_cksum, R_SEC_F | RX_VLAN_F | MARK_F | CKSUM_F) \ + R(sec_vlan_mark_cksum_rss, \ + R_SEC_F | RX_VLAN_F | MARK_F | CKSUM_F | RSS_F) \ + R(sec_vlan_mark_cksum_ptype, \ + R_SEC_F | RX_VLAN_F | MARK_F | CKSUM_F | PTYPE_F) \ + R(sec_vlan_mark_cksum_ptype_rss, \ + R_SEC_F | RX_VLAN_F | MARK_F | CKSUM_F | PTYPE_F | RSS_F) ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #1889: FILE: drivers/net/cnxk/cn9k_rx.h:949: +#define NIX_RX_FASTPATH_MODES_112_127 \ + R(sec_vlan_ts, R_SEC_F | RX_VLAN_F | TS_F) \ + R(sec_vlan_ts_rss, R_SEC_F | RX_VLAN_F | TS_F | RSS_F) \ + R(sec_vlan_ts_ptype, R_SEC_F | RX_VLAN_F | TS_F | PTYPE_F) \ + R(sec_vlan_ts_ptype_rss, R_SEC_F | RX_VLAN_F | TS_F | PTYPE_F | RSS_F) \ + R(sec_vlan_ts_cksum, R_SEC_F | RX_VLAN_F | TS_F | CKSUM_F) \ + R(sec_vlan_ts_cksum_rss, R_SEC_F | RX_VLAN_F | TS_F | CKSUM_F | RSS_F) \ + R(sec_vlan_ts_cksum_ptype, \ + R_SEC_F | RX_VLAN_F | TS_F | CKSUM_F | PTYPE_F) \ + R(sec_vlan_ts_cksum_ptype_rss, \ + R_SEC_F | RX_VLAN_F | TS_F | CKSUM_F | PTYPE_F | RSS_F) \ + R(sec_vlan_ts_mark, R_SEC_F | RX_VLAN_F | TS_F | MARK_F) \ + R(sec_vlan_ts_mark_rss, R_SEC_F | RX_VLAN_F | TS_F | MARK_F | RSS_F) \ + R(sec_vlan_ts_mark_ptype, \ + R_SEC_F | RX_VLAN_F | TS_F | MARK_F | PTYPE_F) \ + R(sec_vlan_ts_mark_ptype_rss, \ + R_SEC_F | RX_VLAN_F | TS_F | MARK_F | PTYPE_F | RSS_F) \ + R(sec_vlan_ts_mark_cksum, \ + R_SEC_F | RX_VLAN_F | TS_F | MARK_F | CKSUM_F) \ + R(sec_vlan_ts_mark_cksum_rss, \ + R_SEC_F | RX_VLAN_F | TS_F | MARK_F | CKSUM_F | RSS_F) \ + R(sec_vlan_ts_mark_cksum_ptype, \ + R_SEC_F | RX_VLAN_F | TS_F | MARK_F | CKSUM_F | PTYPE_F) \ + R(sec_vlan_ts_mark_cksum_ptype_rss, \ + R_SEC_F | RX_VLAN_F | TS_F | MARK_F | CKSUM_F | PTYPE_F | RSS_F) ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #1915: FILE: drivers/net/cnxk/cn9k_rx.h:975: +#define NIX_RX_FASTPATH_MODES \ + NIX_RX_FASTPATH_MODES_0_15 \ + NIX_RX_FASTPATH_MODES_16_31 \ + NIX_RX_FASTPATH_MODES_32_47 \ + NIX_RX_FASTPATH_MODES_48_63 \ + NIX_RX_FASTPATH_MODES_64_79 \ + NIX_RX_FASTPATH_MODES_80_95 \ + NIX_RX_FASTPATH_MODES_96_111 \ + NIX_RX_FASTPATH_MODES_112_127 WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon #1925: FILE: drivers/net/cnxk/cn9k_rx.h:985: +#define R(name, flags) \ uint16_t __rte_noinline __rte_hot cn9k_nix_recv_pkts_##name( \ void *rx_queue, struct rte_mbuf **rx_pkts, uint16_t pkts); \ + \ uint16_t __rte_noinline __rte_hot cn9k_nix_recv_pkts_mseg_##name( \ void *rx_queue, struct rte_mbuf **rx_pkts, uint16_t pkts); \ + \ uint16_t __rte_noinline __rte_hot cn9k_nix_recv_pkts_vec_##name( \ void *rx_queue, struct rte_mbuf **rx_pkts, uint16_t pkts); \ + \ uint16_t __rte_noinline __rte_hot cn9k_nix_recv_pkts_vec_mseg_##name( \ void *rx_queue, struct rte_mbuf **rx_pkts, uint16_t pkts); ERROR:CODE_INDENT: code indent should use tabs where possible #1929: FILE: drivers/net/cnxk/cn9k_rx.h:988: + \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #1929: FILE: drivers/net/cnxk/cn9k_rx.h:988: + \$ ERROR:CODE_INDENT: code indent should use tabs where possible #1933: FILE: drivers/net/cnxk/cn9k_rx.h:991: + \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #1933: FILE: drivers/net/cnxk/cn9k_rx.h:991: + \$ ERROR:CODE_INDENT: code indent should use tabs where possible #1937: FILE: drivers/net/cnxk/cn9k_rx.h:994: + \$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #1937: FILE: drivers/net/cnxk/cn9k_rx.h:994: + \$ WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided #1944: FILE: drivers/net/cnxk/cn9k_rx.h:1001: +#define NIX_RX_RECV(fn, flags) \ + uint16_t __rte_noinline __rte_hot fn( \ + void *rx_queue, struct rte_mbuf **rx_pkts, uint16_t pkts) \ + { \ + return cn9k_nix_recv_pkts(rx_queue, rx_pkts, pkts, (flags)); \ + } CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'flags' may be better as '(flags)' to avoid precedence issues #1951: FILE: drivers/net/cnxk/cn9k_rx.h:1008: +#define NIX_RX_RECV_MSEG(fn, flags) NIX_RX_RECV(fn, flags | NIX_RX_MULTI_SEG_F) WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided #1953: FILE: drivers/net/cnxk/cn9k_rx.h:1010: +#define NIX_RX_RECV_VEC(fn, flags) \ + uint16_t __rte_noinline __rte_hot fn( \ + void *rx_queue, struct rte_mbuf **rx_pkts, uint16_t pkts) \ + { \ + return cn9k_nix_recv_pkts_vector(rx_queue, rx_pkts, pkts, \ + (flags)); \ + } CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'flags' may be better as '(flags)' to avoid precedence issues #1961: FILE: drivers/net/cnxk/cn9k_rx.h:1018: +#define NIX_RX_RECV_VEC_MSEG(fn, flags) \ + NIX_RX_RECV_VEC(fn, flags | NIX_RX_MULTI_SEG_F) ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #2020: FILE: drivers/net/cnxk/cn9k_rx_select.c:27: +#define R(name, flags) [flags] = cn9k_nix_recv_pkts_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #2027: FILE: drivers/net/cnxk/cn9k_rx_select.c:34: +#define R(name, flags) [flags] = cn9k_nix_recv_pkts_mseg_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #2034: FILE: drivers/net/cnxk/cn9k_rx_select.c:41: +#define R(name, flags) [flags] = cn9k_nix_recv_pkts_vec_##name, ERROR:BRACKET_SPACE: space prohibited before open square bracket '[' #2041: FILE: drivers/net/cnxk/cn9k_rx_select.c:48: +#define R(name, flags) [flags] = cn9k_nix_recv_pkts_vec_mseg_##name, total: 56 errors, 16 warnings, 2 checks, 1433 lines checked