Re: [PATCH 1/5] generic __remove_pages() support

2008-03-27 Thread Yasunori Goto
Ok. Thanks. Acked-by: Yasunori Goto <[EMAIL PROTECTED]> > Generic helper function to remove section mappings and sysfs entries > for the section of the memory we are removing. offline_pages() correctly > adjusted zone and marked the pages reserved. > > TODO: Yasunori Goto is working on patche

[PATCH 1/5] generic __remove_pages() support

2008-03-27 Thread Badari Pulavarty
Generic helper function to remove section mappings and sysfs entries for the section of the memory we are removing. offline_pages() correctly adjusted zone and marked the pages reserved. TODO: Yasunori Goto is working on patches to freeup allocations from bootmem. Signed-off-by: Badari Pulavart

Re: [PATCH 1/5] generic __remove_pages() support

2008-03-21 Thread Badari Pulavarty
On Sat, 2008-03-22 at 00:25 +0900, Yasunori Goto wrote: > Badari-san. > > > Index: linux-2.6.25-rc2/mm/memory_hotplug.c > > === > > --- linux-2.6.25-rc2.orig/mm/memory_hotplug.c 2008-02-27 > > 12:58:17.0 -0800 > > +++ l

Re: [PATCH 1/5] generic __remove_pages() support

2008-03-21 Thread Yasunori Goto
Badari-san. > Index: linux-2.6.25-rc2/mm/memory_hotplug.c > === > --- linux-2.6.25-rc2.orig/mm/memory_hotplug.c 2008-02-27 12:58:17.0 > -0800 > +++ linux-2.6.25-rc2/mm/memory_hotplug.c 2008-02-27 16:06:50.0 > -

Re: [PATCH 1/5] generic __remove_pages() support

2008-03-07 Thread Badari Pulavarty
Yasunori Goto wrote: > Hi Badari-san. > > >> On Thu, 2008-03-06 at 12:54 -0800, Dave Hansen wrote: >> >>> On Thu, 2008-03-06 at 10:55 -0800, Badari Pulavarty wrote: >>> + if (memmap) + __kfree_section_memmap(memmap, PAGES_PER_SECTION); >>

Re: [PATCH 1/5] generic __remove_pages() support

2008-03-07 Thread Badari Pulavarty
Yasunori Goto wrote: > Hi Badari-san. > > >> On Thu, 2008-03-06 at 12:54 -0800, Dave Hansen wrote: >> >>> On Thu, 2008-03-06 at 10:55 -0800, Badari Pulavarty wrote: >>> + if (memmap) + __kfree_section_memmap(memmap, PAGES_PER_SECTION); >>

Re: [PATCH 1/5] generic __remove_pages() support

2008-03-06 Thread Yasunori Goto
Hi Badari-san. > On Thu, 2008-03-06 at 12:54 -0800, Dave Hansen wrote: > > On Thu, 2008-03-06 at 10:55 -0800, Badari Pulavarty wrote: > > > + if (memmap) > > > + __kfree_section_memmap(memmap, PAGES_PER_SECTION); > > > + return; > > > + } > >

Re: [PATCH 1/5] generic __remove_pages() support

2008-03-06 Thread Badari Pulavarty
Here is the latest version, addressing Randy and Dave's comments. Thanks, Badari Generic helper function to remove section mappings and sysfs entries for the section of the memory we are removing. offline_pages() correctly adjusted zone and marked the pages reserved. Issue: If mem_map, usemap

Re: [PATCH 1/5] generic __remove_pages() support

2008-03-06 Thread Badari Pulavarty
On Thu, 2008-03-06 at 12:54 -0800, Dave Hansen wrote: > On Thu, 2008-03-06 at 10:55 -0800, Badari Pulavarty wrote: > > + if (memmap) > > + __kfree_section_memmap(memmap, PAGES_PER_SECTION); > > + return; > > + } > > + > > + /* > > +

Re: [PATCH 1/5] generic __remove_pages() support

2008-03-06 Thread Dave Hansen
On Thu, 2008-03-06 at 10:55 -0800, Badari Pulavarty wrote: > + if (memmap) > + __kfree_section_memmap(memmap, PAGES_PER_SECTION); > + return; > + } > + > + /* > +* Allocations came from bootmem - how do I free up ? > +*/

Re: [PATCH 1/5] generic __remove_pages() support

2008-03-06 Thread Randy Dunlap
On Thu, 06 Mar 2008 10:55:34 -0800 Badari Pulavarty wrote: > Generic helper function to remove section mappings and sysfs entries > for the section of the memory we are removing. offline_pages() correctly > adjusted zone and marked the pages reserved. Such generic (exported, non-static) interfa

Re: [PATCH 1/5] generic __remove_pages() support

2008-03-06 Thread Badari Pulavarty
On Thu, 2008-03-06 at 11:08 -0800, Randy Dunlap wrote: > On Thu, 06 Mar 2008 10:55:34 -0800 Badari Pulavarty wrote: > > > Generic helper function to remove section mappings and sysfs entries > > for the section of the memory we are removing. offline_pages() correctly > > adjusted zone and marked

[PATCH 1/5] generic __remove_pages() support

2008-03-06 Thread Badari Pulavarty
Generic helper function to remove section mappings and sysfs entries for the section of the memory we are removing. offline_pages() correctly adjusted zone and marked the pages reserved. Issue: If mem_map, usemap allocation could come from different places - kmalloc, vmalloc, alloc_pages or boot