summaryrefslogtreecommitdiffstats
path: root/drivers/ata
diff options
context:
space:
mode:
authorCatalin Marinas <catalin.marinas@arm.com>2010-02-04 01:04:50 -0500
committerJeff Garzik <jgarzik@redhat.com>2010-02-04 01:04:50 -0500
commit2d68b7fe55d9e19a8a868224ed0dfd6526568521 (patch)
treef18068b57359ddc0be3d5fba6f8eabc097efd1af /drivers/ata
parentcedc9bf906dae044443d403371c887affdb44168 (diff)
downloadop-kernel-dev-2d68b7fe55d9e19a8a868224ed0dfd6526568521.zip
op-kernel-dev-2d68b7fe55d9e19a8a868224ed0dfd6526568521.tar.gz
[libata] Call flush_dcache_page after PIO data transfers in libata-sff.c
flush_dcache_page() must be called after (!ATA_TFLAG_WRITE) the data copying to avoid D-cache aliasing with user space or I-D cache coherency issues (when reading data from an ATA device using PIO, the kernel dirties the D-cache but there is no flush_dcache_page() required on Harvard architectures). Signed-off-by: Catalin Marinas <catalin.marinas@arm.com> Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Diffstat (limited to 'drivers/ata')
-rw-r--r--drivers/ata/libata-sff.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/ata/libata-sff.c b/drivers/ata/libata-sff.c
index 741065c..730ef3c 100644
--- a/drivers/ata/libata-sff.c
+++ b/drivers/ata/libata-sff.c
@@ -893,6 +893,9 @@ static void ata_pio_sector(struct ata_queued_cmd *qc)
do_write);
}
+ if (!do_write)
+ flush_dcache_page(page);
+
qc->curbytes += qc->sect_size;
qc->cursg_ofs += qc->sect_size;
OpenPOWER on IntegriCloud