sata_promise: add ATA engine reset to reset ops
commit7bee179aa2a63087ff80bdd7dcb8bd6da209f359
authorMikael Pettersson <mikpe@it.uu.se>
Fri, 31 Oct 2008 07:03:55 +0000 (31 08:03 +0100)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 7 Nov 2008 03:05:51 +0000 (6 19:05 -0800)
tree957612796163b9d2024b52ab497505c131f2165d
parentdcfdc4faa0dbe2aa4c68a43c79dc7eb630788351
sata_promise: add ATA engine reset to reset ops

commit cadef677e4a9b9c1d069675043767df486782986 upstream

Promise ATA engines need to be reset when errors occur.
That's currently done for errors detected by sata_promise itself,
but it's not done for errors like timeouts detected outside of
the low-level driver.

The effect of this omission is that a timeout tends to result
in a sequence of failed COMRESETs after which libata EH gives
up and disables the port. At that point the port's ATA engine
hangs and even reloading the driver will not resume it.

To fix this, make sata_promise override ->hardreset on SATA
ports with code which calls pdc_reset_port() on the port in
question before calling libata's hardreset. PATA ports don't
use ->hardreset, so for those we override ->softreset instead.

Signed-off-by: Mikael Pettersson <mikpe@it.uu.se>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/ata/sata_promise.c