qla2xxx: Fix START_SP_W_RETRIES checks _rval with positive EAGAIN

Commit Message

(학생) 장인규 (컴퓨터공학과) Nov. 1, 2024, 12:27 a.m. UTC

From 6f450e20d42c29c623654553831bcba74a566da4 Mon Sep 17 00:00:00 2001
From: Ingyu Jang <[e-mail]>
Date: Fri, 1 Nov 2024 09:17:10 +0900
Subject: [PATCH] qla2xxx: Fix  START_SP_W_RETRIES checks _rval with positive
 EAGAIN

The START_SP_W_RETRIES macro incorrectly checked _rval with a positive
EAGAIN, despite qla2x00_start_sp consistently returning -EAGAIN for
retriable conditions. This mismatch could cause improper looping
behavior, failing to retry as intended and potentially leading to
unintended results in the system.

This patch fixes the macro to handle EAGAIN as -EAGAIN, ensuring
consistent error handling.
This change prevents any potential misinterpretation that could
introduce unexpected behavior.

Signed-off-by: Ingyu Jang <[e-mail]>
---
 drivers/scsi/qla2xxx/qla_init.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Patch

diff --git a/drivers/scsi/qla2xxx/qla_init.c b/drivers/scsi/qla2xxx/qla_init.c
index 31fc6a0eca3e..763bdcc47567 100644
--- a/drivers/scsi/qla2xxx/qla_init.c
+++ b/drivers/scsi/qla2xxx/qla_init.c
@@ -2063,7 +2063,7 @@  static void qla_marker_sp_done(srb_t *sp, int res)
 			break; \
 		} \
 		_rval = qla2x00_start_sp(_sp); \
-		if (_rval == EAGAIN) \
+		if (_rval == -EAGAIN) \
 			msleep(1); \
 		else \
 			break; \