diff options
| author | Alan Cox <alan@lxorguk.ukuu.org.uk> | 2006-03-21 15:53:46 +0000 |
|---|---|---|
| committer | Jeff Garzik <jeff@garzik.org> | 2006-03-21 21:21:32 -0500 |
| commit | 2d2020242d14dc8caf83d7682d91368cff64290c (patch) | |
| tree | 75dbc3bdd0e6119139e7a498aeb47000e099fa78 /drivers | |
| parent | b6782728d703fa3f0e5478a8b89e49ea10b1fdd0 (diff) | |
[PATCH] libata: pick a less confusion "um dunno" error
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/scsi/libata-scsi.c | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/drivers/scsi/libata-scsi.c b/drivers/scsi/libata-scsi.c index fc547ca295f..dd954a23b35 100644 --- a/drivers/scsi/libata-scsi.c +++ b/drivers/scsi/libata-scsi.c @@ -511,13 +511,11 @@ void ata_to_sense_error(unsigned id, u8 drv_stat, u8 drv_err, u8 *sk, u8 *asc, printk(KERN_WARNING "ata%u: no sense translation for status: 0x%02x\n", id, drv_stat); - /* For our last chance pick, use medium read error because - * it's much more common than an ATA drive telling you a write - * has failed. - */ - *sk = MEDIUM_ERROR; - *asc = 0x11; /* "unrecovered read error" */ - *ascq = 0x04; /* "auto-reallocation failed" */ + /* We need a sensible error return here, which is tricky, and one + that won't cause people to do things like return a disk wrongly */ + *sk = ABORTED_COMMAND; + *asc = 0x00; + *ascq = 0x00; translate_done: printk(KERN_ERR "ata%u: translated ATA stat/err 0x%02x/%02x to " |