[dpdk-test-report] |WARNING| pw91545 [PATCH] net/bonding: fix add bonded device itself as its slave

checkpatch at dpdk.org checkpatch at dpdk.org
Thu Apr 15 09:10:43 CEST 2021


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

_coding style issues_


WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#52: 
Adding the bond device as its own slave should be forbidden. This

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#59: 
testpmd> add bonding slave 4 4

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#70: 
0x000000000052730c in cmd_add_bonding_slave_parsed ()

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#98: FILE: drivers/net/bonding/eth_bond_private.h:215:
+valid_slave_port_id(struct bond_dev_private *internals, uint16_t port_id);

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#111: FILE: drivers/net/bonding/rte_eth_bond_api.c:59:
+valid_slave_port_id(struct bond_dev_private *internals, uint16_t slave_port_id)

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#111: FILE: drivers/net/bonding/rte_eth_bond_api.c:59:
+valid_slave_port_id(struct bond_dev_private *internals, uint16_t slave_port_id)

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#114: FILE: drivers/net/bonding/rte_eth_bond_api.c:61:
+	RTE_ETH_VALID_PORTID_OR_ERR_RET(slave_port_id, -1);

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#119: FILE: drivers/net/bonding/rte_eth_bond_api.c:63:
+	/* Verify that slave_port_id refers to a non bonded port */

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#120: FILE: drivers/net/bonding/rte_eth_bond_api.c:64:
+	if (check_for_bonded_ethdev(&rte_eth_devices[slave_port_id]) == 0 &&

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#128: FILE: drivers/net/bonding/rte_eth_bond_api.c:72:
+	if (internals->port_id == slave_port_id) {

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#130: FILE: drivers/net/bonding/rte_eth_bond_api.c:74:
+			"Cannot add the bonded device itself as its slave.");

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#142: FILE: drivers/net/bonding/rte_eth_bond_api.c:465:
+	if (valid_slave_port_id(internals, slave_port_id) != 0)

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#142: FILE: drivers/net/bonding/rte_eth_bond_api.c:465:
+	if (valid_slave_port_id(internals, slave_port_id) != 0)

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#157: FILE: drivers/net/bonding/rte_eth_bond_api.c:620:
+	if (valid_slave_port_id(internals, slave_port_id) != 0)

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#157: FILE: drivers/net/bonding/rte_eth_bond_api.c:620:
+	if (valid_slave_port_id(internals, slave_port_id) != 0)

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#168: FILE: drivers/net/bonding/rte_eth_bond_api.c:646:
+	if (valid_slave_port_id(internals, slave_port_id) < 0)

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#168: FILE: drivers/net/bonding/rte_eth_bond_api.c:646:
+	if (valid_slave_port_id(internals, slave_port_id) < 0)

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#177: FILE: drivers/net/bonding/rte_eth_bond_api.c:794:
+	if (valid_slave_port_id(internals, slave_port_id) != 0)

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#177: FILE: drivers/net/bonding/rte_eth_bond_api.c:794:
+	if (valid_slave_port_id(internals, slave_port_id) != 0)

total: 0 errors, 19 warnings, 0 checks, 78 lines checked


More information about the test-report mailing list