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 59E25A00C5 for ; Mon, 6 Jul 2020 19:22:57 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 485DB1DB58; Mon, 6 Jul 2020 19:22:57 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 1017) id A19E61DB5B; Mon, 6 Jul 2020 19:22:56 +0200 (CEST) In-Reply-To: <20200706171925.7092-1-arkadiuszx.kusztal@intel.com> References: <20200706171925.7092-1-arkadiuszx.kusztal@intel.com> To: test-report@dpdk.org Cc: Arek Kusztal Message-Id: <20200706172256.A19E61DB5B@dpdk.org> Date: Mon, 6 Jul 2020 19:22:56 +0200 (CEST) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw73263 [PATCH v3] crypto/qat: add handling of multi process 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/73263 _coding style issues_ Must be a reply to the first patch (--in-reply-to). WARNING:TYPO_SPELLING: 'uint' may be misspelled - perhaps 'unit'? #152: FILE: drivers/common/qat/qat_device.c:85: + uint8_t dev_id; WARNING:TYPO_SPELLING: 'uint' may be misspelled - perhaps 'unit'? #173: FILE: drivers/common/qat/qat_device.c:167: + uint8_t qat_dev_id = 0; WARNING:TYPO_SPELLING: 'uint' may be misspelled - perhaps 'unit'? #514: FILE: drivers/compress/qat/qat_comp_pmd.c:674: + uint64_t capa_size; WARNING:TYPO_SPELLING: 'uint' may be misspelled - perhaps 'unit'? #649: FILE: drivers/crypto/qat/qat_asym_pmd.c:14: +uint8_t qat_asym_driver_id; WARNING:TYPO_SPELLING: 'uint' may be misspelled - perhaps 'unit'? #800: FILE: drivers/crypto/qat/qat_asym_pmd.h:16: +extern uint8_t qat_asym_driver_id; WARNING:TYPO_SPELLING: 'uint' may be misspelled - perhaps 'unit'? #848: FILE: drivers/crypto/qat/qat_sym_pmd.c:22: +uint8_t qat_sym_driver_id; WARNING:LEADING_SPACE: please, no spaces at the start of a line #857: FILE: drivers/crypto/qat/qat_sym_pmd.c:36: + QAT_BASE_GEN1_SYM_CAPABILITIES,$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #858: FILE: drivers/crypto/qat/qat_sym_pmd.c:37: + QAT_EXTRA_GEN2_SYM_CAPABILITIES,$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #859: FILE: drivers/crypto/qat/qat_sym_pmd.c:38: + RTE_CRYPTODEV_END_OF_CAPABILITIES_LIST()$ ERROR:CODE_INDENT: code indent should use tabs where possible #879: FILE: drivers/crypto/qat/qat_sym_pmd.c:314: + &qat_pci_devs[qat_pci_dev->qat_dev_id];$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #879: FILE: drivers/crypto/qat/qat_sym_pmd.c:314: + &qat_pci_devs[qat_pci_dev->qat_dev_id];$ WARNING:TYPO_SPELLING: 'uint' may be misspelled - perhaps 'unit'? #893: FILE: drivers/crypto/qat/qat_sym_pmd.c:327: + uint64_t capa_size; WARNING:BLOCK_COMMENT_STYLE: Block comments should align the * on each line #895: FILE: drivers/crypto/qat/qat_sym_pmd.c:329: + /* + * All processes must use same driver id so they can share sessions. WARNING:LEADING_SPACE: please, no spaces at the start of a line #1008: FILE: drivers/crypto/qat/qat_sym_pmd.c:450: + memcpy(internals->capa_mz->addr, capabilities, capa_size);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #1009: FILE: drivers/crypto/qat/qat_sym_pmd.c:451: + internals->qat_dev_capabilities = internals->capa_mz->addr;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #1029: FILE: drivers/crypto/qat/qat_sym_pmd.c:478: + if (rte_eal_process_type() == RTE_PROC_PRIMARY)$ WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (7, 15) #1029: FILE: drivers/crypto/qat/qat_sym_pmd.c:478: + if (rte_eal_process_type() == RTE_PROC_PRIMARY) + rte_memzone_free(qat_pci_dev->sym_dev->capa_mz); ERROR:CODE_INDENT: code indent should use tabs where possible #1030: FILE: drivers/crypto/qat/qat_sym_pmd.c:479: + rte_memzone_free(qat_pci_dev->sym_dev->capa_mz);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #1030: FILE: drivers/crypto/qat/qat_sym_pmd.c:479: + rte_memzone_free(qat_pci_dev->sym_dev->capa_mz);$ WARNING:TYPO_SPELLING: 'uint' may be misspelled - perhaps 'unit'? #1058: FILE: drivers/crypto/qat/qat_sym_pmd.h:26: +extern uint8_t qat_sym_driver_id; total: 2 errors, 18 warnings, 885 lines checked