On Fri, 20 Oct 2006, Russell King wrote:
> Well, looking at do_wp_page() I'm now quite concerned about ARM and COW.
> I can't see how this code could _possibly_ work with a virtually indexed
> cache as it stands. Yet, the kernel does appear to work.
It really shouldn't need any extra code, exactly because by the time it
hits any page-fault, the caches had better be in sync with the physical
page contents _anyway_ (yes, being virtual, the caches will _duplicate_
the contents, but since the pages are read-only, that aliasing should be
> I'm afraid I'm utterly confused with the Linux MM in this day and age, so
> I don't think I can even consider commenting on this change.
Well, we'd need somebody to verify that it still works, but quite frankly,
the likelihood of it breaking anything seems basically nil.
> However, when I look at this code now, I see _no where_ where we synchronise
> the cache between the userspace mapping and the kernel space mapping before
> copying a COW page.
At the COW, it should be synchronized already, exactly because we did the
cache_flush_mm() when we _created_ the COW mapping in the first place.
It's just that we weren't quite careful enough at that time (and even
then, that would only matter for some really really unlikely and strange
situations that only happen when you fork() from a _threaded_ environment,
so it shouldn't be anything you'd notice under normal load).
> So I'm afraid I'm going to have to hold up my hand and say "I don't
> understand the Linux MM anymore".
There are few enough people who understand it even though they're supposed
to. I certainly have to always go back and look and read the code when
there is anything subtle going on, and even then I want to be backed up by
one of the _competent_ people ;)
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Video 4 Linux]
[Linux for the blind]