sata_promise: add ATA engine reset to reset ops
authorMikael Pettersson <mikpe@it.uu.se>
Fri, 31 Oct 2008 07:03:55 +0000 (08:03 +0100)
committerJeff Garzik <jgarzik@redhat.com>
Tue, 4 Nov 2008 06:08:03 +0000 (01:08 -0500)
commitcadef677e4a9b9c1d069675043767df486782986
tree054bd2b2f71af931c8692ff210a04bbe91450b2d
parenta75952b72a0fff3031124003e62118111aed42c1
sata_promise: add ATA engine reset to reset ops

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>
drivers/ata/sata_promise.c