scsi: qla2xxx: Remove check req_sg_cnt should be equal to rsp_sg_cnt

commit 833c70e212fc40d3e98da941796f4c7bcaecdf58 upstream.

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>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/scsi/qla2xxx/qla_bsg.c b/drivers/scsi/qla2xxx/qla_bsg.c
index d415e816ad0eb61fdb556da69294659893385956..4324f4643e8340b1879c054622c28931dfa46c01 100644 (file)
--- a/drivers/scsi/qla2xxx/qla_bsg.c
+++ b/drivers/scsi/qla2xxx/qla_bsg.c
@@ -471,16 +471,6 @@ qla2x00_process_ct(struct bsg_job *bsg_job)
                goto done;
        }
 
-       if ((req_sg_cnt !=  bsg_job->request_payload.sg_cnt) ||
-           (rsp_sg_cnt != bsg_job->reply_payload.sg_cnt)) {
-               ql_log(ql_log_warn, vha, 0x7011,
-                   "request_sg_cnt: %x dma_request_sg_cnt: %x reply_sg_cnt:%x "
-                   "dma_reply_sg_cnt: %x\n", bsg_job->request_payload.sg_cnt,
-                   req_sg_cnt, bsg_job->reply_payload.sg_cnt, rsp_sg_cnt);
-               rval = -EAGAIN;
-               goto done_unmap_sg;
-       }
-
        if (!vha->flags.online) {
                ql_log(ql_log_warn, vha, 0x7012,
                    "Host is not online.\n");