|WARNING| pw149551 [PATCH v5 3/3] drivers/net: add diagnostics macros to make code portable

checkpatch at dpdk.org checkpatch at dpdk.org
Tue Dec 31 19:56:32 CET 2024


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

_coding style issues_


WARNING:AVOID_EXTERNS: externs should be avoided in .c files
#223: FILE: drivers/net/fm10k/fm10k_rxtx_vec.c:473:
+__rte_diagnostic_pop

WARNING:AVOID_EXTERNS: externs should be avoided in .c files
#398: FILE: drivers/net/i40e/i40e_rxtx_vec_altivec.c:296:
+__rte_diagnostic_pop

WARNING:AVOID_EXTERNS: externs should be avoided in .c files
#718: FILE: drivers/net/i40e/i40e_rxtx_vec_sse.c:473:
+__rte_diagnostic_pop

WARNING:AVOID_EXTERNS: externs should be avoided in .c files
#1050: FILE: drivers/net/iavf/iavf_rxtx_vec_sse.c:586:
+__rte_diagnostic_pop

WARNING:AVOID_EXTERNS: externs should be avoided in .c files
#1085: FILE: drivers/net/iavf/iavf_rxtx_vec_sse.c:878:
+__rte_diagnostic_pop

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (24, 0)
#1228: FILE: drivers/net/ice/ice_rxtx_vec_avx2.c:443:
 			if (rxq->vsi->adapter->pf.dev_data->dev_conf.rxmode.offloads &
[...]
+__rte_diagnostic_push

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (24, 0)
#1300: FILE: drivers/net/ice/ice_rxtx_vec_avx512.c:473:
 			if (rxq->vsi->adapter->pf.dev_data->dev_conf.rxmode.offloads &
[...]
+__rte_diagnostic_push

WARNING:AVOID_EXTERNS: externs should be avoided in .c files
#1399: FILE: drivers/net/ice/ice_rxtx_vec_sse.c:433:
+__rte_diagnostic_pop

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 0)
#1419: FILE: drivers/net/ice/ice_rxtx_vec_sse.c:497:
 		if (rxq->vsi->adapter->pf.dev_data->dev_conf.rxmode.offloads &
[...]
+__rte_diagnostic_push

WARNING:AVOID_EXTERNS: externs should be avoided in .c files
#1533: FILE: drivers/net/ixgbe/ixgbe_rxtx_vec_sse.c:474:
+__rte_diagnostic_pop

ERROR:OPEN_BRACE: open brace '{' following function definitions go on the next line
#1643: FILE: drivers/net/mlx5/mlx5_rxtx_vec_sse.h:133:
+__rte_diagnostic_pop
 	for (pos = 0; pos < mcqe_n; ) {

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (24, 0)
#1667: FILE: drivers/net/mlx5/mlx5_rxtx_vec_sse.h:359:
 			if (!(pos & 0x7) && pos < mcqe_n) {
+__rte_diagnostic_push

total: 1 errors, 11 warnings, 0 checks, 1377 lines checked


More information about the test-report mailing list