Skip to content

Commit 833c70e

Browse files
Saurav Kashyapmartinkpetersen
authored andcommitted
scsi: qla2xxx: Remove check req_sg_cnt should be equal to rsp_sg_cnt
Firmware supports multiple sg_cnt for request and response for CT commands, so remove the redundant check. A check is there where sg_cnt for request and response should be same. This is not required as driver and FW have code to handle multiple and different sg_cnt on request and response. Cc: stable@vger.kernel.org Signed-off-by: Saurav Kashyap <skashyap@marvell.com> Signed-off-by: Nilesh Javali <njavali@marvell.com> Link: https://lore.kernel.org/r/20241115130313.46826-5-njavali@marvell.com Reviewed-by: Himanshu Madhani <himanshu.madhani@oracle.com> Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
1 parent 07c903d commit 833c70e

File tree

1 file changed

+0
-10
lines changed

1 file changed

+0
-10
lines changed

drivers/scsi/qla2xxx/qla_bsg.c

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -494,16 +494,6 @@ qla2x00_process_ct(struct bsg_job *bsg_job)
494494
goto done;
495495
}
496496

497-
if ((req_sg_cnt != bsg_job->request_payload.sg_cnt) ||
498-
(rsp_sg_cnt != bsg_job->reply_payload.sg_cnt)) {
499-
ql_log(ql_log_warn, vha, 0x7011,
500-
"request_sg_cnt: %x dma_request_sg_cnt: %x reply_sg_cnt:%x "
501-
"dma_reply_sg_cnt: %x\n", bsg_job->request_payload.sg_cnt,
502-
req_sg_cnt, bsg_job->reply_payload.sg_cnt, rsp_sg_cnt);
503-
rval = -EAGAIN;
504-
goto done_unmap_sg;
505-
}
506-
507497
if (!vha->flags.online) {
508498
ql_log(ql_log_warn, vha, 0x7012,
509499
"Host is not online.\n");

0 commit comments

Comments
 (0)