scsi: qedf: Don't send ABTS for under run scenario
authorSaurav Kashyap <skashyap@marvell.com>
Tue, 26 Mar 2019 07:38:52 +0000 (00:38 -0700)
committerMartin K. Petersen <martin.petersen@oracle.com>
Thu, 28 Mar 2019 01:54:53 +0000 (21:54 -0400)
 - Command is already completed with underrun so no need to send ABTS.

Signed-off-by: Saurav Kashyap <skashyap@marvell.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/qedf/qedf_io.c

index eeb96c7..889b300 100644 (file)
@@ -1100,7 +1100,7 @@ static void qedf_unmap_sg_list(struct qedf_ctx *qedf, struct qedf_ioreq *io_req)
 void qedf_scsi_completion(struct qedf_ctx *qedf, struct fcoe_cqe *cqe,
        struct qedf_ioreq *io_req)
 {
-       u16 xid, rval;
+       u16 xid;
        struct e4_fcoe_task_context *task_ctx;
        struct scsi_cmnd *sc_cmd;
        struct fcoe_cqe_rsp_info *fcp_rsp;
@@ -1200,14 +1200,6 @@ void qedf_scsi_completion(struct qedf_ctx *qedf, struct fcoe_cqe *cqe,
                else
                        sc_cmd->result = (DID_OK << 16) | io_req->cdb_status;
 
-               /* Abort the command since we did not get all the data */
-               init_completion(&io_req->abts_done);
-               rval = qedf_initiate_abts(io_req, true);
-               if (rval) {
-                       QEDF_ERR(&(qedf->dbg_ctx), "Failed to queue ABTS.\n");
-                       sc_cmd->result = (DID_ERROR << 16) | io_req->cdb_status;
-               }
-
                /*
                 * Set resid to the whole buffer length so we won't try to resue
                 * any previously data.