From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id AA0D7A04B5 for ; Wed, 9 Sep 2020 17:57:59 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 9F16A1C0DA; Wed, 9 Sep 2020 17:57:59 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 1017) id 73E921C0DA; Wed, 9 Sep 2020 17:57:58 +0200 (CEST) In-Reply-To: <20200909155717.29099-1-lance.richardson@broadcom.com> References: <20200909155717.29099-1-lance.richardson@broadcom.com> To: test-report@dpdk.org Cc: Lance Richardson Message-Id: <20200909155758.73E921C0DA@dpdk.org> Date: Wed, 9 Sep 2020 17:57:58 +0200 (CEST) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw77075 [PATCH 11/12] net/bnxt: handle multiple packets per loop in vector PMD 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/77075 _coding style issues_ CHECK:MACRO_ARG_REUSE: Macro argument reuse 'pi' - possible side-effects? #217: FILE: drivers/net/bnxt/bnxt_rxtx_vec_neon.c:25: +#define GET_OL_FLAGS(rss_flags, ol_idx, errors, pi, ol_flags) \ +{ \ + uint32_t tmp, of; \ + \ + of = vgetq_lane_u32((rss_flags), (pi)) | \ + bnxt_ol_flags_table[vgetq_lane_u32((ol_idx), (pi))]; \ + \ + tmp = vgetq_lane_u32((errors), (pi)); \ + if (tmp) \ + of |= bnxt_ol_flags_err_table[tmp]; \ + (ol_flags) = of; \ +} WARNING:TYPO_SPELLING: 'curent' may be misspelled - perhaps 'current'? #488: FILE: drivers/net/bnxt/bnxt_rxtx_vec_neon.c:257: + * Load the four curent descriptors into SSE registers in WARNING:PREFER_FALLTHROUGH: Prefer 'fallthrough;' over fallthrough comment #536: FILE: drivers/net/bnxt/bnxt_rxtx_vec_neon.c:305: + /* FALLTHROUGH */ WARNING:PREFER_FALLTHROUGH: Prefer 'fallthrough;' over fallthrough comment #539: FILE: drivers/net/bnxt/bnxt_rxtx_vec_neon.c:308: + /* FALLTHROUGH */ WARNING:PREFER_FALLTHROUGH: Prefer 'fallthrough;' over fallthrough comment #542: FILE: drivers/net/bnxt/bnxt_rxtx_vec_neon.c:311: + /* FALLTHROUGH */ CHECK:MACRO_ARG_REUSE: Macro argument reuse 'pi' - possible side-effects? #650: FILE: drivers/net/bnxt/bnxt_rxtx_vec_sse.c:25: +#define GET_OL_FLAGS(rss_flags, ol_index, errors, pi, ol_flags) \ +{ \ + uint32_t tmp, of; \ + \ + of = _mm_extract_epi32((rss_flags), (pi)) | \ + bnxt_ol_flags_table[_mm_extract_epi32((ol_index), (pi))]; \ + \ + tmp = _mm_extract_epi32((errors), (pi)); \ + if (tmp) \ + of |= bnxt_ol_flags_err_table[tmp]; \ + (ol_flags) = of; \ +} WARNING:TYPO_SPELLING: 'curent' may be misspelled - perhaps 'current'? #897: FILE: drivers/net/bnxt/bnxt_rxtx_vec_sse.c:237: + * Load the four curent descriptors into SSE registers in WARNING:PREFER_FALLTHROUGH: Prefer 'fallthrough;' over fallthrough comment #956: FILE: drivers/net/bnxt/bnxt_rxtx_vec_sse.c:276: + /* FALLTHROUGH */ WARNING:PREFER_FALLTHROUGH: Prefer 'fallthrough;' over fallthrough comment #959: FILE: drivers/net/bnxt/bnxt_rxtx_vec_sse.c:279: + /* FALLTHROUGH */ WARNING:PREFER_FALLTHROUGH: Prefer 'fallthrough;' over fallthrough comment #962: FILE: drivers/net/bnxt/bnxt_rxtx_vec_sse.c:282: + /* FALLTHROUGH */ total: 0 errors, 8 warnings, 2 checks, 857 lines checked