[dpdk-test-report] |WARNING| pw94975 [dpdk-dev v1] crypto/aesni_gcm: fix performance issue

checkpatch at dpdk.org checkpatch at dpdk.org
Tue Jun 29 14:37:33 CEST 2021


Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/94975

_coding style issues_


ERROR:CODE_INDENT: code indent should use tabs where possible
#72: FILE: drivers/crypto/aesni_gcm/aesni_gcm_pmd.c:845:
+        if (rte_cpu_get_flag_enabled(RTE_CPUFLAG_VAES)) {$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#72: FILE: drivers/crypto/aesni_gcm/aesni_gcm_pmd.c:845:
+        if (rte_cpu_get_flag_enabled(RTE_CPUFLAG_VAES)) {$

ERROR:CODE_INDENT: code indent should use tabs where possible
#73: FILE: drivers/crypto/aesni_gcm/aesni_gcm_pmd.c:846:
+^I        dev->feature_flags |= RTE_CRYPTODEV_FF_CPU_AVX512;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#74: FILE: drivers/crypto/aesni_gcm/aesni_gcm_pmd.c:847:
+            init_mb_mgr_avx512(mb_mgr);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#74: FILE: drivers/crypto/aesni_gcm/aesni_gcm_pmd.c:847:
+            init_mb_mgr_avx512(mb_mgr);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#75: FILE: drivers/crypto/aesni_gcm/aesni_gcm_pmd.c:848:
+        } else {$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#75: FILE: drivers/crypto/aesni_gcm/aesni_gcm_pmd.c:848:
+        } else {$

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 12)
#75: FILE: drivers/crypto/aesni_gcm/aesni_gcm_pmd.c:848:
+        } else {
+            dev->feature_flags |= RTE_CRYPTODEV_FF_CPU_AVX2;

ERROR:CODE_INDENT: code indent should use tabs where possible
#76: FILE: drivers/crypto/aesni_gcm/aesni_gcm_pmd.c:849:
+            dev->feature_flags |= RTE_CRYPTODEV_FF_CPU_AVX2;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#76: FILE: drivers/crypto/aesni_gcm/aesni_gcm_pmd.c:849:
+            dev->feature_flags |= RTE_CRYPTODEV_FF_CPU_AVX2;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#77: FILE: drivers/crypto/aesni_gcm/aesni_gcm_pmd.c:850:
+            init_mb_mgr_avx2(mb_mgr);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#77: FILE: drivers/crypto/aesni_gcm/aesni_gcm_pmd.c:850:
+            init_mb_mgr_avx2(mb_mgr);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#78: FILE: drivers/crypto/aesni_gcm/aesni_gcm_pmd.c:851:
+            vector_mode = RTE_AESNI_GCM_AVX2;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#78: FILE: drivers/crypto/aesni_gcm/aesni_gcm_pmd.c:851:
+            vector_mode = RTE_AESNI_GCM_AVX2;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#79: FILE: drivers/crypto/aesni_gcm/aesni_gcm_pmd.c:852:
+        }$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#79: FILE: drivers/crypto/aesni_gcm/aesni_gcm_pmd.c:852:
+        }$

total: 8 errors, 8 warnings, 16 lines checked


More information about the test-report mailing list