|WARNING| pw122235 [PATCH v7 1/1] common/idpf: add AVX512 data path for split queue model

checkpatch at dpdk.org checkpatch at dpdk.org
Wed Jan 18 04:44:24 CET 2023


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

_coding style issues_


ERROR:CODE_INDENT: code indent should use tabs where possible
#229: FILE: drivers/common/idpf/idpf_common_rxtx_avx512.c:567:
+        rte_atomic64_add(&rx_bufq->rx_stats.mbuf_alloc_failed, IDPF_RXQ_REARM_THRESH);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#229: FILE: drivers/common/idpf/idpf_common_rxtx_avx512.c:567:
+        rte_atomic64_add(&rx_bufq->rx_stats.mbuf_alloc_failed, IDPF_RXQ_REARM_THRESH);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#302: FILE: drivers/common/idpf/idpf_common_rxtx_avx512.c:640:
+            rte_atomic64_add(&rx_bufq->rx_stats.mbuf_alloc_failed, IDPF_RXQ_REARM_THRESH);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#302: FILE: drivers/common/idpf/idpf_common_rxtx_avx512.c:640:
+            rte_atomic64_add(&rx_bufq->rx_stats.mbuf_alloc_failed, IDPF_RXQ_REARM_THRESH);$

CHECK:BRACES: braces {} should be used on all arms of this statement
#1304: FILE: drivers/net/idpf/idpf_rxtx_vec_common.h:86:
+		if (vport->rxq_model == VIRTCHNL2_QUEUE_MODEL_SPLIT) {
[...]
+		} else
[...]

CHECK:BRACES: Unbalanced braces around else statement
#1307: FILE: drivers/net/idpf/idpf_rxtx_vec_common.h:89:
+		} else

total: 2 errors, 2 warnings, 2 checks, 1188 lines checked
Warning in drivers/common/idpf/idpf_common_rxtx_avx512.c:
Using rte_atomicNN_xxx


More information about the test-report mailing list