diff for duplicates of <20090819121017.GA8226@localhost> diff --git a/a/1.txt b/N1/1.txt index 8005d69..fa742d3 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -12,7 +12,7 @@ On Wed, Aug 19, 2009 at 08:05:19PM +0800, KOSAKI Motohiro wrote: > >> > >> Currently, mlock() systemcall doesn't gurantee to mark the page PG_Mlocked > > -> > mark PG_mlocked +> > A A A A A A A A A A A A A A A A A A A A A A A A A A mark PG_mlocked > > > >> because some race prevent page grabbing. > >> In that case, instead vmscan move the page to unevictable lru. @@ -24,7 +24,7 @@ On Wed, Aug 19, 2009 at 08:05:19PM +0800, KOSAKI Motohiro wrote: > >> > >> Plus, vmscan should mark PG_Mlocked when cull mlocked page. > > -> > PG_mlocked +> > A A A A A A A A A A A A A PG_mlocked > > > >> Otherwise vm stastics show strange number. > >> @@ -38,17 +38,17 @@ On Wed, Aug 19, 2009 at 08:05:19PM +0800, KOSAKI Motohiro wrote: > > >> Index: b/mm/rmap.c > >> =================================================================== -> >> --- a/mm/rmap.c 2009-08-18 19:48:14.000000000 +0900 -> >> +++ b/mm/rmap.c 2009-08-18 23:47:34.000000000 +0900 +> >> --- a/mm/rmap.c A A A 2009-08-18 19:48:14.000000000 +0900 +> >> +++ b/mm/rmap.c A A A 2009-08-18 23:47:34.000000000 +0900 > >> @@ -362,7 +362,9 @@ static int page_referenced_one(struct pa -> >> * unevictable list. -> >> */ -> >> if (vma->vm_flags & VM_LOCKED) { -> >> - *mapcount = 1; /* break early from loop */ -> >> + *mapcount = 1; /* break early from loop */ -> >> + *vm_flags |= VM_LOCKED; /* for prevent to move active list */ +> >> A A A A * unevictable list. +> >> A A A A */ +> >> A A A if (vma->vm_flags & VM_LOCKED) { +> >> - A A A A A A *mapcount = 1; A /* break early from loop */ +> >> + A A A A A A *mapcount = 1; A A A A A /* break early from loop */ +> >> + A A A A A A *vm_flags |= VM_LOCKED; /* for prevent to move active list */ > > -> >> + try_set_page_mlocked(vma, page); +> >> + A A A A A A try_set_page_mlocked(vma, page); > > > > That call is not absolutely necessary? > @@ -60,3 +60,9 @@ in this patch. Thanks, Fengguang + +-- +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 caef8cf..7a07680 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -37,7 +37,7 @@ "> >>\n" "> >> Currently, mlock() systemcall doesn't gurantee to mark the page PG_Mlocked\n" "> >\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\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240mark PG_mlocked\n" + "> > A A A A A A A A A A A A A A A A A A A A A A A A A A mark PG_mlocked\n" "> >\n" "> >> because some race prevent page grabbing.\n" "> >> In that case, instead vmscan move the page to unevictable lru.\n" @@ -49,7 +49,7 @@ "> >>\n" "> >> Plus, vmscan should mark PG_Mlocked when cull mlocked page.\n" "> >\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 PG_mlocked\n" + "> > A A A A A A A A A A A A A PG_mlocked\n" "> >\n" "> >> Otherwise vm stastics show strange number.\n" "> >>\n" @@ -63,17 +63,17 @@ "> \n" "> >> Index: b/mm/rmap.c\n" "> >> ===================================================================\n" - "> >> --- a/mm/rmap.c \302\240 \302\240 \302\240 2009-08-18 19:48:14.000000000 +0900\n" - "> >> +++ b/mm/rmap.c \302\240 \302\240 \302\240 2009-08-18 23:47:34.000000000 +0900\n" + "> >> --- a/mm/rmap.c A A A 2009-08-18 19:48:14.000000000 +0900\n" + "> >> +++ b/mm/rmap.c A A A 2009-08-18 23:47:34.000000000 +0900\n" "> >> @@ -362,7 +362,9 @@ static int page_referenced_one(struct pa\n" - "> >> \302\240 \302\240 \302\240 \302\240* unevictable list.\n" - "> >> \302\240 \302\240 \302\240 \302\240*/\n" - "> >> \302\240 \302\240 \302\240 if (vma->vm_flags & VM_LOCKED) {\n" - "> >> - \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 *mapcount = 1; \302\240/* break early from loop */\n" - "> >> + \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 *mapcount = 1; \302\240 \302\240 \302\240 \302\240 \302\240/* break early from loop */\n" - "> >> + \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 *vm_flags |= VM_LOCKED; /* for prevent to move active list */\n" + "> >> A A A A * unevictable list.\n" + "> >> A A A A */\n" + "> >> A A A if (vma->vm_flags & VM_LOCKED) {\n" + "> >> - A A A A A A *mapcount = 1; A /* break early from loop */\n" + "> >> + A A A A A A *mapcount = 1; A A A A A /* break early from loop */\n" + "> >> + A A A A A A *vm_flags |= VM_LOCKED; /* for prevent to move active list */\n" "> >\n" - "> >> + \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 try_set_page_mlocked(vma, page);\n" + "> >> + A A A A A A try_set_page_mlocked(vma, page);\n" "> >\n" "> > That call is not absolutely necessary?\n" "> \n" @@ -84,6 +84,12 @@ "in this patch.\n" "\n" "Thanks,\n" - Fengguang + "Fengguang\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>" -1272fb63e95e552784d44beac5dc8ffa3271ed317556c288fe8b3eae7c807575 +cf614c50139fa4efe2e12c4de1f00ba67bb99d0694f31722b97a9f18687c3594
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.