[dpdk-test-report] |WARNING| pw26089 [PATCH RFC] app/testpmd: add option ring-bind-lcpu to bind Q with CPU

checkpatch at dpdk.org checkpatch at dpdk.org
Fri Jun 30 14:39:15 CEST 2017


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

_coding style issues_


WARNING:TYPO_SPELLING: 'overriden' may be misspelled - perhaps 'overridden'?
#50: FILE: app/test-pmd/parameters.c:135:
+		" It will overriden ring-numa-config or port-numa-config if success."

WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline
#52: FILE: app/test-pmd/parameters.c:137:
+		" It allows one port binds to multiple NUMA nodes. 
");

ERROR:GLOBAL_INITIALISERS: do not initialise globals to 0
#83: FILE: app/test-pmd/testpmd.c:101:
+uint8_t ring_bind_lcpu = 0;

ERROR:CODE_INDENT: code indent should use tabs where possible
#138: FILE: app/test-pmd/testpmd.c:1489:
+                                int socket = port->socket_id;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#138: FILE: app/test-pmd/testpmd.c:1489:
+                                int socket = port->socket_id;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#147: FILE: app/test-pmd/testpmd.c:1492:
+                                        socket = txring_numa[pi];$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#147: FILE: app/test-pmd/testpmd.c:1492:
+                                        socket = txring_numa[pi];$

ERROR:SPACING: space required after that ',' (ctx:VxV)
#210: FILE: app/test-pmd/testpmd.c:1541:
+						nb_rxd,socket,
 						      ^

total: 4 errors, 4 warnings, 167 lines checked


More information about the test-report mailing list