|
Daniel Wagner |
3159e4 |
From: Dan Carpenter <dan.carpenter@oracle.com>
|
|
Daniel Wagner |
3159e4 |
Date: Wed, 22 Jun 2022 09:21:55 +0300
|
|
Daniel Wagner |
3159e4 |
Subject: scsi: qla2xxx: Check correct variable in qla24xx_async_gffid()
|
|
Denis Kirjanov |
718367 |
Patch-mainline: v5.20-rc1
|
|
Daniel Wagner |
3159e4 |
Git-commit: 7c33e477bd883f79cccec418980cb8f7f2d50347
|
|
Daniel Wagner |
3159e4 |
References: bsc#1201958
|
|
Daniel Wagner |
3159e4 |
|
|
Daniel Wagner |
3159e4 |
There is a copy and paste bug here. It should check ".rsp" instead of
|
|
Daniel Wagner |
3159e4 |
".req". The error message is copy and pasted as well so update that too.
|
|
Daniel Wagner |
3159e4 |
|
|
Daniel Wagner |
3159e4 |
Link: https://lore.kernel.org/r/YrK1A/t3L6HKnswO@kili
|
|
Daniel Wagner |
3159e4 |
Fixes: 9c40c36e75ff ("scsi: qla2xxx: edif: Reduce Initiator-Initiator thrashing")
|
|
Daniel Wagner |
3159e4 |
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
|
|
Daniel Wagner |
3159e4 |
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
|
|
Daniel Wagner |
3159e4 |
Acked-by: Daniel Wagner <dwagner@suse.de>
|
|
Daniel Wagner |
3159e4 |
---
|
|
Daniel Wagner |
3159e4 |
drivers/scsi/qla2xxx/qla_gs.c | 4 ++--
|
|
Daniel Wagner |
3159e4 |
1 file changed, 2 insertions(+), 2 deletions(-)
|
|
Daniel Wagner |
3159e4 |
|
|
Daniel Wagner |
3159e4 |
--- a/drivers/scsi/qla2xxx/qla_gs.c
|
|
Daniel Wagner |
3159e4 |
+++ b/drivers/scsi/qla2xxx/qla_gs.c
|
|
Daniel Wagner |
3159e4 |
@@ -3390,9 +3390,9 @@ int qla24xx_async_gffid(scsi_qla_host_t
|
|
Daniel Wagner |
3159e4 |
sp->u.iocb_cmd.u.ctarg.rsp_allocated_size,
|
|
Daniel Wagner |
3159e4 |
&sp->u.iocb_cmd.u.ctarg.rsp_dma,
|
|
Daniel Wagner |
3159e4 |
GFP_KERNEL);
|
|
Daniel Wagner |
3159e4 |
- if (!sp->u.iocb_cmd.u.ctarg.req) {
|
|
Daniel Wagner |
3159e4 |
+ if (!sp->u.iocb_cmd.u.ctarg.rsp) {
|
|
Daniel Wagner |
3159e4 |
ql_log(ql_log_warn, vha, 0xd041,
|
|
Daniel Wagner |
3159e4 |
- "%s: Failed to allocate ct_sns request.\n",
|
|
Daniel Wagner |
3159e4 |
+ "%s: Failed to allocate ct_sns response.\n",
|
|
Daniel Wagner |
3159e4 |
__func__);
|
|
Daniel Wagner |
3159e4 |
goto done_free_sp;
|
|
Daniel Wagner |
3159e4 |
}
|