net/bonding: fix slave id types

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>

diff --git a/drivers/net/bonding/rte_eth_bond_8023ad.c b/drivers/net/bonding/rte_eth_bond_8023ad.c
index e50d946..c6a645a 100644 (file)
--- a/drivers/net/bonding/rte_eth_bond_8023ad.c
+++ b/drivers/net/bonding/rte_eth_bond_8023ad.c
@@ -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: