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 60536A0096 for ; Thu, 6 Jun 2019 13:06:35 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 542EA1B996; Thu, 6 Jun 2019 13:06:35 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 1017) id F0BA41B99A; Thu, 6 Jun 2019 13:06:33 +0200 (CEST) In-Reply-To: <729eff13dd2b38369c2e4f7361d780d2ba624635.1559818024.git.xuanziyang2@huawei.com> References: <729eff13dd2b38369c2e4f7361d780d2ba624635.1559818024.git.xuanziyang2@huawei.com> To: test-report@dpdk.org Cc: Ziyang Xuan Message-Id: <20190606110633.F0BA41B99A@dpdk.org> Date: Thu, 6 Jun 2019 13:06:33 +0200 (CEST) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw54478 [PATCH v4 07/11] net/hinic/base: add various headers 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/54478 _coding style issues_ CHECK:MACRO_ARG_REUSE: Macro argument reuse 'Y' - possible side-effects? #115: FILE: drivers/net/hinic/base/hinic_compat.h:71: +#define DIV_ROUND_UP(X, Y) (((X) + ((Y) - 1)) / (Y)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'Y' - possible side-effects? #118: FILE: drivers/net/hinic/base/hinic_compat.h:74: +#define ROUND_UP(X, Y) (DIV_ROUND_UP(X, Y) * (Y)) CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements #132: FILE: drivers/net/hinic/base/hinic_compat.h:88: + HINIC_DRIVER_NAME": " fmt " ", ##args) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects? #1059: FILE: drivers/net/hinic/base/hinic_qe_def.h:281: +#define WQ_SIZE(wq) (u32)((u64)(wq)->q_depth * (wq)->wqebb_size) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects? #1061: FILE: drivers/net/hinic/base/hinic_qe_def.h:283: +#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? #1064: FILE: drivers/net/hinic/base/hinic_qe_def.h:286: +#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? #1069: FILE: drivers/net/hinic/base/hinic_qe_def.h:291: +#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? #1085: FILE: drivers/net/hinic/base/hinic_qe_def.h:307: +#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? #1089: FILE: drivers/net/hinic/base/hinic_qe_def.h:311: +#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? #1092: FILE: drivers/net/hinic/base/hinic_qe_def.h:314: +#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? #1110: FILE: drivers/net/hinic/base/hinic_qe_def.h:332: +#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? #1114: FILE: drivers/net/hinic/base/hinic_qe_def.h:336: +#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 'rxq' - possible side-effects? #1354: FILE: drivers/net/hinic/hinic_pmd_rx.h:12: +#define HINIC_GET_RQ_LOCAL_CI(rxq) \ + (((rxq)->wq->cons_idx) & HINIC_GET_RQ_WQE_MASK(rxq)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'rxq' - possible side-effects? #1357: FILE: drivers/net/hinic/hinic_pmd_rx.h:15: +#define HINIC_GET_RQ_LOCAL_PI(rxq) \ + (((rxq)->wq->prod_idx) & HINIC_GET_RQ_WQE_MASK(rxq)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'rxq' - possible side-effects? #1360: FILE: drivers/net/hinic/hinic_pmd_rx.h:18: +#define HINIC_UPDATE_RQ_LOCAL_CI(rxq, wqebb_cnt) \ + do { \ + (rxq)->wq->cons_idx += (wqebb_cnt); \ + (rxq)->wq->delta += (wqebb_cnt); \ + } while (0) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wqebb_cnt' - possible side-effects? #1360: FILE: drivers/net/hinic/hinic_pmd_rx.h:18: +#define HINIC_UPDATE_RQ_LOCAL_CI(rxq, wqebb_cnt) \ + do { \ + (rxq)->wq->cons_idx += (wqebb_cnt); \ + (rxq)->wq->delta += (wqebb_cnt); \ + } while (0) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'rxq' - possible side-effects? #1369: FILE: drivers/net/hinic/hinic_pmd_rx.h:27: +#define HINIC_UPDATE_RQ_HW_PI(rxq, pi) \ + (*((rxq)->pi_virt_addr) = \ + cpu_to_be16((pi) & HINIC_GET_RQ_WQE_MASK(rxq))) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'txq' - possible side-effects? #1498: FILE: drivers/net/hinic/hinic_pmd_tx.h:15: +#define HINIC_GET_SQ_HW_CI(txq) \ + ((be16_to_cpu(*(txq)->cons_idx_addr)) & HINIC_GET_SQ_WQE_MASK(txq)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'txq' - possible side-effects? #1501: FILE: drivers/net/hinic/hinic_pmd_tx.h:18: +#define HINIC_GET_SQ_LOCAL_CI(txq) \ + (((txq)->wq->cons_idx) & HINIC_GET_SQ_WQE_MASK(txq)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'txq' - possible side-effects? #1504: FILE: drivers/net/hinic/hinic_pmd_tx.h:21: +#define HINIC_UPDATE_SQ_LOCAL_CI(txq, wqebb_cnt) \ + do { \ + (txq)->wq->cons_idx += wqebb_cnt; \ + (txq)->wq->delta += wqebb_cnt; \ + } while (0) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wqebb_cnt' - possible side-effects? #1504: FILE: drivers/net/hinic/hinic_pmd_tx.h:21: +#define HINIC_UPDATE_SQ_LOCAL_CI(txq, wqebb_cnt) \ + do { \ + (txq)->wq->cons_idx += wqebb_cnt; \ + (txq)->wq->delta += wqebb_cnt; \ + } while (0) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'txq' - possible side-effects? #1513: FILE: drivers/net/hinic/hinic_pmd_tx.h:30: +#define HINIC_IS_SQ_EMPTY(txq) \ + (((txq)->wq->delta) == ((txq)->q_depth)) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'txq' - possible side-effects? #1516: FILE: drivers/net/hinic/hinic_pmd_tx.h:33: +#define HINIC_GET_WQ_TAIL(txq) ((txq)->wq->queue_buf_vaddr + \ + (txq)->wq->wq_buf_size) total: 0 errors, 0 warnings, 23 checks, 1506 lines checked