Re: [PATCH net v1] net/mlx5: Fix HCA caps leak on notifier init failure

From: Cosmin Ratiu

Date: Wed Apr 15 2026 - 04:31:10 EST


On Wed, 2026-04-15 at 01:49 +0100, Prathamesh Deshpande wrote:
> mlx5_mdev_init() allocates HCA caps via mlx5_hca_caps_alloc() before
> calling mlx5_notifiers_init(). If notifier initialization fails, the
> error path jumps to err_hca_caps and skips mlx5_hca_caps_free(),
> leaking
> allocated caps.
>
> Add a dedicated unwind label for notifier-init failure that frees HCA
> caps before continuing the existing cleanup sequence.
>
> Fixes: b6b03097f982 ("net/mlx5: Initialize events outside devlink
> lock")

Thank you for the fix, LGTM!

> Signed-off-by: Prathamesh Deshpande <prathameshdeshpande7@xxxxxxxxx>

Reviewed-by: Cosmin Ratiu <cratiu@xxxxxxxxxx>