|WARNING| pw171152 [PATCH v7 03/25] net/intel/common: add flow engines infrastructure

checkpatch at dpdk.org checkpatch at dpdk.org
Tue Oct 13 14:06:37 CEST 2026


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

_coding style issues_


CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'engine->ops->flow_alloc == NULL'
#622: FILE: drivers/net/intel/common/flow_engine.h:496:
+	if ((engine->ops->flow_alloc == NULL) != (engine->ops->flow_free == NULL))

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'engine->ops->flow_free == NULL'
#622: FILE: drivers/net/intel/common/flow_engine.h:496:
+	if ((engine->ops->flow_alloc == NULL) != (engine->ops->flow_free == NULL))

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'engine->ops->flow_register == NULL'
#625: FILE: drivers/net/intel/common/flow_engine.h:499:
+	if ((engine->ops->flow_register == NULL) != (engine->ops->flow_unregister == NULL))

CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'engine->ops->flow_unregister == NULL'
#625: FILE: drivers/net/intel/common/flow_engine.h:499:
+	if ((engine->ops->flow_register == NULL) != (engine->ops->flow_unregister == NULL))

WARNING:BRACES: braces {} are not necessary for single statement blocks
#1298: FILE: drivers/net/intel/common/flow_engine.h:1172:
+		if (ret == -ENOMEM) {
+			goto unlock;
+		}

WARNING:TYPO_SPELLING: 'clen' may be misspelled - perhaps 'clean'?
#1495: FILE: drivers/net/intel/common/flow_engine.h:1369:
+		const size_t clen =
 		             ^^^^

WARNING:TYPO_SPELLING: 'clen' may be misspelled - perhaps 'clean'?
#1500: FILE: drivers/net/intel/common/flow_engine.h:1374:
+		rte_memdump(file, title, raw + off, clen);
 		                                    ^^^^

total: 0 errors, 3 warnings, 4 checks, 1441 lines checked


More information about the test-report mailing list