patch 2/7:
 Calling ata_altstatus() after the qc completed looks incorrect.
Move it to before the qc is completed.

Signed-off-by: Albert Lee <[EMAIL PROTECTED]>
---

diff -Nrup 01_last_idle/drivers/ata/libata-core.c 
02_flush_fix/drivers/ata/libata-core.c
--- 01_last_idle/drivers/ata/libata-core.c      2007-05-11 10:24:16.000000000 
+0800
+++ 02_flush_fix/drivers/ata/libata-core.c      2007-05-11 11:14:04.000000000 
+0800
@@ -4329,6 +4329,8 @@ static void ata_hsm_qc_complete(struct a
        struct ata_port *ap = qc->ap;
        unsigned long flags;
 
+       ata_altstatus(ap); /* flush */
+
        if (ap->ops->error_handler) {
                if (in_wq) {
                        spin_lock_irqsave(ap->lock, flags);
@@ -4361,8 +4363,6 @@ static void ata_hsm_qc_complete(struct a
                } else
                        ata_qc_complete(qc);
        }
-
-       ata_altstatus(ap); /* flush */
 }
 
 /**


-
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