Re: [dpdk-dev] [PATCH] net/bonding: fix bond 8023ad mode enable using wrong index

2017-12-15 Thread Ferruh Yigit
On 12/12/2017 7:58 AM, Doherty, Declan wrote: > 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(). >

Re: [dpdk-dev] [PATCH] net/bonding: fix bond 8023ad mode enable using wrong index

2017-12-12 Thread Doherty, Declan
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->ac

Re: [dpdk-dev] [PATCH] net/bonding: fix bond 8023ad mode enable using wrong index

2017-11-23 Thread Kyle Larose
Hello, > -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Lilijun (Jerry) > Sent: Monday, November 20, 2017 2:16 AM > To: dev@dpdk.org; tomaszx.kula...@intel.com > Cc: Zhang, Jerry; Wanghanlin > Subject: [dpdk-dev] [PATCH] net/bonding: fix bond

Re: [dpdk-dev] [PATCH] net/bonding: fix bond 8023ad mode enable using wrong index

2017-11-23 Thread Kyle Larose
g; tomaszx.kula...@intel.com > > Cc: Zhang, Jerry; Wanghanlin > > Subject: [dpdk-dev] [PATCH] net/bonding: fix bond 8023ad mode enable > > using wrong index > > > > ... > > > --- a/drivers/net/bonding/rte_eth_bond_8023ad.c > > +++ b/drivers/n

Re: [dpdk-dev] [PATCH] net/bonding: fix bond 8023ad mode enable using wrong index

2017-11-23 Thread Kyle Larose
Hello, > -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Lilijun (Jerry) > Sent: Monday, November 20, 2017 2:16 AM > To: dev@dpdk.org; tomaszx.kula...@intel.com > Cc: Zhang, Jerry; Wanghanlin > Subject: [dpdk-dev] [PATCH] net/bonding: fix bond

[dpdk-dev] [PATCH] net/bonding: fix bond 8023ad mode enable using wrong index

2017-11-19 Thread Lilijun (Jerry)
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 chang