Hi Tejun and list,
This mail just to say I'm alive and testing the patch but so far no
errors (on my hardware).
Tejun Heo wrote:
sata_promise hardreset doesn't seem to be able to acquire the initial
D2H Reg FIS after hardreset leading to hardreset timeouts. Request
follow-up SRST.
http://article.gmane.org/gmane.linux.ide/36186
Signed-off-by: Tejun Heo <tj@xxxxxxxxxx>
---
Peter, can you please test this one too? It's essentially the same
code just slightly prettier. Mikael, what do you think about this?
I'm still testing the new patch (on 2.6.27.7), but so far I haven't had
any failures - it seems my hardware has started to behave well - very
unfortunate ;-)
I'll keep on testing - hopefully I'll see some of the usual errors soon
- and the new patch recovers alright.
Best regards,
I'm still testing the new patch (on 2.6.27.7), but so far I haven't had
any failures - it seems my hardware has started to behave well - very
unfortunate ;-)
> Jeff, please commit only after Peter's verification and Mikael's ACK.
I'll keep on testing - hopefully I'll see some of the usual errors soon
- and the new patch will recover alright. I'll keep you updated.
Best regards,
Peter
drivers/ata/sata_promise.c | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/drivers/ata/sata_promise.c b/drivers/ata/sata_promise.c
index ba9a257..917038a 100644
--- a/drivers/ata/sata_promise.c
+++ b/drivers/ata/sata_promise.c
@@ -710,7 +710,12 @@ static int pdc_sata_hardreset(struct ata_link *link, unsigned int *class,
unsigned long deadline)
{
pdc_reset_port(link->ap);
- return sata_sff_hardreset(link, class, deadline);
+
+ /* sata_promise can't reliably acquire the first D2H Reg FIS
+ * after hardreset. Do non-waiting hardreset and request
+ * follow-up SRST.
+ */
+ return sata_std_hardreset(link, class, deadline);
}
static void pdc_error_handler(struct ata_port *ap)
--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html