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 1C95AA034D for ; Wed, 29 Dec 2021 14:40:42 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 072E54113D; Wed, 29 Dec 2021 14:40:42 +0100 (CET) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id AD2A2410FF for ; Wed, 29 Dec 2021 14:40:40 +0100 (CET) Received: by dpdk.org (Postfix, from userid 65534) id A8C49120D7B; Wed, 29 Dec 2021 14:40:40 +0100 (CET) Subject: |WARNING| pw105476 [PATCH v5 07/26] net/spnic: add interface handling cmdq message In-Reply-To: <9bd5f153c85f940edfd21cd1895ca784d0a94389.1640783513.git.songyl@ramaxel.com> References: <9bd5f153c85f940edfd21cd1895ca784d0a94389.1640783513.git.songyl@ramaxel.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: Yanling Song Message-Id: <20211229134040.A8C49120D7B@dpdk.org> Date: Wed, 29 Dec 2021 14:40:40 +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/105476 _coding style issues_ CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects? #210: FILE: drivers/net/spnic/base/spnic_cmdq.c:145: +#define WQE_NUM_WQEBBS(wqe_size, wq) \ + ((u16)(RTE_ALIGN((u32)(wqe_size), (wq)->wqebb_size) / (wq)->wqebb_size)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'cmdq' - possible side-effects? #213: FILE: drivers/net/spnic/base/spnic_cmdq.c:148: +#define cmdq_to_cmdqs(cmdq) container_of((cmdq) - (cmdq)->cmdq_type, \ + struct spnic_cmdqs, cmdq[0]) WARNING:TYPO_SPELLING: 'wqs' may be misspelled - perhaps 'was'? #708: FILE: drivers/net/spnic/base/spnic_cmdq.c:723: + size_t saved_wqs_size; WARNING:TYPO_SPELLING: 'wqs' may be misspelled - perhaps 'was'? #716: FILE: drivers/net/spnic/base/spnic_cmdq.c:734: + saved_wqs_size = SPNIC_MAX_CMDQ_TYPES * sizeof(struct spnic_wq); WARNING:TYPO_SPELLING: 'wqs' may be misspelled - perhaps 'was'? #717: FILE: drivers/net/spnic/base/spnic_cmdq.c:735: + cmdqs->saved_wqs = rte_zmalloc(NULL, saved_wqs_size, 0); WARNING:TYPO_SPELLING: 'wqs' may be misspelled - perhaps 'was'? #717: FILE: drivers/net/spnic/base/spnic_cmdq.c:735: + cmdqs->saved_wqs = rte_zmalloc(NULL, saved_wqs_size, 0); WARNING:TYPO_SPELLING: 'wqs' may be misspelled - perhaps 'was'? #718: FILE: drivers/net/spnic/base/spnic_cmdq.c:736: + if (!cmdqs->saved_wqs) { WARNING:TYPO_SPELLING: 'wqs' may be misspelled - perhaps 'was'? #719: FILE: drivers/net/spnic/base/spnic_cmdq.c:737: + PMD_DRV_LOG(ERR, "Allocate saved wqs failed"); WARNING:TYPO_SPELLING: 'wqs' may be misspelled - perhaps 'was'? #721: FILE: drivers/net/spnic/base/spnic_cmdq.c:739: + goto alloc_wqs_err; WARNING:TYPO_SPELLING: 'wqs' may be misspelled - perhaps 'was'? #731: FILE: drivers/net/spnic/base/spnic_cmdq.c:755: + err = spnic_cmdq_alloc(cmdqs->saved_wqs, hwdev, SPNIC_MAX_CMDQ_TYPES, WARNING:TYPO_SPELLING: 'wqs' may be misspelled - perhaps 'was'? #756: FILE: drivers/net/spnic/base/spnic_cmdq.c:787: + spnic_cmdq_free(cmdqs->saved_wqs, SPNIC_MAX_CMDQ_TYPES); WARNING:TYPO_SPELLING: 'wqs' may be misspelled - perhaps 'was'? #762: FILE: drivers/net/spnic/base/spnic_cmdq.c:793: + rte_free(cmdqs->saved_wqs); WARNING:TYPO_SPELLING: 'wqs' may be misspelled - perhaps 'was'? #764: FILE: drivers/net/spnic/base/spnic_cmdq.c:795: +alloc_wqs_err: WARNING:TYPO_SPELLING: 'wqs' may be misspelled - perhaps 'was'? #772: FILE: drivers/net/spnic/base/spnic_cmdq.c:811: + spnic_cmdq_free(cmdqs->saved_wqs, SPNIC_MAX_CMDQ_TYPES); WARNING:TYPO_SPELLING: 'WQS' may be misspelled - perhaps 'WAS'? #1151: FILE: drivers/net/spnic/base/spnic_wq.h:12: +#define WQS_BLOCKS_PER_PAGE 4 CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects? #1153: FILE: drivers/net/spnic/base/spnic_wq.h:14: +#define WQ_SIZE(wq) ((u32)((u64)(wq)->q_depth * (wq)->wqebb_size)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects? #1155: FILE: drivers/net/spnic/base/spnic_wq.h:16: +#define WQE_PAGE_NUM(wq, idx) (((idx) >> ((wq)->wqebbs_per_page_shift)) & \ + ((wq)->num_q_pages - 1)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects? #1158: FILE: drivers/net/spnic/base/spnic_wq.h:19: +#define WQE_PAGE_OFF(wq, idx) ((u64)((wq)->wqebb_size) * \ + ((idx) & ((wq)->num_wqebbs_per_page - 1))) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects? #1163: FILE: drivers/net/spnic/base/spnic_wq.h:24: +#define WQ_PAGE_ADDR(wq, idx) \ + ((u8 *)(*(u64 *)((u64)((wq)->shadow_block_vaddr) + \ + (WQE_PAGE_NUM(wq, idx) << WQ_PAGE_ADDR_SIZE_SHIFT)))) WARNING:TYPO_SPELLING: 'WQS' may be misspelled - perhaps 'WAS'? #1168: FILE: drivers/net/spnic/base/spnic_wq.h:29: +#define WQS_PAGE_SIZE (WQS_BLOCKS_PER_PAGE * WQ_BLOCK_SIZE) WARNING:TYPO_SPELLING: 'WQS' may be misspelled - perhaps 'WAS'? #1168: FILE: drivers/net/spnic/base/spnic_wq.h:29: +#define WQS_PAGE_SIZE (WQS_BLOCKS_PER_PAGE * WQ_BLOCK_SIZE) WARNING:TYPO_SPELLING: 'wqs' may be misspelled - perhaps 'was'? #1179: FILE: drivers/net/spnic/base/spnic_wq.h:40: +#define WQ_BASE_VADDR(wqs, wq) \ CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects? #1179: FILE: drivers/net/spnic/base/spnic_wq.h:40: +#define WQ_BASE_VADDR(wqs, wq) \ + ((u64 *)(((u64)((wqs)->page_vaddr[(wq)->page_idx])) \ + + (wq)->block_idx * WQ_BLOCK_SIZE)) WARNING:TYPO_SPELLING: 'wqs' may be misspelled - perhaps 'was'? #1180: FILE: drivers/net/spnic/base/spnic_wq.h:41: + ((u64 *)(((u64)((wqs)->page_vaddr[(wq)->page_idx])) \ WARNING:TYPO_SPELLING: 'wqs' may be misspelled - perhaps 'was'? #1183: FILE: drivers/net/spnic/base/spnic_wq.h:44: +#define WQ_BASE_PADDR(wqs, wq) (((wqs)->page_paddr[(wq)->page_idx]) \ WARNING:TYPO_SPELLING: 'wqs' may be misspelled - perhaps 'was'? #1183: FILE: drivers/net/spnic/base/spnic_wq.h:44: +#define WQ_BASE_PADDR(wqs, wq) (((wqs)->page_paddr[(wq)->page_idx]) \ CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects? #1183: FILE: drivers/net/spnic/base/spnic_wq.h:44: +#define WQ_BASE_PADDR(wqs, wq) (((wqs)->page_paddr[(wq)->page_idx]) \ + + (u64)(wq)->block_idx * WQ_BLOCK_SIZE) WARNING:TYPO_SPELLING: 'wqs' may be misspelled - perhaps 'was'? #1186: FILE: drivers/net/spnic/base/spnic_wq.h:47: +#define WQ_BASE_ADDR(wqs, wq) \ CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects? #1186: FILE: drivers/net/spnic/base/spnic_wq.h:47: +#define WQ_BASE_ADDR(wqs, wq) \ + ((u64 *)(((u64)((wqs)->shadow_page_vaddr[(wq)->page_idx])) \ + + (wq)->block_idx * WQ_BLOCK_SIZE)) WARNING:TYPO_SPELLING: 'wqs' may be misspelled - perhaps 'was'? #1187: FILE: drivers/net/spnic/base/spnic_wq.h:48: + ((u64 *)(((u64)((wqs)->shadow_page_vaddr[(wq)->page_idx])) \ CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects? #1197: FILE: drivers/net/spnic/base/spnic_wq.h:65: +#define WQE_SHADOW_PAGE(wq, wqe) \ + ((u16)(((unsigned long)(wqe) - (unsigned long)(wq)->shadow_wqe) \ + / (wq)->max_wqe_size)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wqe' - possible side-effects? #1201: FILE: drivers/net/spnic/base/spnic_wq.h:69: +#define WQE_IN_RANGE(wqe, start, end) \ + (((unsigned long)(wqe) >= (unsigned long)(start)) && \ + ((unsigned long)(wqe) < (unsigned long)(end))) WARNING:TYPO_SPELLING: 'wqs' may be misspelled - perhaps 'was'? #1205: FILE: drivers/net/spnic/base/spnic_wq.h:73: +#define WQ_NUM_PAGES(num_wqs) \ WARNING:TYPO_SPELLING: 'wqs' may be misspelled - perhaps 'was'? #1206: FILE: drivers/net/spnic/base/spnic_wq.h:74: + (RTE_ALIGN((u32)(num_wqs), WQS_BLOCKS_PER_PAGE) / WQS_BLOCKS_PER_PAGE) WARNING:TYPO_SPELLING: 'WQS' may be misspelled - perhaps 'WAS'? #1206: FILE: drivers/net/spnic/base/spnic_wq.h:74: + (RTE_ALIGN((u32)(num_wqs), WQS_BLOCKS_PER_PAGE) / WQS_BLOCKS_PER_PAGE) WARNING:TYPO_SPELLING: 'WQS' may be misspelled - perhaps 'WAS'? #1206: FILE: drivers/net/spnic/base/spnic_wq.h:74: + (RTE_ALIGN((u32)(num_wqs), WQS_BLOCKS_PER_PAGE) / WQS_BLOCKS_PER_PAGE) total: 0 errors, 25 warnings, 11 checks, 1116 lines checked