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 <albertcc@xxxxxxxxxx> --- 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html