Skip to content

Commit

Permalink
net/bonding: delete redundant code
Browse files Browse the repository at this point in the history
[ upstream commit 2db4cf9 ]

The function valid_bonded_port_id() has already contains function
rte_eth_dev_is_valid_port(), so delete redundant check.

Fixes: 588ae95 ("net/bonding: fix port ID check")

Signed-off-by: Dongyang Pan <197020236@qq.com>
Reviewed-by: Wei Hu (Xavier) <xavier.huwei@huawei.com>
  • Loading branch information
15825435276 authored and kevintraynor committed Aug 27, 2020
1 parent f04772a commit 67e9387
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions drivers/net/bonding/rte_eth_bond_8023ad.c
Expand Up @@ -1589,9 +1589,6 @@ rte_eth_bond_8023ad_dedicated_queues_enable(uint16_t port)
dev = &rte_eth_devices[port];
internals = dev->data->dev_private;

if (check_for_bonded_ethdev(dev) != 0)
return -1;

if (bond_8023ad_slow_pkt_hw_filter_supported(port) != 0)
return -1;

Expand All @@ -1618,9 +1615,6 @@ rte_eth_bond_8023ad_dedicated_queues_disable(uint16_t port)
dev = &rte_eth_devices[port];
internals = dev->data->dev_private;

if (check_for_bonded_ethdev(dev) != 0)
return -1;

/* Device must be stopped to set up slow queue */
if (dev->data->dev_started)
return -1;
Expand Down

0 comments on commit 67e9387

Please sign in to comment.