d/numam-dpdk
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

net/bonding: fix slave id types

Browse Source 
mode_bond_id and mode_band_id are slave ids, stored on 16bits.

Fixes: f8244c6399d9 ("ethdev: increase port id range")
Cc: stable@dpdk.org

Signed-off-by: Hui Zhao <zhaohui8@huawei.com>
Signed-off-by: David Marchand <david.marchand@redhat.com>
Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>
This commit is contained in:
Hui Zhao 2019-03-21 21:28:13 +01:00 committed by Ferruh Yigit
parent 3c8c0b68b5
commit c28aff1e41
1 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
drivers/net/bonding/rte_eth_bond_8023ad.c
View File

@ -639,7 +639,7 @@ tx_machine(struct bond_dev_private *internals, uint16_t slave_id)
SM_FLAG_CLR(port, NTT);
}
static uint8_t
static uint16_t
max_index(uint64_t *a, int n)
{
if (n <= 0)
@ -673,7 +673,8 @@ selection_logic(struct bond_dev_private *internals, uint16_t slave_id)
uint64_t agg_bandwidth[8] = {0};
uint64_t agg_count[8] = {0};
uint16_t default_slave = 0;
uint8_t mode_count_id, mode_band_id;
uint16_t mode_count_id;
uint16_t mode_band_id;
struct rte_eth_link link_info;
int ret;
@ -717,13 +718,11 @@ selection_logic(struct bond_dev_private *internals, uint16_t slave_id)
switch (internals->mode4.agg_selection) {
case AGG_COUNT:
mode_count_id = max_index(
(uint64_t *)agg_count, slaves_count);
mode_count_id = max_index(agg_count, slaves_count);
new_agg_id = mode_count_id;
break;
case AGG_BANDWIDTH:
mode_band_id = max_index(
(uint64_t *)agg_bandwidth, slaves_count);
mode_band_id = max_index(agg_bandwidth, slaves_count);
new_agg_id = mode_band_id;
break;
case AGG_STABLE: