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 E268945A3A for ; Thu, 26 Sep 2024 18:09:13 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id DD03E40693; Thu, 26 Sep 2024 18:09:13 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id 929B14067E for ; Thu, 26 Sep 2024 18:09:12 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id 8E4EC121D9E; Thu, 26 Sep 2024 18:09:12 +0200 (CEST) Subject: |WARNING| pw144471 [PATCH v2 13/18] net/cnxk: support Rx burst scalar for cn20k In-Reply-To: <20240926160158.3206321-14-ndabilpuram@marvell.com> References: <20240926160158.3206321-14-ndabilpuram@marvell.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: Nithin Dabilpuram Message-Id: <20240926160912.8E4EC121D9E@dpdk.org> Date: Thu, 26 Sep 2024 18:09:12 +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/144471 _coding style issues_ CHECK:MACRO_ARG_REUSE: Macro argument reuse 'b' - possible side-effects? #291: 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? #291: 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? #291: 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? #294: 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? #294: 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? #294: 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 #309: 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