[dpdk-test-report] |WARNING| pw30332 [PATCH] ring: guarantee ordering of cons/prod loading when doing enqueue/dequeue

checkpatch at dpdk.org checkpatch at dpdk.org
Fri Oct 13 05:24:32 CEST 2017


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

_coding style issues_


WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#28: 
>> Cc: Olivier MATZ <olivier.matz at 6wind.com>, "dev at dpdk.org" <dev at dpdk.org>,

WARNING:AVOID_EXTERNS: externs should be avoided in .c files
#180: FILE: test/test/test_mbuf.c:87:
+int test_mbuf(void);

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#201: FILE: test/test/test_mbuf.c:749:
+			    __func__, lcore, iter, tref);*/

ERROR:CODE_INDENT: code indent should use tabs where possible
#207: FILE: test/test/test_mbuf.c:755:
+        rte_mempool_dump(stdout, refcnt_pool);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#207: FILE: test/test/test_mbuf.c:755:
+        rte_mempool_dump(stdout, refcnt_pool);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#209: FILE: test/test/test_mbuf.c:757:
+        rte_ring_dump(stdout, refcnt_mbuf_ring);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#209: FILE: test/test/test_mbuf.c:757:
+        rte_ring_dump(stdout, refcnt_mbuf_ring);$

ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s)

total: 3 errors, 5 warnings, 82 lines checked


More information about the test-report mailing list