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 ? Thanks, -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