]> git.droids-corp.org - dpdk.git/commitdiff
bonding: fix socket id for LACP slave
authorSergey Balabanov <balabanovsv@ecotelecom.ru>
Fri, 7 Aug 2015 09:33:18 +0000 (12:33 +0300)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Sun, 9 Aug 2015 23:56:05 +0000 (01:56 +0200)
On slave activation in LACP (8023AD) SOCKET_ANY_ID (which is -1)
is being casted to unsigned char and then to signed int.
The result is that socket_id has value of 255, not -1.
This results to memory allocation failure.

Fixes: 46fb43683679 ("bond: add mode 4")
Signed-off-by: Sergey Balabanov <balabanovsv@ecotelecom.ru>
Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
drivers/net/bonding/rte_eth_bond_8023ad.c

index 97a828e7e683033e6427333ccdfa16b2743d4489..c0f0b9919987b4186e2b5fdd72e6251c93764fea 100644 (file)
@@ -849,7 +849,7 @@ bond_mode_8023ad_activate_slave(struct rte_eth_dev *bond_dev, uint8_t slave_id)
        };
 
        char mem_name[RTE_ETH_NAME_MAX_LEN];
-       uint8_t socket_id;
+       int socket_id;
        unsigned element_size;
 
        /* Given slave mus not be in active list */