On Tue, Mar 11, 2025 at 12:01:41AM +0200, Tariq Toukan wrote:
> From: Shay Drory <shayd@xxxxxxxxxx>
>
> mlx5_irq_pool_get() is a getter for completion IRQ pool only.
> However, after the cited commit, mlx5_irq_pool_get() is called during
> ctrl IRQ release flow to retrieve the pool, resulting in the use of an
> incorrect IRQ pool.
>
> Hence, use the newly introduced mlx5_irq_get_pool() getter to retrieve
> the correct IRQ pool based on the IRQ itself. While at it, rename
> mlx5_irq_pool_get() to mlx5_irq_table_get_comp_irq_pool() which
> accurately reflects its purpose and improves code readability.
>
> Fixes: 0477d5168bbb ("net/mlx5: Expose SFs IRQs")
> Signed-off-by: Shay Drory <shayd@xxxxxxxxxx>
> Reviewed-by: Maher Sanalla <msanalla@xxxxxxxxxx>
> Signed-off-by: Tariq Toukan <tariqt@xxxxxxxxxx>
> ---
> drivers/net/ethernet/mellanox/mlx5/core/eq.c | 2 +-
> .../net/ethernet/mellanox/mlx5/core/irq_affinity.c | 2 +-
> drivers/net/ethernet/mellanox/mlx5/core/mlx5_irq.h | 4 +++-
> drivers/net/ethernet/mellanox/mlx5/core/pci_irq.c | 13 ++++++++++---
> drivers/net/ethernet/mellanox/mlx5/core/pci_irq.h | 2 +-
> 5 files changed, 16 insertions(+), 7 deletions(-)
>
[...]
Reviewed-by: Michal Swiatkowski <michal.swiatkowski@xxxxxxxxxxxxxxx>
> --
> 2.31.1
