From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by dpdk.space (Postfix) with ESMTP id E7833A05F9 for ; Wed, 19 Jun 2019 17:56:31 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 1CE6C1C5A8; Wed, 19 Jun 2019 17:56:17 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 1017) id CDD521D125; Wed, 19 Jun 2019 17:56:15 +0200 (CEST) In-Reply-To: <58113c06486f2fa347e7921beb45f97eef74db66.1560958308.git.xuanziyang2@huawei.com> References: <58113c06486f2fa347e7921beb45f97eef74db66.1560958308.git.xuanziyang2@huawei.com> To: test-report@dpdk.org Cc: Ziyang Xuan Message-Id: <20190619155615.CDD521D125@dpdk.org> Date: Wed, 19 Jun 2019 17:56:15 +0200 (CEST) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw55040 [PATCH v5 09/15] net/hinic/base: add context and work queue support X-BeenThere: test-report@dpdk.org X-Mailman-Version: 2.1.15 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 Sender: "test-report" Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/55040 _coding style issues_ CHECK:MACRO_ARG_REUSE: Macro argument reuse 'max_sqs' - possible side-effects? #973: FILE: drivers/net/hinic/base/hinic_pmd_nicio.h:32: +#define RQ_CTXT_OFFSET(max_sqs, max_rqs, q_id) \ + (((max_rqs) + (max_sqs)) * TSO_LRO_CTXT_SIZE + \ + (max_sqs) * Q_CTXT_SIZE + (q_id) * Q_CTXT_SIZE) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects? #1407: FILE: drivers/net/hinic/base/hinic_pmd_wq.h:10: +#define WQ_SIZE(wq) (u32)((u64)(wq)->q_depth * (wq)->wqebb_size) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects? #1409: FILE: drivers/net/hinic/base/hinic_pmd_wq.h:12: +#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? #1412: FILE: drivers/net/hinic/base/hinic_pmd_wq.h:15: +#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? #1417: FILE: drivers/net/hinic/base/hinic_pmd_wq.h:20: +#define WQ_PAGE_ADDR(wq, idx) \ + (u8 *)(*(u64 *)((u64)((wq)->shadow_block_vaddr) + \ + (WQE_PAGE_NUM(wq, idx) << WQ_PAGE_ADDR_SIZE_SHIFT))) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects? #1433: FILE: drivers/net/hinic/base/hinic_pmd_wq.h:36: +#define WQ_BASE_VADDR(wqs, wq) \ + (u64 *)(((u64)((wqs)->page_vaddr[(wq)->page_idx])) \ + + (wq)->block_idx * WQ_BLOCK_SIZE) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects? #1437: FILE: drivers/net/hinic/base/hinic_pmd_wq.h:40: +#define WQ_BASE_PADDR(wqs, wq) (((wqs)->page_paddr[(wq)->page_idx]) \ + + (u64)(wq)->block_idx * WQ_BLOCK_SIZE) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects? #1440: FILE: drivers/net/hinic/base/hinic_pmd_wq.h:43: +#define WQ_BASE_ADDR(wqs, wq) \ + (u64 *)(((u64)((wqs)->shadow_page_vaddr[(wq)->page_idx])) \ + + (wq)->block_idx * WQ_BLOCK_SIZE) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects? #1458: FILE: drivers/net/hinic/base/hinic_pmd_wq.h:61: +#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? #1462: FILE: drivers/net/hinic/base/hinic_pmd_wq.h:65: +#define WQE_IN_RANGE(wqe, start, end) \ + (((unsigned long)(wqe) >= (unsigned long)(start)) && \ + ((unsigned long)(wqe) < (unsigned long)(end))) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects? #1469: FILE: drivers/net/hinic/base/hinic_pmd_wq.h:72: +#define WQ_WQE_ADDR(wq, idx) ((void *)((u64)((wq)->queue_buf_vaddr) + \ + ((idx) << (wq)->wqebb_shift))) total: 0 errors, 0 warnings, 11 checks, 1475 lines checked