On Thu, May 09, 2013 at 01:12:43PM -0700, Linus Torvalds wrote: > On Thu, May 9, 2013 at 12:26 PM, Al Viro <viro@xxxxxxxxxxxxxxxxxx> wrote: > > Regression fix from Geert + yet another open-coded kernel_read(). > > Hmm. I get one more commit - a racy usbmon thing. Oops - pull stats not updated... My apologies. > Which looks fine per > se, but would look even better if it just made mmap_active an atomic_t > instead, wouldn't you say? The only place checking that sucker is in a fairly large area protected by ->b_lock (in mon_bin_event()); I really don't want to dig deep enough to tell if having it changed right after it had been checked is safe. OTOH, from a cursory look through that code it appears that the whole map_count thing is completely misguided - it seems to assume that ->open() is called for each VMA, including the one created by mmap(2). Sigh... OK, it might make more sense to drop that last commit for now. It's obviously more broken in mmap-related code than just a buggered locking for counter and it probably doesn't make much sense to patch just that one. And I'll have several more fixes of similar turds tomorrow, when mchehab finishes testing videobuf ones. Up to you, really. -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
