• Paul Jackson's avatar
    x86 boot: proper use of ARRAY_SIZE instead of repeated E820MAX constant · c3965bd1
    Paul Jackson authored
    This patch is motivated by a subsequent patch which will allow for more
    memory map entries on EFI supported systems than can be passed via the x86
    legacy BIOS E820 interface.  The legacy interface is limited to E820MAX ==
    128 memory entries, and that "E820MAX" manifest constant was used as the
    size for several arrays and loops over those arrays.
    
    The primary change in this patch is to change code loop sizes over those
    arrays from using the constant E820MAX, to using the ARRAY_SIZE() macro
    evaluated for the array being looped.  That way, a subsequent patch can
    change the size of some of these arrays, without breaking this code.
    
    This patch also adds a parameter to the sanitize_e820_map() routine,
    which had an implicit size for the array passed it of E820MAX entries.
    This new parameter explicitly passes the size of said array.  Once again,
    this will allow a subsequent patch to change that array size for some
    calls to sanitize_e820_map() without breaking the code.
    
    As part of enhancing the sanitize_e820_map() interface this way, I further
    combined the unnecessarily distinct x86_32 and x86_64 declarations for
    this routine into a single, commonly used, declaration.
    
    This patch in itself should make no difference to the resulting kernel
    binary.
    
    [ mingo@elte.hu: merged to -tip ]
    Signed-off-by: default avatarPaul Jackson <pj@sgi.com>
    Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
    c3965bd1
setup.c 4.31 KB