* [PATCH] mm: memory hotplug: Check if pages are correctly reserved on a per-section basis @ 2011-10-10 7:11 Mel Gorman 2011-10-10 22:00 ` Andrew Morton 0 siblings, 1 reply; 13+ messages in thread From: Mel Gorman @ 2011-10-10 7:11 UTC (permalink / raw) To: Andrew Morton; +Cc: linux-mm, LKML, nfont, KAMEZAWA Hiroyuki It is expected that memory being brought online is PageReserved similar to what happens when the page allocator is being brought up. Memory is onlined in "memory blocks" which consist of one or more sections. Unfortunately, the code that verifies PageReserved is currently assuming that the memmap backing all these pages is virtually contiguous which is only the case when CONFIG_SPARSEMEM_VMEMMAP is set. As a result, memory hot-add is failing on !VMEMMAP configurations with the message; kernel: section number XXX page number 256 not reserved, was it already online? This patch updates the PageReserved check to lookup struct page once per section to guarantee the correct struct page is being checked. [original patch by: nfont@linux.vnet.ibm.com] Signed-off-by: Mel Gorman <mgorman@suse.de> --- drivers/base/memory.c | 58 +++++++++++++++++++++++++++++++++--------------- 1 files changed, 40 insertions(+), 18 deletions(-) diff --git a/drivers/base/memory.c b/drivers/base/memory.c index 2840ed4..ffb69cd 100644 --- a/drivers/base/memory.c +++ b/drivers/base/memory.c @@ -224,13 +224,48 @@ int memory_isolate_notify(unsigned long val, void *v) } /* + * The probe routines leave the pages reserved, just as the bootmem code does. + * Make sure they're still that way. + */ +static bool pages_correctly_reserved(unsigned long start_pfn, + unsigned long nr_pages) +{ + int i, j; + struct page *page; + unsigned long pfn = start_pfn; + + /* + * memmap between sections is not contiguous except with + * SPARSEMEM_VMEMMAP. We lookup the page once per section + * and assume memmap is contiguous within each section + */ + for (i = 0; i < sections_per_block; i++, pfn += PAGES_PER_SECTION) { + if (WARN_ON_ONCE(!pfn_valid(pfn))) + return false; + page = pfn_to_page(pfn); + + for (j = 0; j < PAGES_PER_SECTION; j++) { + if (PageReserved(page + i)) + continue; + + printk(KERN_WARNING "section number %ld page number %d " + "not reserved, was it already online?\n", + pfn_to_section_nr(pfn), j); + + return false; + } + } + + return true; +} + +/* * MEMORY_HOTPLUG depends on SPARSEMEM in mm/Kconfig, so it is * OK to have direct references to sparsemem variables in here. */ static int memory_block_action(unsigned long phys_index, unsigned long action) { - int i; unsigned long start_pfn, start_paddr; unsigned long nr_pages = PAGES_PER_SECTION * sections_per_block; struct page *first_page; @@ -238,26 +273,13 @@ memory_block_action(unsigned long phys_index, unsigned long action) first_page = pfn_to_page(phys_index << PFN_SECTION_SHIFT); - /* - * The probe routines leave the pages reserved, just - * as the bootmem code does. Make sure they're still - * that way. - */ - if (action == MEM_ONLINE) { - for (i = 0; i < nr_pages; i++) { - if (PageReserved(first_page+i)) - continue; - - printk(KERN_WARNING "section number %ld page number %d " - "not reserved, was it already online?\n", - phys_index, i); - return -EBUSY; - } - } - switch (action) { case MEM_ONLINE: start_pfn = page_to_pfn(first_page); + + if (!pages_correctly_reserved(start_pfn, nr_pages)) + return -EBUSY; + ret = online_pages(start_pfn, nr_pages); break; case MEM_OFFLINE: -- Mel Gorman SUSE Labs -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a> ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH] mm: memory hotplug: Check if pages are correctly reserved on a per-section basis 2011-10-10 7:11 [PATCH] mm: memory hotplug: Check if pages are correctly reserved on a per-section basis Mel Gorman @ 2011-10-10 22:00 ` Andrew Morton 2011-10-10 23:24 ` Greg KH 0 siblings, 1 reply; 13+ messages in thread From: Andrew Morton @ 2011-10-10 22:00 UTC (permalink / raw) To: Mel Gorman; +Cc: linux-mm, LKML, nfont, KAMEZAWA Hiroyuki, Greg KH On Mon, 10 Oct 2011 08:11:19 +0100 Mel Gorman <mgorman@suse.de> wrote: > It is expected that memory being brought online is PageReserved > similar to what happens when the page allocator is being brought up. > Memory is onlined in "memory blocks" which consist of one or more > sections. Unfortunately, the code that verifies PageReserved is > currently assuming that the memmap backing all these pages is virtually > contiguous which is only the case when CONFIG_SPARSEMEM_VMEMMAP is set. > As a result, memory hot-add is failing on !VMEMMAP configurations > with the message; > > kernel: section number XXX page number 256 not reserved, was it already online? > > This patch updates the PageReserved check to lookup struct page once > per section to guarantee the correct struct page is being checked. > Nathan's earlier version of this patch is already in linux-next, via Greg. We should drop the old version and get the new one merged instead. -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a> ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] mm: memory hotplug: Check if pages are correctly reserved on a per-section basis 2011-10-10 22:00 ` Andrew Morton @ 2011-10-10 23:24 ` Greg KH 2011-10-10 23:28 ` Andrew Morton 0 siblings, 1 reply; 13+ messages in thread From: Greg KH @ 2011-10-10 23:24 UTC (permalink / raw) To: Andrew Morton; +Cc: Mel Gorman, linux-mm, LKML, nfont, KAMEZAWA Hiroyuki On Mon, Oct 10, 2011 at 03:00:38PM -0700, Andrew Morton wrote: > On Mon, 10 Oct 2011 08:11:19 +0100 > Mel Gorman <mgorman@suse.de> wrote: > > > It is expected that memory being brought online is PageReserved > > similar to what happens when the page allocator is being brought up. > > Memory is onlined in "memory blocks" which consist of one or more > > sections. Unfortunately, the code that verifies PageReserved is > > currently assuming that the memmap backing all these pages is virtually > > contiguous which is only the case when CONFIG_SPARSEMEM_VMEMMAP is set. > > As a result, memory hot-add is failing on !VMEMMAP configurations > > with the message; > > > > kernel: section number XXX page number 256 not reserved, was it already online? > > > > This patch updates the PageReserved check to lookup struct page once > > per section to guarantee the correct struct page is being checked. > > > > Nathan's earlier version of this patch is already in linux-next, via > Greg. We should drop the old version and get the new one merged > instead. Ok, care to send me what exactly needs to be reverted and what needs to be added? thanks, greg k-h -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a> ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] mm: memory hotplug: Check if pages are correctly reserved on a per-section basis 2011-10-10 23:24 ` Greg KH @ 2011-10-10 23:28 ` Andrew Morton 2011-10-10 23:35 ` Greg KH 2011-10-11 7:27 ` Mel Gorman 0 siblings, 2 replies; 13+ messages in thread From: Andrew Morton @ 2011-10-10 23:28 UTC (permalink / raw) To: Greg KH; +Cc: Mel Gorman, linux-mm, LKML, nfont, KAMEZAWA Hiroyuki On Mon, 10 Oct 2011 16:24:03 -0700 Greg KH <greg@kroah.com> wrote: > On Mon, Oct 10, 2011 at 03:00:38PM -0700, Andrew Morton wrote: > > On Mon, 10 Oct 2011 08:11:19 +0100 > > Mel Gorman <mgorman@suse.de> wrote: > > > > > It is expected that memory being brought online is PageReserved > > > similar to what happens when the page allocator is being brought up. > > > Memory is onlined in "memory blocks" which consist of one or more > > > sections. Unfortunately, the code that verifies PageReserved is > > > currently assuming that the memmap backing all these pages is virtually > > > contiguous which is only the case when CONFIG_SPARSEMEM_VMEMMAP is set. > > > As a result, memory hot-add is failing on !VMEMMAP configurations > > > with the message; > > > > > > kernel: section number XXX page number 256 not reserved, was it already online? > > > > > > This patch updates the PageReserved check to lookup struct page once > > > per section to guarantee the correct struct page is being checked. > > > > > > > Nathan's earlier version of this patch is already in linux-next, via > > Greg. We should drop the old version and get the new one merged > > instead. > > Ok, care to send me what exactly needs to be reverted and what needs to > be added? Drop commit 54f23eb7ba7619de85d8edca6e5336bc33072dbd Author: Nathan Fontenot <nfont@austin.ibm.com> Date: Mon Sep 26 10:22:33 2011 -0500 memory hotplug: Correct page reservation checking and replace it with start-of-this-thread. That's assuming that Mel's update passes Nathan's review and testing :) -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a> ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] mm: memory hotplug: Check if pages are correctly reserved on a per-section basis 2011-10-10 23:28 ` Andrew Morton @ 2011-10-10 23:35 ` Greg KH 2011-10-10 23:41 ` Andrew Morton 2011-10-11 7:27 ` Mel Gorman 1 sibling, 1 reply; 13+ messages in thread From: Greg KH @ 2011-10-10 23:35 UTC (permalink / raw) To: Andrew Morton; +Cc: Mel Gorman, linux-mm, LKML, nfont, KAMEZAWA Hiroyuki On Mon, Oct 10, 2011 at 04:28:13PM -0700, Andrew Morton wrote: > On Mon, 10 Oct 2011 16:24:03 -0700 > Greg KH <greg@kroah.com> wrote: > > > On Mon, Oct 10, 2011 at 03:00:38PM -0700, Andrew Morton wrote: > > > On Mon, 10 Oct 2011 08:11:19 +0100 > > > Mel Gorman <mgorman@suse.de> wrote: > > > > > > > It is expected that memory being brought online is PageReserved > > > > similar to what happens when the page allocator is being brought up. > > > > Memory is onlined in "memory blocks" which consist of one or more > > > > sections. Unfortunately, the code that verifies PageReserved is > > > > currently assuming that the memmap backing all these pages is virtually > > > > contiguous which is only the case when CONFIG_SPARSEMEM_VMEMMAP is set. > > > > As a result, memory hot-add is failing on !VMEMMAP configurations > > > > with the message; > > > > > > > > kernel: section number XXX page number 256 not reserved, was it already online? > > > > > > > > This patch updates the PageReserved check to lookup struct page once > > > > per section to guarantee the correct struct page is being checked. > > > > > > > > > > Nathan's earlier version of this patch is already in linux-next, via > > > Greg. We should drop the old version and get the new one merged > > > instead. > > > > Ok, care to send me what exactly needs to be reverted and what needs to > > be added? > > Drop > > commit 54f23eb7ba7619de85d8edca6e5336bc33072dbd > Author: Nathan Fontenot <nfont@austin.ibm.com> > Date: Mon Sep 26 10:22:33 2011 -0500 > > memory hotplug: Correct page reservation checking > > and replace it with start-of-this-thread. > > That's assuming that Mel's update passes Nathan's review and testing :) Ok, I'll wait for that review and testing, and then can someone send me the patch at the start-of-this-thread as I no longer seem to be able to find it :( greg k-h -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a> ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] mm: memory hotplug: Check if pages are correctly reserved on a per-section basis 2011-10-10 23:35 ` Greg KH @ 2011-10-10 23:41 ` Andrew Morton 2011-10-11 4:26 ` David Rientjes 0 siblings, 1 reply; 13+ messages in thread From: Andrew Morton @ 2011-10-10 23:41 UTC (permalink / raw) To: Greg KH; +Cc: Mel Gorman, linux-mm, LKML, nfont, KAMEZAWA Hiroyuki On Mon, 10 Oct 2011 16:35:31 -0700 Greg KH <greg@kroah.com> wrote: > > > Ok, care to send me what exactly needs to be reverted and what needs to > > > be added? > > > > Drop > > > > commit 54f23eb7ba7619de85d8edca6e5336bc33072dbd > > Author: Nathan Fontenot <nfont@austin.ibm.com> > > Date: Mon Sep 26 10:22:33 2011 -0500 > > > > memory hotplug: Correct page reservation checking > > > > and replace it with start-of-this-thread. > > > > That's assuming that Mel's update passes Nathan's review and testing :) > > Ok, I'll wait for that review and testing, and then can someone send me > the patch at the start-of-this-thread as I no longer seem to be able to > find it :( It sounds like your new email setup is working as well as mine :( From: Mel Gorman <mgorman@suse.de> Subject: [PATCH] mm: memory hotplug: Check if pages are correctly reserved on It is expected that memory being brought online is PageReserved similar to what happens when the page allocator is being brought up. Memory is onlined in "memory blocks" which consist of one or more sections. Unfortunately, the code that verifies PageReserved is currently assuming that the memmap backing all these pages is virtually contiguous which is only the case when CONFIG_SPARSEMEM_VMEMMAP is set. As a result, memory hot-add is failing on !VMEMMAP configurations with the message; kernel: section number XXX page number 256 not reserved, was it already online? This patch updates the PageReserved check to lookup struct page once per section to guarantee the correct struct page is being checked. [original patch by: nfont@linux.vnet.ibm.com] Signed-off-by: Mel Gorman <mgorman@suse.de> --- drivers/base/memory.c | 58 +++++++++++++++++++++++++++++++++--------------- 1 files changed, 40 insertions(+), 18 deletions(-) diff --git a/drivers/base/memory.c b/drivers/base/memory.c index 2840ed4..ffb69cd 100644 --- a/drivers/base/memory.c +++ b/drivers/base/memory.c @@ -224,13 +224,48 @@ int memory_isolate_notify(unsigned long val, void *v) } /* + * The probe routines leave the pages reserved, just as the bootmem code does. + * Make sure they're still that way. + */ +static bool pages_correctly_reserved(unsigned long start_pfn, + unsigned long nr_pages) +{ + int i, j; + struct page *page; + unsigned long pfn = start_pfn; + + /* + * memmap between sections is not contiguous except with + * SPARSEMEM_VMEMMAP. We lookup the page once per section + * and assume memmap is contiguous within each section + */ + for (i = 0; i < sections_per_block; i++, pfn += PAGES_PER_SECTION) { + if (WARN_ON_ONCE(!pfn_valid(pfn))) + return false; + page = pfn_to_page(pfn); + + for (j = 0; j < PAGES_PER_SECTION; j++) { + if (PageReserved(page + i)) + continue; + + printk(KERN_WARNING "section number %ld page number %d " + "not reserved, was it already online?\n", + pfn_to_section_nr(pfn), j); + + return false; + } + } + + return true; +} + +/* * MEMORY_HOTPLUG depends on SPARSEMEM in mm/Kconfig, so it is * OK to have direct references to sparsemem variables in here. */ static int memory_block_action(unsigned long phys_index, unsigned long action) { - int i; unsigned long start_pfn, start_paddr; unsigned long nr_pages = PAGES_PER_SECTION * sections_per_block; struct page *first_page; @@ -238,26 +273,13 @@ memory_block_action(unsigned long phys_index, unsigned long action) first_page = pfn_to_page(phys_index << PFN_SECTION_SHIFT); - /* - * The probe routines leave the pages reserved, just - * as the bootmem code does. Make sure they're still - * that way. - */ - if (action == MEM_ONLINE) { - for (i = 0; i < nr_pages; i++) { - if (PageReserved(first_page+i)) - continue; - - printk(KERN_WARNING "section number %ld page number %d " - "not reserved, was it already online?\n", - phys_index, i); - return -EBUSY; - } - } - switch (action) { case MEM_ONLINE: start_pfn = page_to_pfn(first_page); + + if (!pages_correctly_reserved(start_pfn, nr_pages)) + return -EBUSY; + ret = online_pages(start_pfn, nr_pages); break; case MEM_OFFLINE: -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a> ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH] mm: memory hotplug: Check if pages are correctly reserved on a per-section basis 2011-10-10 23:41 ` Andrew Morton @ 2011-10-11 4:26 ` David Rientjes 0 siblings, 0 replies; 13+ messages in thread From: David Rientjes @ 2011-10-11 4:26 UTC (permalink / raw) To: Andrew Morton, Mel Gorman Cc: Greg KH, linux-mm, LKML, nfont, KAMEZAWA Hiroyuki On Mon, 10 Oct 2011, Andrew Morton wrote: > diff --git a/drivers/base/memory.c b/drivers/base/memory.c > index 2840ed4..ffb69cd 100644 > --- a/drivers/base/memory.c > +++ b/drivers/base/memory.c > @@ -224,13 +224,48 @@ int memory_isolate_notify(unsigned long val, void *v) > } > > /* > + * The probe routines leave the pages reserved, just as the bootmem code does. > + * Make sure they're still that way. > + */ > +static bool pages_correctly_reserved(unsigned long start_pfn, > + unsigned long nr_pages) > +{ > + int i, j; > + struct page *page; > + unsigned long pfn = start_pfn; > + > + /* > + * memmap between sections is not contiguous except with > + * SPARSEMEM_VMEMMAP. We lookup the page once per section > + * and assume memmap is contiguous within each section > + */ > + for (i = 0; i < sections_per_block; i++, pfn += PAGES_PER_SECTION) { > + if (WARN_ON_ONCE(!pfn_valid(pfn))) > + return false; > + page = pfn_to_page(pfn); > + > + for (j = 0; j < PAGES_PER_SECTION; j++) { > + if (PageReserved(page + i)) page + j > + continue; > + > + printk(KERN_WARNING "section number %ld page number %d " > + "not reserved, was it already online?\n", > + pfn_to_section_nr(pfn), j); > + > + return false; > + } > + } > + > + return true; > +} > + > +/* > * MEMORY_HOTPLUG depends on SPARSEMEM in mm/Kconfig, so it is > * OK to have direct references to sparsemem variables in here. > */ > static int > memory_block_action(unsigned long phys_index, unsigned long action) > { > - int i; > unsigned long start_pfn, start_paddr; > unsigned long nr_pages = PAGES_PER_SECTION * sections_per_block; > struct page *first_page; > @@ -238,26 +273,13 @@ memory_block_action(unsigned long phys_index, unsigned long action) > > first_page = pfn_to_page(phys_index << PFN_SECTION_SHIFT); > > - /* > - * The probe routines leave the pages reserved, just > - * as the bootmem code does. Make sure they're still > - * that way. > - */ > - if (action == MEM_ONLINE) { > - for (i = 0; i < nr_pages; i++) { > - if (PageReserved(first_page+i)) > - continue; > - > - printk(KERN_WARNING "section number %ld page number %d " > - "not reserved, was it already online?\n", > - phys_index, i); > - return -EBUSY; > - } > - } > - > switch (action) { > case MEM_ONLINE: > start_pfn = page_to_pfn(first_page); > + > + if (!pages_correctly_reserved(start_pfn, nr_pages)) > + return -EBUSY; > + > ret = online_pages(start_pfn, nr_pages); > break; > case MEM_OFFLINE: -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a> ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] mm: memory hotplug: Check if pages are correctly reserved on a per-section basis 2011-10-10 23:28 ` Andrew Morton 2011-10-10 23:35 ` Greg KH @ 2011-10-11 7:27 ` Mel Gorman 2011-10-11 14:44 ` Nathan Fontenot ` (2 more replies) 1 sibling, 3 replies; 13+ messages in thread From: Mel Gorman @ 2011-10-11 7:27 UTC (permalink / raw) To: Andrew Morton; +Cc: Greg KH, linux-mm, LKML, nfont, KAMEZAWA Hiroyuki, rientjes On Mon, Oct 10, 2011 at 04:28:13PM -0700, Andrew Morton wrote: > On Mon, 10 Oct 2011 16:24:03 -0700 > Greg KH <greg@kroah.com> wrote: > > > On Mon, Oct 10, 2011 at 03:00:38PM -0700, Andrew Morton wrote: > > > On Mon, 10 Oct 2011 08:11:19 +0100 > > > Mel Gorman <mgorman@suse.de> wrote: > > > > > > > It is expected that memory being brought online is PageReserved > > > > similar to what happens when the page allocator is being brought up. > > > > Memory is onlined in "memory blocks" which consist of one or more > > > > sections. Unfortunately, the code that verifies PageReserved is > > > > currently assuming that the memmap backing all these pages is virtually > > > > contiguous which is only the case when CONFIG_SPARSEMEM_VMEMMAP is set. > > > > As a result, memory hot-add is failing on !VMEMMAP configurations > > > > with the message; > > > > > > > > kernel: section number XXX page number 256 not reserved, was it already online? > > > > > > > > This patch updates the PageReserved check to lookup struct page once > > > > per section to guarantee the correct struct page is being checked. > > > > > > > > > > Nathan's earlier version of this patch is already in linux-next, via > > > Greg. We should drop the old version and get the new one merged > > > instead. > > > > Ok, care to send me what exactly needs to be reverted and what needs to > > be added? > > Drop > > commit 54f23eb7ba7619de85d8edca6e5336bc33072dbd > Author: Nathan Fontenot <nfont@austin.ibm.com> > Date: Mon Sep 26 10:22:33 2011 -0500 > > memory hotplug: Correct page reservation checking > > and replace it with start-of-this-thread. > > That's assuming that Mel's update passes Nathan's review and testing :) It passed review and testing with IBM based on a SUSE bug. I thought Nathan's patch had been lost as it was posted to linuxppc-dev instead of linux-mm. This rework was to improve the changelog and readability. David correctly pointed out a bug that passed testing because it was still checking one page per section. As long as that page was reserved, memory hot-add would go ahead. Here is a corrected version. Thanks ==== CUT HERE ==== mm: memory hotplug: Check if pages are correctly reserved on a per-section basis It is expected that memory being brought online is PageReserved similar to what happens when the page allocator is being brought up. Memory is onlined in "memory blocks" which consist of one or more sections. Unfortunately, the code that verifies PageReserved is currently assuming that the memmap backing all these pages is virtually contiguous which is only the case when CONFIG_SPARSEMEM_VMEMMAP is set. As a result, memory hot-add is failing on those configurations with the message; kernel: section number XXX page number 256 not reserved, was it already online? This patch updates the PageReserved check to lookup struct page once per section to guarantee the correct struct page is being checked. [Check pages within sections properly: rientjes@google.com] [original patch by: nfont@linux.vnet.ibm.com] Signed-off-by: Mel Gorman <mgorman@suse.de> --- drivers/base/memory.c | 58 +++++++++++++++++++++++++++++++++--------------- 1 files changed, 40 insertions(+), 18 deletions(-) diff --git a/drivers/base/memory.c b/drivers/base/memory.c index 2840ed4..ffb69cd 100644 --- a/drivers/base/memory.c +++ b/drivers/base/memory.c @@ -224,13 +224,48 @@ int memory_isolate_notify(unsigned long val, void *v) } /* + * The probe routines leave the pages reserved, just as the bootmem code does. + * Make sure they're still that way. + */ +static bool pages_correctly_reserved(unsigned long start_pfn, + unsigned long nr_pages) +{ + int i, j; + struct page *page; + unsigned long pfn = start_pfn; + + /* + * memmap between sections is not contiguous except with + * SPARSEMEM_VMEMMAP. We lookup the page once per section + * and assume memmap is contiguous within each section + */ + for (i = 0; i < sections_per_block; i++, pfn += PAGES_PER_SECTION) { + if (WARN_ON_ONCE(!pfn_valid(pfn))) + return false; + page = pfn_to_page(pfn); + + for (j = 0; j < PAGES_PER_SECTION; j++) { + if (PageReserved(page + j)) + continue; + + printk(KERN_WARNING "section number %ld page number %d " + "not reserved, was it already online?\n", + pfn_to_section_nr(pfn), j); + + return false; + } + } + + return true; +} + +/* * MEMORY_HOTPLUG depends on SPARSEMEM in mm/Kconfig, so it is * OK to have direct references to sparsemem variables in here. */ static int memory_block_action(unsigned long phys_index, unsigned long action) { - int i; unsigned long start_pfn, start_paddr; unsigned long nr_pages = PAGES_PER_SECTION * sections_per_block; struct page *first_page; @@ -238,26 +273,13 @@ memory_block_action(unsigned long phys_index, unsigned long action) first_page = pfn_to_page(phys_index << PFN_SECTION_SHIFT); - /* - * The probe routines leave the pages reserved, just - * as the bootmem code does. Make sure they're still - * that way. - */ - if (action == MEM_ONLINE) { - for (i = 0; i < nr_pages; i++) { - if (PageReserved(first_page+i)) - continue; - - printk(KERN_WARNING "section number %ld page number %d " - "not reserved, was it already online?\n", - phys_index, i); - return -EBUSY; - } - } - switch (action) { case MEM_ONLINE: start_pfn = page_to_pfn(first_page); + + if (!pages_correctly_reserved(start_pfn, nr_pages)) + return -EBUSY; + ret = online_pages(start_pfn, nr_pages); break; case MEM_OFFLINE: -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a> ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH] mm: memory hotplug: Check if pages are correctly reserved on a per-section basis 2011-10-11 7:27 ` Mel Gorman @ 2011-10-11 14:44 ` Nathan Fontenot 2011-10-12 1:16 ` IBM 2011-10-17 14:38 ` Mel Gorman 2 siblings, 0 replies; 13+ messages in thread From: Nathan Fontenot @ 2011-10-11 14:44 UTC (permalink / raw) To: Mel Gorman Cc: Andrew Morton, Greg KH, linux-mm, LKML, KAMEZAWA Hiroyuki, rientjes On 10/11/2011 02:27 AM, Mel Gorman wrote: > On Mon, Oct 10, 2011 at 04:28:13PM -0700, Andrew Morton wrote: >> On Mon, 10 Oct 2011 16:24:03 -0700 >> Greg KH <greg@kroah.com> wrote: >> >>> On Mon, Oct 10, 2011 at 03:00:38PM -0700, Andrew Morton wrote: >>>> On Mon, 10 Oct 2011 08:11:19 +0100 >>>> Mel Gorman <mgorman@suse.de> wrote: >>>> >>>>> It is expected that memory being brought online is PageReserved >>>>> similar to what happens when the page allocator is being brought up. >>>>> Memory is onlined in "memory blocks" which consist of one or more >>>>> sections. Unfortunately, the code that verifies PageReserved is >>>>> currently assuming that the memmap backing all these pages is virtually >>>>> contiguous which is only the case when CONFIG_SPARSEMEM_VMEMMAP is set. >>>>> As a result, memory hot-add is failing on !VMEMMAP configurations >>>>> with the message; >>>>> >>>>> kernel: section number XXX page number 256 not reserved, was it already online? >>>>> >>>>> This patch updates the PageReserved check to lookup struct page once >>>>> per section to guarantee the correct struct page is being checked. >>>>> >>>> >>>> Nathan's earlier version of this patch is already in linux-next, via >>>> Greg. We should drop the old version and get the new one merged >>>> instead. >>> >>> Ok, care to send me what exactly needs to be reverted and what needs to >>> be added? >> >> Drop >> >> commit 54f23eb7ba7619de85d8edca6e5336bc33072dbd >> Author: Nathan Fontenot <nfont@austin.ibm.com> >> Date: Mon Sep 26 10:22:33 2011 -0500 >> >> memory hotplug: Correct page reservation checking >> >> and replace it with start-of-this-thread. >> >> That's assuming that Mel's update passes Nathan's review and testing :) > > It passed review and testing with IBM based on a SUSE bug. I thought > Nathan's patch had been lost as it was posted to linuxppc-dev instead > of linux-mm. This rework was to improve the changelog and readability. > > David correctly pointed out a bug that passed testing because it was > still checking one page per section. As long as that page was reserved, > memory hot-add would go ahead. Here is a corrected version. > Previous patch passed testing, working on testing Mel's updated patch... -Nathan > Thanks > > ==== CUT HERE ==== > mm: memory hotplug: Check if pages are correctly reserved on a per-section basis > > It is expected that memory being brought online is PageReserved > similar to what happens when the page allocator is being brought up. > Memory is onlined in "memory blocks" which consist of one or more > sections. Unfortunately, the code that verifies PageReserved is > currently assuming that the memmap backing all these pages is virtually > contiguous which is only the case when CONFIG_SPARSEMEM_VMEMMAP is set. > As a result, memory hot-add is failing on those configurations with > the message; > > kernel: section number XXX page number 256 not reserved, was it already online? > > This patch updates the PageReserved check to lookup struct page once > per section to guarantee the correct struct page is being checked. > > [Check pages within sections properly: rientjes@google.com] > [original patch by: nfont@linux.vnet.ibm.com] > Signed-off-by: Mel Gorman <mgorman@suse.de> > --- > drivers/base/memory.c | 58 +++++++++++++++++++++++++++++++++--------------- > 1 files changed, 40 insertions(+), 18 deletions(-) > > diff --git a/drivers/base/memory.c b/drivers/base/memory.c > index 2840ed4..ffb69cd 100644 > --- a/drivers/base/memory.c > +++ b/drivers/base/memory.c > @@ -224,13 +224,48 @@ int memory_isolate_notify(unsigned long val, void *v) > } > > /* > + * The probe routines leave the pages reserved, just as the bootmem code does. > + * Make sure they're still that way. > + */ > +static bool pages_correctly_reserved(unsigned long start_pfn, > + unsigned long nr_pages) > +{ > + int i, j; > + struct page *page; > + unsigned long pfn = start_pfn; > + > + /* > + * memmap between sections is not contiguous except with > + * SPARSEMEM_VMEMMAP. We lookup the page once per section > + * and assume memmap is contiguous within each section > + */ > + for (i = 0; i < sections_per_block; i++, pfn += PAGES_PER_SECTION) { > + if (WARN_ON_ONCE(!pfn_valid(pfn))) > + return false; > + page = pfn_to_page(pfn); > + > + for (j = 0; j < PAGES_PER_SECTION; j++) { > + if (PageReserved(page + j)) > + continue; > + > + printk(KERN_WARNING "section number %ld page number %d " > + "not reserved, was it already online?\n", > + pfn_to_section_nr(pfn), j); > + > + return false; > + } > + } > + > + return true; > +} > + > +/* > * MEMORY_HOTPLUG depends on SPARSEMEM in mm/Kconfig, so it is > * OK to have direct references to sparsemem variables in here. > */ > static int > memory_block_action(unsigned long phys_index, unsigned long action) > { > - int i; > unsigned long start_pfn, start_paddr; > unsigned long nr_pages = PAGES_PER_SECTION * sections_per_block; > struct page *first_page; > @@ -238,26 +273,13 @@ memory_block_action(unsigned long phys_index, unsigned long action) > > first_page = pfn_to_page(phys_index << PFN_SECTION_SHIFT); > > - /* > - * The probe routines leave the pages reserved, just > - * as the bootmem code does. Make sure they're still > - * that way. > - */ > - if (action == MEM_ONLINE) { > - for (i = 0; i < nr_pages; i++) { > - if (PageReserved(first_page+i)) > - continue; > - > - printk(KERN_WARNING "section number %ld page number %d " > - "not reserved, was it already online?\n", > - phys_index, i); > - return -EBUSY; > - } > - } > - > switch (action) { > case MEM_ONLINE: > start_pfn = page_to_pfn(first_page); > + > + if (!pages_correctly_reserved(start_pfn, nr_pages)) > + return -EBUSY; > + > ret = online_pages(start_pfn, nr_pages); > break; > case MEM_OFFLINE: > -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a> ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] mm: memory hotplug: Check if pages are correctly reserved on a per-section basis 2011-10-11 7:27 ` Mel Gorman 2011-10-11 14:44 ` Nathan Fontenot @ 2011-10-12 1:16 ` IBM 2011-10-17 14:38 ` Mel Gorman 2 siblings, 0 replies; 13+ messages in thread From: IBM @ 2011-10-12 1:16 UTC (permalink / raw) To: Mel Gorman Cc: Andrew Morton, Greg KH, linux-mm, LKML, nfont@linux.vnet.ibm.com, KAMEZAWA Hiroyuki, rientjes@google.com On Oct 11, 2011, at 2:27 AM, Mel Gorman <mgorman@suse.de> wrote: > On Mon, Oct 10, 2011 at 04:28:13PM -0700, Andrew Morton wrote: >> On Mon, 10 Oct 2011 16:24:03 -0700 >> Greg KH <greg@kroah.com> wrote: >> >>> On Mon, Oct 10, 2011 at 03:00:38PM -0700, Andrew Morton wrote: >>>> On Mon, 10 Oct 2011 08:11:19 +0100 >>>> Mel Gorman <mgorman@suse.de> wrote: >>>> >>>>> It is expected that memory being brought online is PageReserved >>>>> similar to what happens when the page allocator is being brought up. >>>>> Memory is onlined in "memory blocks" which consist of one or more >>>>> sections. Unfortunately, the code that verifies PageReserved is >>>>> currently assuming that the memmap backing all these pages is virtually >>>>> contiguous which is only the case when CONFIG_SPARSEMEM_VMEMMAP is set. >>>>> As a result, memory hot-add is failing on !VMEMMAP configurations >>>>> with the message; >>>>> >>>>> kernel: section number XXX page number 256 not reserved, was it already online? >>>>> >>>>> This patch updates the PageReserved check to lookup struct page once >>>>> per section to guarantee the correct struct page is being checked. >>>>> >>>> >>>> Nathan's earlier version of this patch is already in linux-next, via >>>> Greg. We should drop the old version and get the new one merged >>>> instead. >>> >>> Ok, care to send me what exactly needs to be reverted and what needs to >>> be added? >> >> Drop >> >> commit 54f23eb7ba7619de85d8edca6e5336bc33072dbd >> Author: Nathan Fontenot <nfont@austin.ibm.com> >> Date: Mon Sep 26 10:22:33 2011 -0500 >> >> memory hotplug: Correct page reservation checking >> >> and replace it with start-of-this-thread. >> >> That's assuming that Mel's update passes Nathan's review and testing :) > > It passed review and testing with IBM based on a SUSE bug. I thought > Nathan's patch had been lost as it was posted to linuxppc-dev instead > of linux-mm. This rework was to improve the changelog and readability. > > David correctly pointed out a bug that passed testing because it was > still checking one page per section. As long as that page was reserved, > memory hot-add would go ahead. Here is a corrected version. > Updated patch has been tested. -Nathan > Thanks > > ==== CUT HERE ==== > mm: memory hotplug: Check if pages are correctly reserved on a per-section basis > > It is expected that memory being brought online is PageReserved > similar to what happens when the page allocator is being brought up. > Memory is onlined in "memory blocks" which consist of one or more > sections. Unfortunately, the code that verifies PageReserved is > currently assuming that the memmap backing all these pages is virtually > contiguous which is only the case when CONFIG_SPARSEMEM_VMEMMAP is set. > As a result, memory hot-add is failing on those configurations with > the message; > > kernel: section number XXX page number 256 not reserved, was it already online? > > This patch updates the PageReserved check to lookup struct page once > per section to guarantee the correct struct page is being checked. > > [Check pages within sections properly: rientjes@google.com] > [original patch by: nfont@linux.vnet.ibm.com] > Signed-off-by: Mel Gorman <mgorman@suse.de> > --- > drivers/base/memory.c | 58 +++++++++++++++++++++++++++++++++--------------- > 1 files changed, 40 insertions(+), 18 deletions(-) > > diff --git a/drivers/base/memory.c b/drivers/base/memory.c > index 2840ed4..ffb69cd 100644 > --- a/drivers/base/memory.c > +++ b/drivers/base/memory.c > @@ -224,13 +224,48 @@ int memory_isolate_notify(unsigned long val, void *v) > } > > /* > + * The probe routines leave the pages reserved, just as the bootmem code does. > + * Make sure they're still that way. > + */ > +static bool pages_correctly_reserved(unsigned long start_pfn, > + unsigned long nr_pages) > +{ > + int i, j; > + struct page *page; > + unsigned long pfn = start_pfn; > + > + /* > + * memmap between sections is not contiguous except with > + * SPARSEMEM_VMEMMAP. We lookup the page once per section > + * and assume memmap is contiguous within each section > + */ > + for (i = 0; i < sections_per_block; i++, pfn += PAGES_PER_SECTION) { > + if (WARN_ON_ONCE(!pfn_valid(pfn))) > + return false; > + page = pfn_to_page(pfn); > + > + for (j = 0; j < PAGES_PER_SECTION; j++) { > + if (PageReserved(page + j)) > + continue; > + > + printk(KERN_WARNING "section number %ld page number %d " > + "not reserved, was it already online?\n", > + pfn_to_section_nr(pfn), j); > + > + return false; > + } > + } > + > + return true; > +} > + > +/* > * MEMORY_HOTPLUG depends on SPARSEMEM in mm/Kconfig, so it is > * OK to have direct references to sparsemem variables in here. > */ > static int > memory_block_action(unsigned long phys_index, unsigned long action) > { > - int i; > unsigned long start_pfn, start_paddr; > unsigned long nr_pages = PAGES_PER_SECTION * sections_per_block; > struct page *first_page; > @@ -238,26 +273,13 @@ memory_block_action(unsigned long phys_index, unsigned long action) > > first_page = pfn_to_page(phys_index << PFN_SECTION_SHIFT); > > - /* > - * The probe routines leave the pages reserved, just > - * as the bootmem code does. Make sure they're still > - * that way. > - */ > - if (action == MEM_ONLINE) { > - for (i = 0; i < nr_pages; i++) { > - if (PageReserved(first_page+i)) > - continue; > - > - printk(KERN_WARNING "section number %ld page number %d " > - "not reserved, was it already online?\n", > - phys_index, i); > - return -EBUSY; > - } > - } > - > switch (action) { > case MEM_ONLINE: > start_pfn = page_to_pfn(first_page); > + > + if (!pages_correctly_reserved(start_pfn, nr_pages)) > + return -EBUSY; > + > ret = online_pages(start_pfn, nr_pages); > break; > case MEM_OFFLINE: > -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a> ^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH] mm: memory hotplug: Check if pages are correctly reserved on a per-section basis 2011-10-11 7:27 ` Mel Gorman 2011-10-11 14:44 ` Nathan Fontenot 2011-10-12 1:16 ` IBM @ 2011-10-17 14:38 ` Mel Gorman 2011-10-18 0:45 ` KAMEZAWA Hiroyuki 2011-10-20 5:19 ` David Rientjes 2 siblings, 2 replies; 13+ messages in thread From: Mel Gorman @ 2011-10-17 14:38 UTC (permalink / raw) To: Greg KH, Andrew Morton; +Cc: linux-mm, LKML, nfont, KAMEZAWA Hiroyuki, rientjes (Resending as I am not seeing it in -next so maybe it got lost) mm: memory hotplug: Check if pages are correctly reserved on a per-section basis It is expected that memory being brought online is PageReserved similar to what happens when the page allocator is being brought up. Memory is onlined in "memory blocks" which consist of one or more sections. Unfortunately, the code that verifies PageReserved is currently assuming that the memmap backing all these pages is virtually contiguous which is only the case when CONFIG_SPARSEMEM_VMEMMAP is set. As a result, memory hot-add is failing on those configurations with the message; kernel: section number XXX page number 256 not reserved, was it already online? This patch updates the PageReserved check to lookup struct page once per section to guarantee the correct struct page is being checked. [Check pages within sections properly: rientjes@google.com] [original patch by: nfont@linux.vnet.ibm.com] Signed-off-by: Mel Gorman <mgorman@suse.de> --- drivers/base/memory.c | 58 +++++++++++++++++++++++++++++++++--------------- 1 files changed, 40 insertions(+), 18 deletions(-) diff --git a/drivers/base/memory.c b/drivers/base/memory.c index 2840ed4..ffb69cd 100644 --- a/drivers/base/memory.c +++ b/drivers/base/memory.c @@ -224,13 +224,48 @@ int memory_isolate_notify(unsigned long val, void *v) } /* + * The probe routines leave the pages reserved, just as the bootmem code does. + * Make sure they're still that way. + */ +static bool pages_correctly_reserved(unsigned long start_pfn, + unsigned long nr_pages) +{ + int i, j; + struct page *page; + unsigned long pfn = start_pfn; + + /* + * memmap between sections is not contiguous except with + * SPARSEMEM_VMEMMAP. We lookup the page once per section + * and assume memmap is contiguous within each section + */ + for (i = 0; i < sections_per_block; i++, pfn += PAGES_PER_SECTION) { + if (WARN_ON_ONCE(!pfn_valid(pfn))) + return false; + page = pfn_to_page(pfn); + + for (j = 0; j < PAGES_PER_SECTION; j++) { + if (PageReserved(page + j)) + continue; + + printk(KERN_WARNING "section number %ld page number %d " + "not reserved, was it already online?\n", + pfn_to_section_nr(pfn), j); + + return false; + } + } + + return true; +} + +/* * MEMORY_HOTPLUG depends on SPARSEMEM in mm/Kconfig, so it is * OK to have direct references to sparsemem variables in here. */ static int memory_block_action(unsigned long phys_index, unsigned long action) { - int i; unsigned long start_pfn, start_paddr; unsigned long nr_pages = PAGES_PER_SECTION * sections_per_block; struct page *first_page; @@ -238,26 +273,13 @@ memory_block_action(unsigned long phys_index, unsigned long action) first_page = pfn_to_page(phys_index << PFN_SECTION_SHIFT); - /* - * The probe routines leave the pages reserved, just - * as the bootmem code does. Make sure they're still - * that way. - */ - if (action == MEM_ONLINE) { - for (i = 0; i < nr_pages; i++) { - if (PageReserved(first_page+i)) - continue; - - printk(KERN_WARNING "section number %ld page number %d " - "not reserved, was it already online?\n", - phys_index, i); - return -EBUSY; - } - } - switch (action) { case MEM_ONLINE: start_pfn = page_to_pfn(first_page); + + if (!pages_correctly_reserved(start_pfn, nr_pages)) + return -EBUSY; + ret = online_pages(start_pfn, nr_pages); break; case MEM_OFFLINE: -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a> ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH] mm: memory hotplug: Check if pages are correctly reserved on a per-section basis 2011-10-17 14:38 ` Mel Gorman @ 2011-10-18 0:45 ` KAMEZAWA Hiroyuki 2011-10-20 5:19 ` David Rientjes 1 sibling, 0 replies; 13+ messages in thread From: KAMEZAWA Hiroyuki @ 2011-10-18 0:45 UTC (permalink / raw) To: Mel Gorman; +Cc: Greg KH, Andrew Morton, linux-mm, LKML, nfont, rientjes On Mon, 17 Oct 2011 16:38:20 +0200 Mel Gorman <mgorman@suse.de> wrote: > (Resending as I am not seeing it in -next so maybe it got lost) > > mm: memory hotplug: Check if pages are correctly reserved on a per-section basis > > It is expected that memory being brought online is PageReserved > similar to what happens when the page allocator is being brought up. > Memory is onlined in "memory blocks" which consist of one or more > sections. Unfortunately, the code that verifies PageReserved is > currently assuming that the memmap backing all these pages is virtually > contiguous which is only the case when CONFIG_SPARSEMEM_VMEMMAP is set. > As a result, memory hot-add is failing on those configurations with > the message; > > kernel: section number XXX page number 256 not reserved, was it already online? > > This patch updates the PageReserved check to lookup struct page once > per section to guarantee the correct struct page is being checked. > > [Check pages within sections properly: rientjes@google.com] > [original patch by: nfont@linux.vnet.ibm.com] > Signed-off-by: Mel Gorman <mgorman@suse.de> Thank you. Acked-by: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com> -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a> ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] mm: memory hotplug: Check if pages are correctly reserved on a per-section basis 2011-10-17 14:38 ` Mel Gorman 2011-10-18 0:45 ` KAMEZAWA Hiroyuki @ 2011-10-20 5:19 ` David Rientjes 1 sibling, 0 replies; 13+ messages in thread From: David Rientjes @ 2011-10-20 5:19 UTC (permalink / raw) To: Mel Gorman Cc: Greg KH, Andrew Morton, linux-mm, LKML, nfont, KAMEZAWA Hiroyuki On Mon, 17 Oct 2011, Mel Gorman wrote: > (Resending as I am not seeing it in -next so maybe it got lost) > > mm: memory hotplug: Check if pages are correctly reserved on a per-section basis > > It is expected that memory being brought online is PageReserved > similar to what happens when the page allocator is being brought up. > Memory is onlined in "memory blocks" which consist of one or more > sections. Unfortunately, the code that verifies PageReserved is > currently assuming that the memmap backing all these pages is virtually > contiguous which is only the case when CONFIG_SPARSEMEM_VMEMMAP is set. > As a result, memory hot-add is failing on those configurations with > the message; > > kernel: section number XXX page number 256 not reserved, was it already online? > > This patch updates the PageReserved check to lookup struct page once > per section to guarantee the correct struct page is being checked. > > [Check pages within sections properly: rientjes@google.com] > [original patch by: nfont@linux.vnet.ibm.com] > Signed-off-by: Mel Gorman <mgorman@suse.de> Acked-by: David Rientjes <rientjes@google.com> -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a> ^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2011-10-20 5:20 UTC | newest] Thread overview: 13+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2011-10-10 7:11 [PATCH] mm: memory hotplug: Check if pages are correctly reserved on a per-section basis Mel Gorman 2011-10-10 22:00 ` Andrew Morton 2011-10-10 23:24 ` Greg KH 2011-10-10 23:28 ` Andrew Morton 2011-10-10 23:35 ` Greg KH 2011-10-10 23:41 ` Andrew Morton 2011-10-11 4:26 ` David Rientjes 2011-10-11 7:27 ` Mel Gorman 2011-10-11 14:44 ` Nathan Fontenot 2011-10-12 1:16 ` IBM 2011-10-17 14:38 ` Mel Gorman 2011-10-18 0:45 ` KAMEZAWA Hiroyuki 2011-10-20 5:19 ` David Rientjes
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).