Douglas Gilbert wrote:
Alan Stern wrote:
James:
This is an updated and unmangled version of the patch sent in by
Martin Peschke. Apparently some drives report Hardware Error sense
for problems which do improve after retrying, so the patch retries
these supposedly "unrecoverable" errors for such devices.
Recent SPC-3 and SBC-2 drafts treat the sense keys of
MEDIUM ERROR and HARDWARE ERROR in a similar way.
Both can return an "info" field which has the same
meaning (lba of first failure). The distinction is that
MEDIUM ERROR is a little more precise (at least for
magnetic rotating media) **. For flash ram the distinction
is moot.
I believe MEDIUM ERROR and HARDWARE ERROR should be
treated the same way in scsi_check_sense() (i.e.
both return NEEDS_RETRY). That way an extra black list
category is avoided.
** HARDWARE ERROR is returned in cases of self diagnostic
failure and lack of available blocks for reassignment.
It seems valid for a device to return a HARDWARE ERROR
sense key both for these cases and unrecoverable data
errors (and ignore MEDIUM ERROR).
... after a bit further thought, a retry (arguably) is only
needed when an unrecoverable (data) error is detected. If we
assume the "info" field indicates an unrecoverable error
then the following patch combines the processing of
MEDIUM and HARDWARE ERROR sense keys without the need for
a black list category.
Doug Gilbert
--- linux/drivers/scsi/scsi_error.c 2005-02-13 20:46:31.000000000 +1000
+++ linux/drivers/scsi/scsi_error.c2611rc4he 2005-02-17 14:55:44.000000000 +1000
@@ -279,6 +279,7 @@
static int scsi_check_sense(struct scsi_cmnd *scmd)
{
struct scsi_sense_hdr sshdr;
+ u64 info;
if (! scsi_command_normalize_sense(scmd, &sshdr))
return FAILED; /* no valid sense data */
@@ -348,12 +349,15 @@
return SUCCESS;
case MEDIUM_ERROR:
- return NEEDS_RETRY;
-
+ case HARDWARE_ERROR:
+ if (scsi_get_sense_info_fld(scmd->sense_buffer,
+ sizeof(scmd->sense_buffer), &info))
+ return NEEDS_RETRY;
+ else
+ return SUCCESS;
case ILLEGAL_REQUEST:
case BLANK_CHECK:
case DATA_PROTECT:
- case HARDWARE_ERROR:
default:
return SUCCESS;
}
