Tejun Heo wrote:
07_NCQ_ahci-stop-dma-before-reset.patch

        AHCI 1.1 mandates stopping dma before issueing COMMRESET.  The
        original code didn't and it resulted in occasional lockup of
        the controller during EH recovery.  This patch fixes the
        problem.

Signed-off-by: Tejun Heo <[EMAIL PROTECTED]>

 ahci.c |    2 ++
 1 files changed, 2 insertions(+)

Index: work/drivers/scsi/ahci.c
===================================================================
--- work.orig/drivers/scsi/ahci.c       2005-06-27 00:20:31.000000000 +0900
+++ work/drivers/scsi/ahci.c    2005-06-27 00:20:31.000000000 +0900
@@ -474,7 +474,9 @@ static void ahci_phy_reset(struct ata_po
        struct ata_device *dev = &ap->device[0];
        u32 tmp;
+ ahci_stop_dma(ap);
        __sata_phy_reset(ap);
+       ahci_start_dma(ap);

This is a bit worrisome, because we really shouldn't be calling ahci_phy_reset() when DMA is -not- stopped. That's a violation of the state machine.

        Jeff



-
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to