提交 51674644 编写于 作者: A Andrew Morton 提交者: Linus Torvalds

revert "memory hotplug: allocate usemap on the section with pgdat"

This:

commit 86f6dae1
Author: Yasunori Goto <y-goto@jp.fujitsu.com>
Date:   Mon Apr 28 02:13:33 2008 -0700

    memory hotplug: allocate usemap on the section with pgdat

    Usemaps are allocated on the section which has pgdat by this.

    Because usemap size is very small, many other sections usemaps are allocated
    on only one page.  If a section has usemap, it can't be removed until removing
    other sections.  This dependency is not desirable for memory removing.

    Pgdat has similar feature.  When a section has pgdat area, it must be the last
    section for removing on the node.  So, if section A has pgdat and section B
    has usemap for section A, Both sections can't be removed due to dependency
    each other.

    To solve this issue, this patch collects usemap on same section with pgdat.
    If other sections doesn't have any dependency, this section will be able to be
    removed finally.
Signed-off-by: NYasunori Goto <y-goto@jp.fujitsu.com>
    Cc: Badari Pulavarty <pbadari@us.ibm.com>
    Cc: Yinghai Lu <yhlu.kernel@gmail.com>
    Cc: Yasunori Goto <y-goto@jp.fujitsu.com>
Signed-off-by: NAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: NLinus Torvalds <torvalds@linux-foundation.org>

broke davem's sparc64 bootup.  Revert it while we work out what went wrong.

Cc: Yasunori Goto <y-goto@jp.fujitsu.com>
Cc: Badari Pulavarty <pbadari@us.ibm.com>
Cc: Yinghai Lu <yhlu.kernel@gmail.com>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: NAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: NLinus Torvalds <torvalds@linux-foundation.org>
上级 3a902c5f
...@@ -250,22 +250,11 @@ static unsigned long *__kmalloc_section_usemap(void) ...@@ -250,22 +250,11 @@ static unsigned long *__kmalloc_section_usemap(void)
static unsigned long *__init sparse_early_usemap_alloc(unsigned long pnum) static unsigned long *__init sparse_early_usemap_alloc(unsigned long pnum)
{ {
unsigned long *usemap, section_nr; unsigned long *usemap;
struct mem_section *ms = __nr_to_section(pnum); struct mem_section *ms = __nr_to_section(pnum);
int nid = sparse_early_nid(ms); int nid = sparse_early_nid(ms);
struct pglist_data *pgdat = NODE_DATA(nid);
/* usemap = alloc_bootmem_node(NODE_DATA(nid), usemap_size());
* Usemap's page can't be freed until freeing other sections
* which use it. And, Pgdat has same feature.
* If section A has pgdat and section B has usemap for other
* sections (includes section A), both sections can't be removed,
* because there is the dependency each other.
* To solve above issue, this collects all usemap on the same section
* which has pgdat.
*/
section_nr = pfn_to_section_nr(__pa(pgdat) >> PAGE_SHIFT);
usemap = alloc_bootmem_section(usemap_size(), section_nr);
if (usemap) if (usemap)
return usemap; return usemap;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册