Re: [dpdk-dev] [PATCH v3] net/bonding: fix selection logic

2019-10-11 Thread Chas Williams
This looks better. While reviewing this I noticed that a few lines: case AGG_STABLE: if (default_slave == slaves_count) new_agg_id = slave_id; < else new_agg_id = slaves[default_slave];

[dpdk-dev] [PATCH v3] net/bonding: fix selection logic

2019-10-10 Thread kkanas
From: Krzysztof Kanas Arrays agg_count and agg_bandwidth should be indexed by slave_id not by aggregator port_id. Fixes: 6d72657ce379 ("net/bonding: add other aggregator modes") Cc: danielx.t.mrzyg...@intel.com Signed-off-by: Krzysztof Kanas --- v3: * fix incorrect reabse v2: * rebase patch