|WARNING| pw124048 [PATCH v2 1/2] net/bonding: add independent LACP sending function

checkpatch at dpdk.org checkpatch at dpdk.org
Thu Feb 16 09:34:33 CET 2023


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

_coding style issues_


WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#177: FILE: drivers/net/bonding/rte_eth_bond_8023ad.c:1783:
+	uint16_t slave_tx_count;

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#178: FILE: drivers/net/bonding/rte_eth_bond_8023ad.c:1784:
+	uint16_t active_slave_count;

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#179: FILE: drivers/net/bonding/rte_eth_bond_8023ad.c:1785:
+	uint16_t active_slave_ids[RTE_MAX_ETHPORTS];

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#183: FILE: drivers/net/bonding/rte_eth_bond_8023ad.c:1789:
+	active_slave_count = internals->active_slave_count;

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#183: FILE: drivers/net/bonding/rte_eth_bond_8023ad.c:1789:
+	active_slave_count = internals->active_slave_count;

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#184: FILE: drivers/net/bonding/rte_eth_bond_8023ad.c:1790:
+	if (unlikely(active_slave_count == 0))

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#187: FILE: drivers/net/bonding/rte_eth_bond_8023ad.c:1793:
+	rte_memcpy(active_slave_ids, internals->active_slaves,

WARNING:TYPO_SPELLING: 'slaves' may be misspelled - perhaps 'secondaries'?
#187: FILE: drivers/net/bonding/rte_eth_bond_8023ad.c:1793:
+	rte_memcpy(active_slave_ids, internals->active_slaves,

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#188: FILE: drivers/net/bonding/rte_eth_bond_8023ad.c:1794:
+		sizeof(active_slave_ids[0]) * active_slave_count);

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#188: FILE: drivers/net/bonding/rte_eth_bond_8023ad.c:1794:
+		sizeof(active_slave_ids[0]) * active_slave_count);

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#191: FILE: drivers/net/bonding/rte_eth_bond_8023ad.c:1797:
+	for (i = 0; i < active_slave_count; i++) {

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#193: FILE: drivers/net/bonding/rte_eth_bond_8023ad.c:1799:
+		struct port *port = &bond_mode_8023ad_ports[active_slave_ids[i]];

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#199: FILE: drivers/net/bonding/rte_eth_bond_8023ad.c:1805:
+			slave_tx_count = rte_eth_tx_prepare(active_slave_ids[i],

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#199: FILE: drivers/net/bonding/rte_eth_bond_8023ad.c:1805:
+			slave_tx_count = rte_eth_tx_prepare(active_slave_ids[i],

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#201: FILE: drivers/net/bonding/rte_eth_bond_8023ad.c:1807:
+			slave_tx_count = rte_eth_tx_burst(active_slave_ids[i],

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#201: FILE: drivers/net/bonding/rte_eth_bond_8023ad.c:1807:
+			slave_tx_count = rte_eth_tx_burst(active_slave_ids[i],

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#202: FILE: drivers/net/bonding/rte_eth_bond_8023ad.c:1808:
+					bd_tx_q->queue_id, &ctrl_pkt, slave_tx_count);

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#207: FILE: drivers/net/bonding/rte_eth_bond_8023ad.c:1813:
+			if (slave_tx_count != 1)

total: 0 errors, 18 warnings, 0 checks, 97 lines checked
ERROR: symbol rte_eth_bond_8023ad_dedicated_queues_get is added in the DPDK_23 section, but is expected to be added in the EXPERIMENTAL section of the version map
ERROR: symbol rte_eth_bond_8023ad_lacp_send_one is added in the DPDK_23 section, but is expected to be added in the EXPERIMENTAL section of the version map


More information about the test-report mailing list