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 8BF36A0C41 for ; Wed, 15 Sep 2021 15:41:28 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 887914014F; Wed, 15 Sep 2021 15:41:28 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id 373194003C for ; Wed, 15 Sep 2021 15:41:27 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id 284DD122EE7; Wed, 15 Sep 2021 15:41:27 +0200 (CEST) In-Reply-To: <20210915133211.1310791-3-radu.nicolau@intel.com> References: <20210915133211.1310791-3-radu.nicolau@intel.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: Radu Nicolau Message-Id: <20210915134127.284DD122EE7@dpdk.org> Date: Wed, 15 Sep 2021 15:41:27 +0200 (CEST) Subject: [dpdk-test-report] |WARNING| pw98918 [PATCH v2 2/4] net/iavf: add iAVF IPsec inline crypto support 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 Sender: "test-report" Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/98918 _coding style issues_ WARNING:TYPO_SPELLING: 'acknowledgement' may be misspelled - perhaps 'acknowledgment'? #67: operations. IPsec Crypto requests receive an initial acknowledgement ERROR:CODE_INDENT: code indent should use tabs where possible #2046: FILE: drivers/net/iavf/iavf_ipsec_crypto.c:1687: +^I && actions[1].type == RTE_FLOW_ACTION_TYPE_END) {$ CHECK:LOGICAL_CONTINUATIONS: Logical continuations should be on the previous line #2046: FILE: drivers/net/iavf/iavf_ipsec_crypto.c:1687: + if (actions[0].type == RTE_FLOW_ACTION_TYPE_SECURITY + && actions[1].type == RTE_FLOW_ACTION_TYPE_END) { WARNING:TYPO_SPELLING: 'metdata' may be misspelled - perhaps 'metadata'? #3350: FILE: drivers/net/iavf/iavf_rxtx.c:2419: +iavf_ipsec_crypto_get_pkt_metdata(const struct iavf_tx_queue *txq, WARNING:TYPO_SPELLING: 'metdata' may be misspelled - perhaps 'metadata'? #3454: FILE: drivers/net/iavf/iavf_rxtx.c:2473: + ipsec_md = iavf_ipsec_crypto_get_pkt_metdata(txq, mb); WARNING:DEEP_INDENTATION: Too many leading tabs - consider code refactoring #3502: FILE: drivers/net/iavf/iavf_rxtx.c:2507: + if (idx == 0) CHECK:CAMELCASE: Avoid CamelCase: #3824: FILE: drivers/net/iavf/iavf_rxtx.h:317: + "QW0: 0x%04"PRIx16" %04"PRIx16" %04"PRIx16" %04"PRIx16", " CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements #3824: FILE: drivers/net/iavf/iavf_rxtx.h:317: + "QW0: 0x%04"PRIx16" %04"PRIx16" %04"PRIx16" %04"PRIx16", " CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements #3825: FILE: drivers/net/iavf/iavf_rxtx.h:318: + "QW1: 0x%04"PRIx16" %04"PRIx16" %04"PRIx16" %04"PRIx16" ", CHECK:CAMELCASE: Avoid CamelCase: #4355: FILE: drivers/net/iavf/iavf_rxtx.h:1061: + printf("QW0: Buffer Address : 0x%016"PRIx64" ", CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements #4355: FILE: drivers/net/iavf/iavf_rxtx.h:1061: + printf("QW0: Buffer Address : 0x%016"PRIx64" ", CHECK:OPEN_ENDED_LINE: Lines should not end with a '(' #4436: FILE: drivers/net/iavf/iavf_rxtx_vec_sse.c:366: + const __m128i ptype_mask = _mm_set_epi16( CHECK:LOGICAL_CONTINUATIONS: Logical continuations should be on the previous line #4512: FILE: drivers/net/iavf/iavf_vchnl.c:349: + if (msg_opc == VIRTCHNL_OP_INLINE_IPSEC_CRYPTO + && imsg->ipsec_opcode == CHECK:OPEN_ENDED_LINE: Lines should not end with a '(' #4534: FILE: drivers/net/iavf/iavf_vchnl.c:365: + if (rte_atomic32_read( total: 1 errors, 4 warnings, 9 checks, 4698 lines checked Warning in drivers/net/iavf/iavf.h: Using rte_atomicNN_xxx