[PATCH 20.11] net/mlx5: destroy indirect actions on port stop

Xueming(Steven) Li xuemingl at nvidia.com
Fri Aug 12 09:21:05 CEST 2022


Thanks, applied to 20.11 LTS queue.

> -----Original Message-----
> From: Dmitry Kozlyuk <dkozlyuk at nvidia.com>
> Sent: Thursday, August 11, 2022 3:37 PM
> To: stable at dpdk.org
> Cc: Xueming(Steven) Li <xuemingl at nvidia.com>
> Subject: [PATCH 20.11] net/mlx5: destroy indirect actions on port stop
> 
> [ Upstream commit ec4e11d41d129ebc7c395b567827492e56fb08b7
>   cannot be backported, because it avoids the issue
>   by advertising and using a new capability
>   of keeping indirect actions across restart.
>   The behavior was unspecified before then,
>   so it is correct for 20.11 with this patch
>   to behave differently compared to the upstream. ]
> 
> Shared RSS actions were not destroyed on port stop, preventing release of involved RX queue resources.
> As a result, associated pool of such queues was exhaused after a number of port stop and start operations.
> Diagnostic message from rte_eth_dev_start():
> 
>     Rx queue allocation failed: Cannot allocate memory
> 
> When built with RTE_LIBRTE_MLX5_DEBUG,
> the following assert was triggered:
> 
>     testpmd> port start 0
>     PANIC in mlx5_rxq_start():
>     line 169        assert "!rxq_ctrl->obj" failed
> 
> Destroy indirect actions on port stop to release the resources.
> 
> Fixes: d7cfcddded61 ("net/mlx5: translate shared action for RSS action")
> 
> Signed-off-by: Dmitry Kozlyuk <dkozlyuk at nvidia.com>
> ---
>  drivers/net/mlx5/mlx5_trigger.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/net/mlx5/mlx5_trigger.c b/drivers/net/mlx5/mlx5_trigger.c index 6273122e40..c8dc0398ea 100644
> --- a/drivers/net/mlx5/mlx5_trigger.c
> +++ b/drivers/net/mlx5/mlx5_trigger.c
> @@ -13,6 +13,7 @@
>  #include <mlx5_malloc.h>
> 
>  #include "mlx5.h"
> +#include "mlx5_flow.h"
>  #include "mlx5_mr.h"
>  #include "mlx5_rxtx.h"
>  #include "mlx5_utils.h"
> @@ -1192,6 +1193,7 @@ mlx5_dev_stop(struct rte_eth_dev *dev)
>  	mlx5_traffic_disable(dev);
>  	/* All RX queue flags will be cleared in the flush interface. */
>  	mlx5_flow_list_flush(dev, &priv->flows, true);
> +	mlx5_shared_action_flush(dev);
>  	mlx5_rx_intr_vec_disable(dev);
>  	priv->sh->port[priv->dev_port - 1].ih_port_id = RTE_MAX_ETHPORTS;
>  	priv->sh->port[priv->dev_port - 1].devx_ih_port_id = RTE_MAX_ETHPORTS;
> --
> 2.25.1



More information about the stable mailing list