Re: [dpdk-stable] [PATCH] net/mlx5: fix MTU update

[Luca Boccassi <bluca@debian.org>]

On Wed, 2018-02-07 at 17:04 +0200, Erez Ferber wrote:
> Changing the MTU is not related to changing the number of segments,
> activating or not the multi-segment support should be handled by the
> application.
> 
> Signed-off-by: Erez Ferber <erezf@mellanox.com>
> ---
> 
> This backport is intended for 16-11 and is based on commit
> a0edafe4099b1ef139242abb7baa2c2a48b83fd2
> 
> ---
>  
> drivers/net/mlx5/mlx5_ethdev.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/drivers/net/mlx5/mlx5_ethdev.c
> b/drivers/net/mlx5/mlx5_ethdev.c
> index c0f73e9..f73d26e 100644
> --- a/drivers/net/mlx5/mlx5_ethdev.c
> +++ b/drivers/net/mlx5/mlx5_ethdev.c
> @@ -930,9 +930,7 @@ recover:
>  		/* Provide new values to rxq_setup(). */
>  		dev->data->dev_conf.rxmode.jumbo_frame = sp;
>  		dev->data->dev_conf.rxmode.max_rx_pkt_len =
> max_frame_len;
> -		if (rehash)
> -			ret = rxq_rehash(dev, rxq_ctrl);
> -		else
> +		if (!rehash)
>  			ret = rxq_ctrl_setup(dev, rxq_ctrl, 1 <<
> rxq->elts_n,
>  					     rxq_ctrl->socket, NULL,
> rxq->mp);
>  		if (!ret)

Thanks, applied, will be pushed with the updates from -rc2 and -rc3
shortly.

-- 
Kind regards,
Luca Boccassi