scsi: hisi_sas: remove task free'ing for timeouts
authorJohn Garry <john.garry@huawei.com>
Wed, 22 Mar 2017 17:25:30 +0000 (01:25 +0800)
committerMartin K. Petersen <martin.petersen@oracle.com>
Thu, 23 Mar 2017 15:12:02 +0000 (11:12 -0400)
When a TMF or internal abort times-out, do not free slot. We expect this
to be done upon later escalated error handling.

Signed-off-by: John Garry <john.garry@huawei.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/hisi_sas/hisi_sas_main.c

index f9ea5ccd1e844ade32fe754c3f30cee1f34e8b18..3d63a24fea5426bbe1faf139ea90e6a4336ab614 100644 (file)
@@ -743,14 +743,6 @@ static int hisi_sas_exec_internal_tmf_task(struct domain_device *device,
                if ((task->task_state_flags & SAS_TASK_STATE_ABORTED)) {
                        if (!(task->task_state_flags & SAS_TASK_STATE_DONE)) {
                                dev_err(dev, "abort tmf: TMF task timeout\n");
-                               if (task->lldd_task) {
-                                       struct hisi_sas_slot *slot =
-                                               task->lldd_task;
-
-                                       hisi_sas_slot_task_free(hisi_hba,
-                                                               task, slot);
-                               }
-
                                goto ex_err;
                        }
                }
@@ -1248,15 +1240,10 @@ hisi_sas_internal_task_abort(struct hisi_hba *hisi_hba,
                goto exit;
        }
 
-       /* TMF timed out, return direct. */
+       /* Internal abort timed out */
        if ((task->task_state_flags & SAS_TASK_STATE_ABORTED)) {
                if (!(task->task_state_flags & SAS_TASK_STATE_DONE)) {
                        dev_err(dev, "internal task abort: timeout.\n");
-                       if (task->lldd_task) {
-                               struct hisi_sas_slot *slot = task->lldd_task;
-
-                               hisi_sas_slot_task_free(hisi_hba, task, slot);
-                       }
                }
        }