On Fri, 19 Dec 2008 11:21:25 +0900 KAMEZAWA Hiroyuki <kamezawa.hiroyu@xxxxxxxxxxxxxx> wrote: > On Thu, 18 Dec 2008 16:29:52 +0100 > Andrea Arcangeli <aarcange@xxxxxxxxxx> wrote: > > > @@ -484,11 +476,34 @@ > > if (page) { > > get_page(page); > > page_dup_rmap(page); > > + if (is_cow_mapping(vm_flags) && PageAnon(page)) { > > + if (unlikely(TestSetPageLocked(page))) > > + forcecow = 1; > > + else { > > + if (unlikely(page_count(page) != > > + page_mapcount(page) > > + + !!PageSwapCache(page))) > > + forcecow = 1; > > + unlock_page(page); > > + } > > + } > > rss[!!PageAnon(page)]++; > > } > - Why do you check only Anon rather than all MAP_PRIVATE mappings ? > Sorry, ignore this quesiton. -Kame -- 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