[lkml]   [2008]   [Jan]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    SubjectRe: bugreport kernel panic on early stage, with HIGHMEM4G:

    * Ingo Molnar <> wrote:

    > thanks for the detailed report, i think i know what's going on. Could
    > you try the patch below, does it fix your problem?

    find below the fix with a more complete changelog and with no debugging


    Subject: x86: fix boot crash on HIGHMEM4G && SPARSEMEM
    From: Ingo Molnar <>

    Denys Fedoryshchenko reported a bootup crash when he upgraded
    his system from 3GB to 4GB RAM:

    the bug is due to HIGHMEM4G && SPARSEMEM kernels making pfn_to_page() to
    return an invalid pointer when the pfn is in a memory hole. The 256 MB
    PCI aperture at the end of RAM was not mapped by sparsemem, and hence
    the pfn was not valid. But set_highmem_pages_init() iterated this range
    without checking the pfn's validity first - crashing the bootup.

    this bug was probably present in the sparsemem code ever since sparsemem
    has been introduced in v2.6.13. It was masked due to HIGHMEM64G using
    larger memory regions in sparsemem_32.h:

    #ifdef CONFIG_X86_PAE
    #define SECTION_SIZE_BITS 30
    #define MAX_PHYSADDR_BITS 36
    #define MAX_PHYSMEM_BITS 36
    #define SECTION_SIZE_BITS 26
    #define MAX_PHYSADDR_BITS 32
    #define MAX_PHYSMEM_BITS 32

    which creates 1GB sparsemem regions instead of 64MB sparsemem regions.
    So in practice we only ever created true sparsemem holes on x86 with
    HIGHMEM4G - but that was rarely used by distros.

    ( btw., we could probably save 2MB of mem_map[]s on X86_PAE if we reduced
    the sparsemem region size to 256 MB. )

    Signed-off-by: Ingo Molnar <>
    arch/x86/mm/init_32.c | 9 +++++++--
    1 file changed, 7 insertions(+), 2 deletions(-)

    Index: linux/arch/x86/mm/init_32.c
    --- linux.orig/arch/x86/mm/init_32.c
    +++ linux/arch/x86/mm/init_32.c
    @@ -321,8 +321,13 @@ extern void set_highmem_pages_init(int);
    static void __init set_highmem_pages_init(int bad_ppro)
    int pfn;
    - for (pfn = highstart_pfn; pfn < highend_pfn; pfn++)
    - add_one_highpage_init(pfn_to_page(pfn), pfn, bad_ppro);
    + for (pfn = highstart_pfn; pfn < highend_pfn; pfn++) {
    + /*
    + * Holes under sparsemem might not have no mem_map[]:
    + */
    + if (pfn_valid(pfn))
    + add_one_highpage_init(pfn_to_page(pfn), pfn, bad_ppro);
    + }
    totalram_pages += totalhigh_pages;
    #endif /* CONFIG_FLATMEM */

     \ /
      Last update: 2008-01-15 14:17    [W:0.053 / U:1.636 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site