From: Hannes Reinecke <hare@suse.de>
Date: Wed, 17 Jul 2013 13:31:48 +0200
Subject: scsi: Always retry internal target error
References: bnc#745640, bnc#825227
Patch-Mainline: n/a
EMC Symmetrix is returning 'internal target error' for a variety
of conditions, most of which will be transient.
So we should always retry it, even with failfast set.
Otherwise we'd get spurious path flaps with multipath.
Signed-off-by: Hannes Reinecke <hare@suse.de>
diff --git a/drivers/scsi/scsi_error.c b/drivers/scsi/scsi_error.c
index c51b834..86d3975 100644
--- a/drivers/scsi/scsi_error.c
+++ b/drivers/scsi/scsi_error.c
@@ -382,6 +382,16 @@ static int scsi_check_sense(struct scsi_cmnd *scmd)
if (sshdr.asc == 0x10) /* DIF */
return SUCCESS;
+ if (!strncmp(scmd->device->vendor, "EMC", 3) &&
+ !strncmp(scmd->device->model, "SYMMETRIX", 9) &&
+ (sshdr.asc == 0x44) && (sshdr.ascq == 0x0)) {
+ /*
+ * EMC Symmetrix returns 'Internal target failure'
+ * for a variety of internal issues, all of which
+ * can be recovered by retry.
+ */
+ return ADD_TO_MLQUEUE;
+ }
return NEEDS_RETRY;
case NOT_READY:
case UNIT_ATTENTION:
@@ -1714,6 +1724,9 @@ int scsi_decide_disposition(struct scsi_cmnd *scmd)
rtn = scsi_check_sense(scmd);
if (rtn == NEEDS_RETRY)
goto maybe_retry;
+ else if (rtn == ADD_TO_MLQUEUE)
+ /* Always enforce a retry for ADD_TO_MLQUEUE */
+ rtn = NEEDS_RETRY;
/* if rtn == FAILED, we have no sense information;
* returning FAILED will wake the error handler thread
* to collect the sense and redo the decide