[dpdk-dev] [PATCH 01/18] net/bonding: fix link speed update in broadcast mode
Chas Williams
3chas3 at gmail.com
Wed Sep 11 01:01:44 CEST 2019
Acked-by: Chas Williams <chas3 at att.com>
On 9/10/19 4:25 AM, Andrew Rybchenko wrote:
> From: Igor Romanov <igor.romanov at oktetlabs.ru>
>
> Fix the issue that the link speed of the bond device was set to the
> link speed of the first active slave in broadcast mode.
>
> Set the link speed of the bond device to the minimum value across
> all of the slaves in that case.
>
> Fixes: deba8a2f8b0b ("net/bonding: fix link properties management")
> Cc: stable at dpdk.org
>
> Signed-off-by: Igor Romanov <igor.romanov at oktetlabs.ru>
> Signed-off-by: Andrew Rybchenko <arybchenko at solarflare.com>
> ---
> drivers/net/bonding/rte_eth_bond_pmd.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/bonding/rte_eth_bond_pmd.c b/drivers/net/bonding/rte_eth_bond_pmd.c
> index f774e5677..fed71bd95 100644
> --- a/drivers/net/bonding/rte_eth_bond_pmd.c
> +++ b/drivers/net/bonding/rte_eth_bond_pmd.c
> @@ -2395,8 +2395,8 @@ bond_ethdev_link_update(struct rte_eth_dev *ethdev, int wait_to_complete)
> * packet loss will occur on this slave if transmission at rates
> * greater than this are attempted
> */
> - for (idx = 1; idx < bond_ctx->active_slave_count; idx++) {
> - link_update(bond_ctx->active_slaves[0], &slave_link);
> + for (idx = 0; idx < bond_ctx->active_slave_count; idx++) {
> + link_update(bond_ctx->active_slaves[idx], &slave_link);
>
> if (slave_link.link_speed <
> ethdev->data->dev_link.link_speed)
>
More information about the dev
mailing list