[dpdk-test-report] |WARNING| pw70733 [RFC 2/3] net/mlx5: add default miss action to flow engine

checkpatch at dpdk.org checkpatch at dpdk.org
Tue Jun 2 11:40:15 CEST 2020


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

_coding style issues_


CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
#177: FILE: drivers/net/mlx5/mlx5_flow.c:1259:
+{
+

WARNING:TYPO_SPELLING: 'Deafult' may be misspelled - perhaps 'Default'?
#194: FILE: drivers/net/mlx5/mlx5_flow.c:5015:
+ * Deafult miss flow

ERROR:TRAILING_WHITESPACE: trailing whitespace
#230: FILE: drivers/net/mlx5/mlx5_flow.c:5051:
+^I$

ERROR:TRAILING_WHITESPACE: trailing whitespace
#233: FILE: drivers/net/mlx5/mlx5_flow.c:5054:
+^I$

ERROR:TRAILING_WHITESPACE: trailing whitespace
#326: FILE: drivers/net/mlx5/mlx5_flow_dv.c:2699:
+^I$

ERROR:SPACING: space required before the open brace '{'
#328: FILE: drivers/net/mlx5/mlx5_flow_dv.c:2701:
+	if (!cnt){

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 16)
#331: FILE: drivers/net/mlx5/mlx5_flow_dv.c:2704:
+		if (sh->default_miss_action)
+		return rte_flow_error_set(error, ENOMEM,

ERROR:TRAILING_WHITESPACE: trailing whitespace
#351: FILE: drivers/net/mlx5/mlx5_flow_dv.c:5257:
+^I^I^Iret = $

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (24, 24)
#367: FILE: drivers/net/mlx5/mlx5_flow_dv.c:8229:
+			if (flow_dv_default_miss_resource_register
[...]
+			return -rte_errno;

ERROR:TRAILING_WHITESPACE: trailing whitespace
#371: FILE: drivers/net/mlx5/mlx5_flow_dv.c:8233:
+^I^I^Idev_flow->handle->fate_action = $

ERROR:SPACING: space required before the open brace '{'
#411: FILE: drivers/net/mlx5/mlx5_flow_dv.c:8811:
+	if (rte_atomic32_dec_and_test(&sh->default_miss_refcnt)){

total: 7 errors, 3 warnings, 1 checks, 271 lines checked


More information about the test-report mailing list