On Wed, 2009-12-09 at 22:45 +0000, Russell King wrote: > On Wed, Dec 09, 2009 at 04:39:06PM -0600, James Bottomley wrote: > > Realistically, if I were thinking about the API properly, I'd probably > > have kunmap flush the kernel view if the dirty bit were set, thus > > relieving the user of the need to know anything at all about any of > > this. > > Which dirty bit are you referring to? > > New page cache pages do not have any dirty bit for this. There is > PG_arch_1, but this is set by flush_dcache_page on architectures > playing the delayed-dcache-flush game... so making a call to > some cache flushing function conditional on it isn't going to work. Sorry, was thinking of the one in the architecture page tables, not the one in struct page ... the only reason for checking is that page flush is an expensive op (at least on parisc) so we really don't want to do it if the page was never written to. James -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
