lkml.org 
[lkml]   [2009]   [Dec]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[PATCH 1/3] x86: call early_res_to_bootmem one time

    simplify setup_node_mem, do use bootmem from other node.
    instead just find_e820_area in early_node_mem.

    so we can keep the boundary between early_res and boot mem more clear.
    and only call civertion one time instead of for all nodes.

    Signed-off-by: Yinghai Lu <yinghai@kernel.org>

    ---
    arch/x86/kernel/setup.c | 1
    arch/x86/mm/init_32.c | 1
    arch/x86/mm/init_64.c | 3 --
    arch/x86/mm/numa_64.c | 62 +++++++++++++++---------------------------------
    4 files changed, 22 insertions(+), 45 deletions(-)

    Index: linux-2.6/arch/x86/kernel/setup.c
    ===================================================================
    --- linux-2.6.orig/arch/x86/kernel/setup.c
    +++ linux-2.6/arch/x86/kernel/setup.c
    @@ -942,6 +942,7 @@ void __init setup_arch(char **cmdline_p)
    #endif

    initmem_init(0, max_pfn, acpi, k8);
    + early_res_to_bootmem(0, max_low_pfn<<PAGE_SHIFT);

    #ifdef CONFIG_X86_64
    /*
    Index: linux-2.6/arch/x86/mm/init_32.c
    ===================================================================
    --- linux-2.6.orig/arch/x86/mm/init_32.c
    +++ linux-2.6/arch/x86/mm/init_32.c
    @@ -764,7 +764,6 @@ static unsigned long __init setup_node_b
    printk(KERN_INFO " node %d bootmap %08lx - %08lx\n",
    nodeid, bootmap, bootmap + bootmap_size);
    free_bootmem_with_active_regions(nodeid, end_pfn);
    - early_res_to_bootmem(start_pfn<<PAGE_SHIFT, end_pfn<<PAGE_SHIFT);

    return bootmap + bootmap_size;
    }
    Index: linux-2.6/arch/x86/mm/init_64.c
    ===================================================================
    --- linux-2.6.orig/arch/x86/mm/init_64.c
    +++ linux-2.6/arch/x86/mm/init_64.c
    @@ -578,13 +578,12 @@ void __init initmem_init(unsigned long s
    PAGE_SIZE);
    if (bootmap == -1L)
    panic("Cannot find bootmem map of size %ld\n", bootmap_size);
    + reserve_early(bootmap, bootmap + bootmap_size, "BOOTMAP");
    /* don't touch min_low_pfn */
    bootmap_size = init_bootmem_node(NODE_DATA(0), bootmap >> PAGE_SHIFT,
    0, end_pfn);
    e820_register_active_regions(0, start_pfn, end_pfn);
    free_bootmem_with_active_regions(0, end_pfn);
    - early_res_to_bootmem(0, end_pfn<<PAGE_SHIFT);
    - reserve_bootmem(bootmap, bootmap_size, BOOTMEM_DEFAULT);
    }
    #endif

    Index: linux-2.6/arch/x86/mm/numa_64.c
    ===================================================================
    --- linux-2.6.orig/arch/x86/mm/numa_64.c
    +++ linux-2.6/arch/x86/mm/numa_64.c
    @@ -164,18 +164,21 @@ static void * __init early_node_mem(int
    unsigned long align)
    {
    unsigned long mem = find_e820_area(start, end, size, align);
    - void *ptr;

    if (mem != -1L)
    return __va(mem);

    - ptr = __alloc_bootmem_nopanic(size, align, __pa(MAX_DMA_ADDRESS));
    - if (ptr == NULL) {
    - printk(KERN_ERR "Cannot find %lu bytes in node %d\n",
    +
    + start = __pa(MAX_DMA_ADDRESS);
    + end = max_low_pfn_mapped << PAGE_SHIFT;
    + mem = find_e820_area(start, end, size, align);
    + if (mem != -1L)
    + return __va(mem);
    +
    + printk(KERN_ERR "Cannot find %lu bytes in node %d\n",
    size, nodeid);
    - return NULL;
    - }
    - return ptr;
    +
    + return NULL;
    }

    /* Initialize bootmem allocator for a node */
    @@ -211,8 +214,12 @@ setup_node_bootmem(int nodeid, unsigned
    if (node_data[nodeid] == NULL)
    return;
    nodedata_phys = __pa(node_data[nodeid]);
    + reserve_early(nodedata_phys, nodedata_phys + pgdat_size, "NODE_DATA");
    printk(KERN_INFO " NODE_DATA [%016lx - %016lx]\n", nodedata_phys,
    nodedata_phys + pgdat_size - 1);
    + nid = phys_to_nid(nodedata_phys);
    + if (nid != nodeid)
    + printk(KERN_INFO " NODE_DATA(%d) on node %d\n", nodeid, nid);

    memset(NODE_DATA(nodeid), 0, sizeof(pg_data_t));
    NODE_DATA(nodeid)->bdata = &bootmem_node_data[nodeid];
    @@ -227,11 +234,7 @@ setup_node_bootmem(int nodeid, unsigned
    * of alloc_bootmem, that could clash with reserved range
    */
    bootmap_pages = bootmem_bootmap_pages(last_pfn - start_pfn);
    - nid = phys_to_nid(nodedata_phys);
    - if (nid == nodeid)
    - bootmap_start = roundup(nodedata_phys + pgdat_size, PAGE_SIZE);
    - else
    - bootmap_start = roundup(start, PAGE_SIZE);
    + bootmap_start = roundup(nodedata_phys + pgdat_size, PAGE_SIZE);
    /*
    * SMP_CACHE_BYTES could be enough, but init_bootmem_node like
    * to use that to align to PAGE_SIZE
    @@ -239,18 +242,13 @@ setup_node_bootmem(int nodeid, unsigned
    bootmap = early_node_mem(nodeid, bootmap_start, end,
    bootmap_pages<<PAGE_SHIFT, PAGE_SIZE);
    if (bootmap == NULL) {
    - if (nodedata_phys < start || nodedata_phys >= end) {
    - /*
    - * only need to free it if it is from other node
    - * bootmem
    - */
    - if (nid != nodeid)
    - free_bootmem(nodedata_phys, pgdat_size);
    - }
    + free_early(nodedata_phys, nodedata_phys + pgdat_size);
    node_data[nodeid] = NULL;
    return;
    }
    bootmap_start = __pa(bootmap);
    + reserve_early(bootmap_start, bootmap_start+(bootmap_pages<<PAGE_SHIFT),
    + "BOOTMAP");

    bootmap_size = init_bootmem_node(NODE_DATA(nodeid),
    bootmap_start >> PAGE_SHIFT,
    @@ -259,31 +257,11 @@ setup_node_bootmem(int nodeid, unsigned
    printk(KERN_INFO " bootmap [%016lx - %016lx] pages %lx\n",
    bootmap_start, bootmap_start + bootmap_size - 1,
    bootmap_pages);
    -
    - free_bootmem_with_active_regions(nodeid, end);
    -
    - /*
    - * convert early reserve to bootmem reserve earlier
    - * otherwise early_node_mem could use early reserved mem
    - * on previous node
    - */
    - early_res_to_bootmem(start, end);
    -
    - /*
    - * in some case early_node_mem could use alloc_bootmem
    - * to get range on other node, don't reserve that again
    - */
    - if (nid != nodeid)
    - printk(KERN_INFO " NODE_DATA(%d) on node %d\n", nodeid, nid);
    - else
    - reserve_bootmem_node(NODE_DATA(nodeid), nodedata_phys,
    - pgdat_size, BOOTMEM_DEFAULT);
    nid = phys_to_nid(bootmap_start);
    if (nid != nodeid)
    printk(KERN_INFO " bootmap(%d) on node %d\n", nodeid, nid);
    - else
    - reserve_bootmem_node(NODE_DATA(nodeid), bootmap_start,
    - bootmap_pages<<PAGE_SHIFT, BOOTMEM_DEFAULT);
    +
    + free_bootmem_with_active_regions(nodeid, end);

    node_set_online(nodeid);
    }

    \
     
     \ /
      Last update: 2009-12-15 03:09    [W:0.048 / U:83.108 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site