On 12/11/2025 07:40, Bing Zhao wrote:
> From: Roi Dayan <[email protected]>
> 
> Looks like commit the cited commit removed the representor
> interrupt handler cleanup by mistake. Fix it.
> 
> Fixes: 5cf0707fc7e9 ("net/mlx5: remove Rx queue data list from device")
> Cc: [email protected]
> 

Hi,

Some of this series (see below) is fixing issues in older releases, but
does not include stable tag to indicate the fixes should be backported.

Just checking if this is deliberate, or an oversight and they should be
backported ?

thanks,
Kevin.

25.11 dbaed15366 - F net/mlx5: release representor interrupt handler
(Fixes issue in 21.11)

25.11 aef94343d3 - F common/mlx5: release unused mempool entries (Fixes
issue in  21.11 (partially fixed in 8947eebc999e @ 21.11))

25.11 8d1fe10768 - F net/mlx5/hws: fix buddy memory allocation (Fixes
issue in 22.11)


> Signed-off-by: Roi Dayan <[email protected]>
> ---
>  drivers/net/mlx5/mlx5.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/drivers/net/mlx5/mlx5.c b/drivers/net/mlx5/mlx5.c
> index 666216c21a..591420c77f 100644
> --- a/drivers/net/mlx5/mlx5.c
> +++ b/drivers/net/mlx5/mlx5.c
> @@ -2389,6 +2389,11 @@ mlx5_dev_close(struct rte_eth_dev *dev)
>       /* Free the eCPRI flex parser resource. */
>       mlx5_flex_parser_ecpri_release(dev);
>       mlx5_flex_item_port_cleanup(dev);
> +     if (priv->representor) {
> +             /* Each representor has a dedicated interrupts handler */
> +             rte_intr_instance_free(dev->intr_handle);
> +             dev->intr_handle = NULL;
> +     }
>       mlx5_indirect_list_handles_release(dev);
>  #ifdef HAVE_MLX5_HWS_SUPPORT
>       mlx5_nta_sample_context_free(dev);

Reply via email to