[dpdk-test-report] |WARNING| pw43718 net/bonding: per-slave intermediate rx ring

checkpatch at dpdk.org checkpatch at dpdk.org
Wed Aug 15 17:47:30 CEST 2018


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

_coding style issues_


WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#45: FILE: drivers/net/bonding/rte_eth_bond_api.c:529:
+		unsigned j, count;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#83: FILE: drivers/net/bonding/rte_eth_bond_pmd.c:407:
+	unsigned rx_ring_avail = rte_ring_free_count(internals->rx_ring);

WARNING:LONG_LINE: line over 90 characters
#143: FILE: drivers/net/bonding/rte_eth_bond_pmd.c:463:
+				rte_prefetch0(rte_pktmbuf_mtod(mbuf_bounce[j + 3], void *));

WARNING:LONG_LINE: line over 90 characters
#153: FILE: drivers/net/bonding/rte_eth_bond_pmd.c:471:
+			if (unlikely(is_lacp_packets(hdr->ether_type, subtype, mbuf_bounce[j]) ||

CHECK:BRACES: braces {} should be used on all arms of this statement
#153: FILE: drivers/net/bonding/rte_eth_bond_pmd.c:471:
+			if (unlikely(is_lacp_packets(hdr->ether_type, subtype, mbuf_bounce[j]) ||
[...]
-						(num_rx_total - j));
[...]

WARNING:LONG_LINE: line over 90 characters
#173: FILE: drivers/net/bonding/rte_eth_bond_pmd.c:485:
+					memmove(&mbuf_bounce[j], &mbuf_bounce[j + 1], sizeof(mbuf_bounce[0]) *

total: 0 errors, 5 warnings, 1 checks, 205 lines checked


More information about the test-report mailing list