Messages in this thread | | | Date | Sat, 8 Jan 2005 13:56:36 -0800 | From | "David S. Miller" <> | Subject | Re: Prezeroing V3 [1/4]: Allow request for zeroed memory |
| |
On Sat, 8 Jan 2005 21:12:10 +0000 (GMT) Hugh Dickins <hugh@veritas.com> wrote:
> Christoph, a late comment: doesn't this effectively replace > do_anonymous_page's clear_user_highpage by clear_highpage, which would > be a bad idea (inefficient? or corrupting?) on those few architectures > which actually do something with that user addr?
Good catch, it probably does. We really do need to use the page clearing routines that pass in the user virtual address when preparing new anonymous pages or else we'll get cache aliasing problems on sparc, sparc64, and mips at the very least. That is what the virtual address argument was added for to begin with.
The other way to deal with this is to make whatever routine the kscrubd thing invokes do all the cache flushing et al. magic so that the above works when taking pages from the pre-zero'd pool (only, if no pre-zero'd pages are available we sill need to invoke clear_user_highpage() with the proper virtual address). - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |