[dpdk-test-report] |WARNING| pw31375 [PATCH] mbuf: cleanup rte_pktmbuf_lastseg() fix atomic usage

checkpatch at dpdk.org checkpatch at dpdk.org
Wed Nov 15 09:18:33 CET 2017


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

_coding style issues_


WARNING:LEADING_SPACE: please, no spaces at the start of a line
#30: FILE: lib/librte_mbuf/rte_mbuf.h:1165:
+    if (m->next != NULL) {$

ERROR:CODE_INDENT: code indent should use tabs where possible
#31: FILE: lib/librte_mbuf/rte_mbuf.h:1166:
+        m->next = NULL;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#31: FILE: lib/librte_mbuf/rte_mbuf.h:1166:
+        m->next = NULL;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#32: FILE: lib/librte_mbuf/rte_mbuf.h:1167:
+        m->nb_segs = 1;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#32: FILE: lib/librte_mbuf/rte_mbuf.h:1167:
+        m->nb_segs = 1;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#33: FILE: lib/librte_mbuf/rte_mbuf.h:1168:
+    }$

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 8)
#42: FILE: lib/librte_mbuf/rte_mbuf.h:1334:
 	if (rte_mbuf_refcnt_update(md, -1) == 0) {
+        rte_pktmbuf_reset_before_free(m);

ERROR:CODE_INDENT: code indent should use tabs where possible
#45: FILE: lib/librte_mbuf/rte_mbuf.h:1335:
+        rte_pktmbuf_reset_before_free(m);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#45: FILE: lib/librte_mbuf/rte_mbuf.h:1335:
+        rte_pktmbuf_reset_before_free(m);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#58: FILE: lib/librte_mbuf/rte_mbuf.h:1365:
+        rte_pktmbuf_reset_before_free(m);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#58: FILE: lib/librte_mbuf/rte_mbuf.h:1365:
+        rte_pktmbuf_reset_before_free(m);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#63: FILE: lib/librte_mbuf/rte_mbuf.h:1368:
+       } else if (rte_mbuf_refcnt_update(md, -1) == 0) {$

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (7, 16)
#63: FILE: lib/librte_mbuf/rte_mbuf.h:1368:
+       } else if (rte_mbuf_refcnt_update(md, -1) == 0) {
[...]
 		if (RTE_MBUF_INDIRECT(m))

ERROR:CODE_INDENT: code indent should use tabs where possible
#72: FILE: lib/librte_mbuf/rte_mbuf.h:1373:
+        rte_pktmbuf_reset_before_free(m);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#72: FILE: lib/librte_mbuf/rte_mbuf.h:1373:
+        rte_pktmbuf_reset_before_free(m);$

total: 5 errors, 10 warnings, 52 lines checked


More information about the test-report mailing list