[net PATCH] octeontx2-af: Free NIX_AF_INT_VEC_GEN irq

Geetha sowjanya posted 1 patch 8 months, 3 weeks ago
drivers/net/ethernet/marvell/octeontx2/af/rvu_devlink.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[net PATCH] octeontx2-af: Free NIX_AF_INT_VEC_GEN irq
Posted by Geetha sowjanya 8 months, 3 weeks ago
Due to the incorrect initial vector number in 
rvu_nix_unregister_interrupts(), NIX_AF_INT_VEC_GEN is not
geeting free. Fix the vector number to include NIX_AF_INT_VEC_GEN
irq.

Fixes: 5ed66306eab6 ("octeontx2-af: Add devlink health reporters for NIX")
Signed-off-by: Geetha sowjanya <gakula@marvell.com>
---
 drivers/net/ethernet/marvell/octeontx2/af/rvu_devlink.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/marvell/octeontx2/af/rvu_devlink.c b/drivers/net/ethernet/marvell/octeontx2/af/rvu_devlink.c
index dab4deca893f..27c3a2daaaa9 100644
--- a/drivers/net/ethernet/marvell/octeontx2/af/rvu_devlink.c
+++ b/drivers/net/ethernet/marvell/octeontx2/af/rvu_devlink.c
@@ -207,7 +207,7 @@ static void rvu_nix_unregister_interrupts(struct rvu *rvu)
 		rvu->irq_allocated[offs + NIX_AF_INT_VEC_RVU] = false;
 	}
 
-	for (i = NIX_AF_INT_VEC_AF_ERR; i < NIX_AF_INT_VEC_CNT; i++)
+	for (i = NIX_AF_INT_VEC_GEN; i < NIX_AF_INT_VEC_CNT; i++)
 		if (rvu->irq_allocated[offs + i]) {
 			free_irq(pci_irq_vector(rvu->pdev, offs + i), rvu_dl);
 			rvu->irq_allocated[offs + i] = false;
-- 
2.25.1
Re: [net PATCH] octeontx2-af: Free NIX_AF_INT_VEC_GEN irq
Posted by Simon Horman 8 months, 3 weeks ago
On Thu, Mar 27, 2025 at 03:10:54PM +0530, Geetha sowjanya wrote:
> Due to the incorrect initial vector number in 
> rvu_nix_unregister_interrupts(), NIX_AF_INT_VEC_GEN is not
> geeting free. Fix the vector number to include NIX_AF_INT_VEC_GEN
> irq.
> 
> Fixes: 5ed66306eab6 ("octeontx2-af: Add devlink health reporters for NIX")
> Signed-off-by: Geetha sowjanya <gakula@marvell.com>

Reviewed-by: Simon Horman <horms@kernel.org>