On 2/22/2019 1:52 AM, Chas Williams wrote: > > > On 2/20/19 10:16 AM, Hyong Youb Kim wrote: >> On Wed, Feb 20, 2019 at 02:56:36PM +0000, Radu Nicolau wrote: >>> >>> >>> On 2/20/2019 12:33 PM, Ferruh Yigit wrote: >>>> On 2/18/2019 3:58 PM, Radu Nicolau wrote: >>>>> >>>>> On 2/18/2019 11:59 AM, Hari Kumar Vemula wrote: >>>>>> test_alb_reply_from_client test fails due to incorrect active slave >>>>>> array's index. This was due to invalid active slave count. >>>>>> >>>>>> Count of internals->active_slave is not updated even when active slave >>>>>> is deactivated. >>>>>> Hence active slave count always keeps incrementing beyond the actual >>>>>> active slaves. >>>>>> >>>>>> Fix is to set the internals->active_slave to starting index 0 whenever >>>>>> it exceeds the number of slaves in active slave list and also update >>>>>> the active slave count during slave de-activation. >>>>>> >>>>>> Fixes: e1110e977648 ("net/bonding: fix Rx slave fairness") >>>>>> Cc: sta...@dpdk.org >>>>>> >>>>>> Signed-off-by: Hari Kumar Vemula <hari.kumarx.vem...@intel.com> >>>>>> --- >>>>>> >>>>> Acked-by: Radu Nicolau <radu.nico...@intel.com > > Acked-by: Chas Williams <ch...@att.com>
Applied to dpdk-next-net/master, thanks.