[dpdk-test-report] |WARNING| [PATCH 0/5] virtio/mbuf: fix virtio tso with shared mbufs

checkpatch at dpdk.org checkpatch at dpdk.org
Thu Nov 24 09:58:00 CET 2016


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

_coding style issues_


ERROR:DIFF_IN_COMMIT_MSG: Avoid using diff content in the commit message - patch(1) might not work
#61: 
--- a/lib/librte_mbuf/rte_mbuf.c

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#68: FILE: lib/librte_mbuf/rte_mbuf.c:282:
+       if (0 && off + len <= rte_pktmbuf_data_len(seg))$

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (7, 15)
#68: FILE: lib/librte_mbuf/rte_mbuf.c:282:
+       if (0 && off + len <= rte_pktmbuf_data_len(seg))
                return rte_pktmbuf_mtod_offset(seg, char *, off);

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#81: FILE: lib/librte_mbuf/rte_mbuf.h:1674:
+       if (likely(0 && off + len <= rte_pktmbuf_data_len(m)))$

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (7, 15)
#81: FILE: lib/librte_mbuf/rte_mbuf.h:1674:
+       if (likely(0 && off + len <= rte_pktmbuf_data_len(m)))
                return rte_pktmbuf_mtod_offset(m, char *, off);

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#95: FILE: drivers/net/virtio/virtio_rxtx.c:228:
+       if (unlikely(1 || rte_pktmbuf_data_is_shared(m, 0, hdrlen))) {$

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (7, 15)
#95: FILE: drivers/net/virtio/virtio_rxtx.c:228:
+       if (unlikely(1 || rte_pktmbuf_data_is_shared(m, 0, hdrlen))) {
                shared = 1;

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

total: 2 errors, 6 warnings, 0 checks, 24 lines checked


More information about the test-report mailing list