[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