Messages in this thread | | | Date | Fri, 5 Mar 2010 21:16:29 +0000 | From | Russell King - ARM Linux <> | Subject | Re: [RFC PATCH] ARM: Assume new page cache pages have dirty D-cache |
| |
On Fri, Mar 05, 2010 at 04:52:40PM +0000, Catalin Marinas wrote: > As Ben said, I think we can set PG_dcache_clean in the > clear/copy_user_page() functions. My doubt with these functions is the > highmem cases where kunmap_atomic() only flushes the D-cache in one > situation, the other just calling kunmap_high() which doesn't seem to do > anything to the caches.
In which case you're totally missing the point with these functions. The copy_user_page and clear_user_page functions specifically do tricks to ensure that they can avoid additional cache maintainence - or any cache maintainence at all.
For instance, on aliasing VIPT, they will map the user page in using the same colour as the ultimate userspace address, ensuring that any cache lines created will be visible to the userspace application.
So what kunmap_atomic() does with caches is not really relevant to the coherency issue.
| |