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 1E4B445954 for ; Tue, 10 Sep 2024 11:36:05 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 19CEB427B6; Tue, 10 Sep 2024 11:36:05 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id 332E6402A1 for ; Tue, 10 Sep 2024 11:36:04 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id 2A106121D10; Tue, 10 Sep 2024 11:36:04 +0200 (CEST) Subject: |WARNING| pw143879 [PATCH 21/33] net/cnxk: support Rx burst scalar for cn20k In-Reply-To: <20240910085909.1514457-22-ndabilpuram@marvell.com> References: <20240910085909.1514457-22-ndabilpuram@marvell.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: Nithin Dabilpuram Message-Id: <20240910093604.2A106121D10@dpdk.org> Date: Tue, 10 Sep 2024 11:36:04 +0200 (CEST) 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/143879 _coding style issues_ CHECK:MACRO_ARG_REUSE: Macro argument reuse 'b' - possible side-effects? #290: FILE: drivers/net/cnxk/cn20k_rx.h:37: +#define CQE_PTR_OFF(b, i, o, f) \ + (((f) & NIX_RX_VWQE_F) ? (uint64_t *)(((uintptr_t)((uint64_t *)(b))[i]) + (o)) : \ + (uint64_t *)(((uintptr_t)(b)) + CQE_SZ(i) + (o))) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i' - possible side-effects? #290: FILE: drivers/net/cnxk/cn20k_rx.h:37: +#define CQE_PTR_OFF(b, i, o, f) \ + (((f) & NIX_RX_VWQE_F) ? (uint64_t *)(((uintptr_t)((uint64_t *)(b))[i]) + (o)) : \ + (uint64_t *)(((uintptr_t)(b)) + CQE_SZ(i) + (o))) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'o' - possible side-effects? #290: FILE: drivers/net/cnxk/cn20k_rx.h:37: +#define CQE_PTR_OFF(b, i, o, f) \ + (((f) & NIX_RX_VWQE_F) ? (uint64_t *)(((uintptr_t)((uint64_t *)(b))[i]) + (o)) : \ + (uint64_t *)(((uintptr_t)(b)) + CQE_SZ(i) + (o))) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'b' - possible side-effects? #293: FILE: drivers/net/cnxk/cn20k_rx.h:40: +#define CQE_PTR_DIFF(b, i, o, f) \ + (((f) & NIX_RX_VWQE_F) ? (uint64_t *)(((uintptr_t)((uint64_t *)(b))[i]) - (o)) : \ + (uint64_t *)(((uintptr_t)(b)) + CQE_SZ(i) - (o))) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i' - possible side-effects? #293: FILE: drivers/net/cnxk/cn20k_rx.h:40: +#define CQE_PTR_DIFF(b, i, o, f) \ + (((f) & NIX_RX_VWQE_F) ? (uint64_t *)(((uintptr_t)((uint64_t *)(b))[i]) - (o)) : \ + (uint64_t *)(((uintptr_t)(b)) + CQE_SZ(i) - (o))) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'o' - possible side-effects? #293: FILE: drivers/net/cnxk/cn20k_rx.h:40: +#define CQE_PTR_DIFF(b, i, o, f) \ + (((f) & NIX_RX_VWQE_F) ? (uint64_t *)(((uintptr_t)((uint64_t *)(b))[i]) - (o)) : \ + (uint64_t *)(((uintptr_t)(b)) + CQE_SZ(i) - (o))) WARNING:BRACES: braces {} are not necessary for single statement blocks #308: FILE: drivers/net/cnxk/cn20k_rx.h:55: + if (m->nb_segs == 1 && m->next) { + rte_panic("mbuf->next[%p] valid when mbuf->nb_segs is %d", m->next, m->nb_segs); + } total: 0 errors, 1 warnings, 6 checks, 756 lines checked Warning in drivers/net/cnxk/cn20k_rx.h: Using rte_panic/rte_exit