diff for duplicates of <20100402085251.GD621@csn.ul.ie> diff --git a/a/1.txt b/N1/1.txt index e964f1d..88eceb4 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -3,25 +3,25 @@ On Fri, Apr 02, 2010 at 09:21:50AM +0900, KAMEZAWA Hiroyuki wrote: > Mel Gorman <mel@csn.ul.ie> wrote: > > > > > == -> > > > skip_remap = 0; -> > > > if (PageAnon(page)) { -> > > > rcu_read_lock(); -> > > > if (!page_mapped(page)) { -> > > > if (!PageSwapCache(page)) -> > > > goto rcu_unlock; -> > > > /* -> > > > * We can't convice this anon_vma is valid or not because -> > > > * !page_mapped(page). Then, we do migration(radix-tree replacement) -> > > > * but don't remap it which touches anon_vma in page->mapping. -> > > > */ -> > > > skip_remap = 1; -> > > > goto skip_unmap; -> > > > } else { -> > > > anon_vma = page_anon_vma(page); -> > > > atomic_inc(&anon_vma->external_refcount); -> > > > } -> > > > } -> > > > .....copy page, radix-tree replacement,.... +> > > > skip_remap = 0; +> > > > if (PageAnon(page)) { +> > > > rcu_read_lock(); +> > > > if (!page_mapped(page)) { +> > > > if (!PageSwapCache(page)) +> > > > goto rcu_unlock; +> > > > /* +> > > > * We can't convice this anon_vma is valid or not because +> > > > * !page_mapped(page). Then, we do migration(radix-tree replacement) +> > > > * but don't remap it which touches anon_vma in page->mapping. +> > > > */ +> > > > skip_remap = 1; +> > > > goto skip_unmap; +> > > > } else { +> > > > anon_vma = page_anon_vma(page); +> > > > atomic_inc(&anon_vma->external_refcount); +> > > > } +> > > > } +> > > > .....copy page, radix-tree replacement,.... > > > > > > > > > > It's not enough. @@ -113,3 +113,9 @@ True, done. Mel Gorman Part-time Phd Student Linux Technology Center University of Limerick IBM Dublin Software Lab + +-- +To unsubscribe, send a message with 'unsubscribe linux-mm' in +the body to majordomo@kvack.org. For more info on Linux MM, +see: http://www.linux-mm.org/ . +Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a> diff --git a/a/content_digest b/N1/content_digest index c18a506..1b85998 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -30,25 +30,25 @@ "> Mel Gorman <mel@csn.ul.ie> wrote:\n" "> \n" "> > > > ==\n" - "> > > > \302\240 \302\240 \302\240 \302\240skip_remap = 0;\n" - "> > > > \302\240 \302\240 \302\240 \302\240if (PageAnon(page)) {\n" - "> > > > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240rcu_read_lock();\n" - "> > > > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240if (!page_mapped(page)) {\n" - "> > > > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240if (!PageSwapCache(page))\n" - "> > > > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240goto rcu_unlock;\n" - "> > > > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240/*\n" - "> > > > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 * We can't convice this anon_vma is valid or not because\n" - "> > > > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 * !page_mapped(page). Then, we do migration(radix-tree replacement)\n" - "> > > > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 * but don't remap it which touches anon_vma in page->mapping.\n" - "> > > > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 */\n" - "> > > > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240skip_remap = 1;\n" - "> > > > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240goto skip_unmap;\n" - "> > > > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240} else {\n" - "> > > > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240anon_vma = page_anon_vma(page);\n" - "> > > > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240atomic_inc(&anon_vma->external_refcount);\n" - "> > > > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240}\n" - "> > > > \302\240 \302\240 \302\240 \302\240}\n" - "> > > > \302\240 \302\240 \302\240 \302\240.....copy page, radix-tree replacement,....\n" + "> > > > skip_remap = 0;\n" + "> > > > if (PageAnon(page)) {\n" + "> > > > rcu_read_lock();\n" + "> > > > if (!page_mapped(page)) {\n" + "> > > > if (!PageSwapCache(page))\n" + "> > > > goto rcu_unlock;\n" + "> > > > /*\n" + "> > > > * We can't convice this anon_vma is valid or not because\n" + "> > > > * !page_mapped(page). Then, we do migration(radix-tree replacement)\n" + "> > > > * but don't remap it which touches anon_vma in page->mapping.\n" + "> > > > */\n" + "> > > > skip_remap = 1;\n" + "> > > > goto skip_unmap;\n" + "> > > > } else {\n" + "> > > > anon_vma = page_anon_vma(page);\n" + "> > > > atomic_inc(&anon_vma->external_refcount);\n" + "> > > > }\n" + "> > > > }\n" + "> > > > .....copy page, radix-tree replacement,....\n" "> > > >\n" "> > > \n" "> > > It's not enough.\n" @@ -139,6 +139,12 @@ "-- \n" "Mel Gorman\n" "Part-time Phd Student Linux Technology Center\n" - University of Limerick IBM Dublin Software Lab + "University of Limerick IBM Dublin Software Lab\n" + "\n" + "--\n" + "To unsubscribe, send a message with 'unsubscribe linux-mm' in\n" + "the body to majordomo@kvack.org. For more info on Linux MM,\n" + "see: http://www.linux-mm.org/ .\n" + "Don't email: <a href=mailto:\"dont@kvack.org\"> email@kvack.org </a>" -4c412a7ead46d83610901eeff282057aafb982a8b79adc5f56a26bbdd7a62439 +a82f872ba86774eae94b19f7ab7d59bc0dec6ceceef3a79eb6cf9e400e5bfa3e
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.