On Tue, 2010-02-02 at 15:21 -0800, David Miller wrote: > From: Jeff Garzik <jgarzik@xxxxxxxxx> > Date: Tue, 02 Feb 2010 18:14:40 -0500 > > > The patch in question only affects PIO transfers, not DMA. Data is > > transferred from a kernel buffer to hardware via out[bwl] via > > > > page data -> CPU register -> out[bwl] > > > > or, data is transferred from hardware to a kernel buffer via > > > > in[bwl] -> CPU register -> page data > > > > So what are the flushing rules given those conditions? > > Any time you touch a page cache page with the cpu, you have > to get it purged from aliasing caches before it gets mapped > into userspace or similar. > > That's the problem on these machines. > > Actually, sparc64 is probably susceptible to the same problem. > > And it's not an issue in the IDE layer, know why? :-) > > In the IDE layer we have arch specific ide_*() interfaces which is > where all of this PIO dma flushing stuff deserves to live. > > ATA should do something similar instead of randomly scattering > flush_dcache_page() calls all over the place. Agree with this, except it shouldn't be ide specific ... any driver that does PIO (we have some examples in SCSI) is vulnerable to this issue and should have the same fix. The ide interfaces are very low level .. they're the ide_in/out[sw] APIs which aren't necessarily the most useful (some of the SCSI "PIO" stuff actually gets fed in via memory transfers to registers rather than port in/out). What it looks like we need is kmap_for_pio() and kunmap_for_pio() which take flags (or API extensions) indicating whether we're transferring to or from the device, which return the correct kernel address to the page (mapping it if it's in highmem) and perform the correct flushes/invalidates. James -- 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