On 20/11/2017 7:15 AM, Lilijun (Jerry) wrote:
Hi all,In the function bond_mode_8023ad_enable(), the var i is to used to as the second parameter to pass the slave dev's dpdk port id to the function bond_mode_8023ad_activate_slave(). I think this variable is only a index for array internals->active_slaves. So its need to be fixed and change i to internals->active_slaves[i]. [Patch] net/bonding: fix bond 8023ad mode enable using wrong index. Signed-off-by: Lilijun <jerry.lili...@huawei.com> diff --git a/drivers/net/bonding/rte_eth_bond_8023ad.c b/drivers/net/bonding/rte_eth_bond_8023ad.c index a2313b3..20a08dc 100644 --- a/drivers/net/bonding/rte_eth_bond_8023ad.c +++ b/drivers/net/bonding/rte_eth_bond_8023ad.c @@ -1159,7 +1159,7 @@ uint8_t i; for (i = 0; i < internals->active_slave_count; i++) - bond_mode_8023ad_activate_slave(bond_dev, i); + bond_mode_8023ad_activate_slave(bond_dev, internals->active_slaves[i]); return 0; }
Acked-by: Declan Doherty <declan.dohe...@intel.com>
