[SCSI] libsas: kill invocation of scsi_eh_finish_cmd from sas_ata_task_done
authorDan Williams <dan.j.williams@intel.com>
Tue, 29 Nov 2011 01:11:33 +0000 (17:11 -0800)
committerJames Bottomley <JBottomley@Parallels.com>
Sun, 19 Feb 2012 19:57:01 +0000 (13:57 -0600)
Prior to the conversion to the new-style libata-eh sas_ata_task_done()
may have been the last opportunity to clean up the scmd, but now
libata-eh explicitly handles this case.  It also races against sas-eh.
If a lldd completes a task after SAS_TASK_STATE_ABORTED is set it could
trigger a spurious decrement of shost->host_failed.  Current lldds have
the band-aid of checking SAS_TASK_STATE_ABORTED before calling
->task_done(), but better to just let the scmds escalate to libata for
race free cleanup.

Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
drivers/scsi/libsas/sas_ata.c

index 71af919b856c1a79bbc7d599c527a79064e2c25c..5cb0a2ae5924a62a574c38af49060ee0fe76b44b 100644 (file)
@@ -145,20 +145,6 @@ static void sas_ata_task_done(struct sas_task *task)
        ata_qc_complete(qc);
        spin_unlock_irqrestore(dev->sata_dev.ap->lock, flags);
 
-       /*
-        * If the sas_task has an ata qc, a scsi_cmnd and the aborted
-        * flag is set, then we must have come in via the libsas EH
-        * functions.  When we exit this function, we need to put the
-        * scsi_cmnd on the list of finished errors.  The ata_qc_complete
-        * call cleans up the libata side of things but we're protected
-        * from the scsi_cmnd going away because the scsi_cmnd is owned
-        * by the EH, making libata's call to scsi_done a NOP.
-        */
-       spin_lock_irqsave(&task->task_state_lock, flags);
-       if (qc->scsicmd && task->task_state_flags & SAS_TASK_STATE_ABORTED)
-               scsi_eh_finish_cmd(qc->scsicmd, &sas_ha->eh_done_q);
-       spin_unlock_irqrestore(&task->task_state_lock, flags);
-
 qc_already_gone:
        list_del_init(&task->list);
        sas_free_task(task);