scsi: sd: Return SUCCESS in sd_eh_action() after device offline
authorHannes Reinecke <hare@suse.de>
Thu, 6 Apr 2017 13:36:30 +0000 (15:36 +0200)
committerMartin K. Petersen <martin.petersen@oracle.com>
Thu, 6 Apr 2017 17:07:32 +0000 (13:07 -0400)
If sd_eh_action() decides to take the device offline there is
no point in returning FAILED, as taking the device offline
is the ultimate step in SCSI EH anyway.
So further escalation via SCSI EH is not likely to make a
difference and we can as well return SUCCESS.

Cc: Benjamin Block <bblock@linux.vnet.ibm.com>
Signed-off-by: Hannes Reinecke <hare@suse.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/sd.c

index bd2a38ef46f51211ca237e7517652a74cf2ad686..00b168b6a5fcc7dff17690e8af324af36dfb836a 100644 (file)
@@ -1751,7 +1751,7 @@ static int sd_eh_action(struct scsi_cmnd *scmd, int eh_disp)
                            "Medium access timeout failure. Offlining disk!\n");
                scsi_device_set_state(scmd->device, SDEV_OFFLINE);
 
-               return FAILED;
+               return SUCCESS;
        }
 
        return eh_disp;