[dpdk-dev] [PATCH] net/bnx2x: fix FW cmd timeout duting device stop

[dpdk-dev] [PATCH] net/bnx2x: fix FW cmd timeout duting device stop

[Rasesh Mody]

This patch fixes firmware command timeout error seen during device stop
while stopping queues. It patially reverts an earlier preventive change
commit 91b7e432bcef ("net/bnx2x: disable fast path interrupts") to now
enable fast path interrupts.

The original issue of performance degradation is not observed anymore,
with or without the fix.

Fixes: 91b7e432bcef ("net/bnx2x: disable fast path interrupts")
Cc: stable@dpdk.org

Signed-off-by: Rasesh Mody <rasesh.mody@cavium.com>
---
 drivers/net/bnx2x/bnx2x.c |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/net/bnx2x/bnx2x.c b/drivers/net/bnx2x/bnx2x.c
index 84ade5f..6228744 100644
--- a/drivers/net/bnx2x/bnx2x.c
+++ b/drivers/net/bnx2x/bnx2x.c
@@ -4490,6 +4490,8 @@ static void bnx2x_handle_fp_tq(struct bnx2x_fastpath *fp, int scan_fp)
 	struct bnx2x_softc *sc = fp->sc;
 	uint8_t more_rx = FALSE;
 
+	PMD_DRV_LOG(DEBUG, "---> FP TASK QUEUE (%d) <--", fp->index);
+
 	/* update the fastpath index */
 	bnx2x_update_fp_sb_idx(fp);
 
@@ -4506,7 +4508,7 @@ static void bnx2x_handle_fp_tq(struct bnx2x_fastpath *fp, int scan_fp)
 	}
 
 	bnx2x_ack_sb(sc, fp->igu_sb_id, USTORM_ID,
-		   le16toh(fp->fp_hc_idx), IGU_INT_DISABLE, 1);
+		   le16toh(fp->fp_hc_idx), IGU_INT_ENABLE, 1);
 }
 
 /*
-- 
1.7.10.3

Re: [dpdk-dev] [PATCH] net/bnx2x: fix FW cmd timeout duting device stop

[Thomas Monjalon]

01/08/2018 20:19, Rasesh Mody:
> This patch fixes firmware command timeout error seen during device stop
> while stopping queues. It patially reverts an earlier preventive change
> commit 91b7e432bcef ("net/bnx2x: disable fast path interrupts") to now
> enable fast path interrupts.
> 
> The original issue of performance degradation is not observed anymore,
> with or without the fix.
> 
> Fixes: 91b7e432bcef ("net/bnx2x: disable fast path interrupts")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Rasesh Mody <rasesh.mody@cavium.com>

Applied, thanks