[lkml]   [2006]   [Aug]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [Lhms-devel] [PATCH 4/10] hot-add-mem x86_64: Enable SPARSEMEM in srat.c
    On Fri, 2006-08-04 at 18:17 +0300, Mika Penttilä wrote:
    > Keith Mannthey wrote:
    > > From: Keith Mannthey <>
    > >
    > > Enable x86_64 srat.c to share code between both reserve and sparsemem based add memory
    > > paths. Both paths need the hot-add area node locality infomration (nodes_add). This
    > > code refactors the code path to allow this.
    > >
    > > Signed-off-by: Keith Mannthey<>
    > >
    > Ok nice, but.... hotadd_enough_memory() is broken, it does weird things
    > with nd->start and nd->end which haven't been assigned even values yet.
    > Also, mysterious business with find_e820_area and last_area_end...These
    > areas are not in e820...
    Thats for pointing out the breakage in hotadd_enough_memory. I think
    the find_e820_area stuff is to make sure there box has the memory to
    reserve the maps....but it doesn't look to do it right. I can take a
    pass at a re-write for this function.

    STAT hot-add memroy areas can be outside the e820. The e820 just
    exposes the end of memory the is present in the box even though there
    maybe add area on the other size of that.

    For example my memory is layed out as follows.

    SRAT: Node 0 PXM 0 0-80000000
    SRAT: Node 0 PXM 0 0-470000000
    SRAT: Node 0 PXM 0 0-1070000000
    SRAT: hot plug zone found 470000000 - 1070000000
    SRAT: Node 1 PXM 1 1070000000-1160000000
    SRAT: Node 1 PXM 1 1070000000-3200000000
    SRAT: hot plug zone found 1160000000 - 3200000000

    The e820 ends at 1160000000 but there is still a possible add zone on
    the other side of that.

    The first hot plug zone is reserved by the e820 but no the 2nd.
    > And why the reserve_bootmem_node()? Areas not RAM (per e820) are
    > reserved anyways.

    To make sure the areas are outside of the e820 are reserved.


    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2006-08-04 21:39    [W:0.021 / U:8.248 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site