diff options
| author | Tejun Heo <htejun@gmail.com> | 2007-07-16 01:29:41 -0400 |
|---|---|---|
| committer | Jeff Garzik <jeff@garzik.org> | 2007-07-20 08:26:26 -0400 |
| commit | 4e57c517b3cbaceb7438eeec879ca129fc17442c (patch) | |
| tree | c77ee2e10ff0b9d83d454937e5890c38314cbabf /drivers/ata/libata-eh.c | |
| parent | fccb6ea5c240b9f29baa55448488fd6aee49f5a5 (diff) | |
libata: schedule probing after SError access failure during autopsy
If SError isn't accessible, EH can't tell whether hotplug has happened
or not. Report SError read failure with AC_ERR_OTHER and schedule
probing with hardreset. This will be mainly useful for PMPs.
Signed-off-by: Tejun Heo <htejun@gmail.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/ata/libata-eh.c')
| -rw-r--r-- | drivers/ata/libata-eh.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/drivers/ata/libata-eh.c b/drivers/ata/libata-eh.c index a5a8f8453061..e7e2ba24ce66 100644 --- a/drivers/ata/libata-eh.c +++ b/drivers/ata/libata-eh.c | |||
| @@ -1478,8 +1478,12 @@ static void ata_eh_autopsy(struct ata_port *ap) | |||
| 1478 | if (rc == 0) { | 1478 | if (rc == 0) { |
| 1479 | ehc->i.serror |= serror; | 1479 | ehc->i.serror |= serror; |
| 1480 | ata_eh_analyze_serror(ap); | 1480 | ata_eh_analyze_serror(ap); |
| 1481 | } else if (rc != -EOPNOTSUPP) | 1481 | } else if (rc != -EOPNOTSUPP) { |
| 1482 | /* SError read failed, force hardreset and probing */ | ||
| 1483 | ata_ehi_schedule_probe(&ehc->i); | ||
| 1482 | ehc->i.action |= ATA_EH_HARDRESET; | 1484 | ehc->i.action |= ATA_EH_HARDRESET; |
| 1485 | ehc->i.err_mask |= AC_ERR_OTHER; | ||
| 1486 | } | ||
| 1483 | 1487 | ||
| 1484 | /* analyze NCQ failure */ | 1488 | /* analyze NCQ failure */ |
| 1485 | ata_eh_analyze_ncq_error(ap); | 1489 | ata_eh_analyze_ncq_error(ap); |
