lkml.org 
[lkml]   [2006]   [Sep]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] x86: update vmlinux.lds.S to place .data section on a page boundary
    On Thu, Sep 28, 2006 at 10:42:20PM +0200, Sam Ravnborg wrote:
    > On Thu, Sep 28, 2006 at 04:12:49PM -0400, Neil Horman wrote:
    > > Patch to update vmlinux linker script so that .data section is on a page
    > > boundary. without this change the kernel's .data section is on a non-4k
    > > boundary, and this prevents kexec from loading a new kernel. Tested
    > > successfully by me.
    > NAK
    >
    > > + . = ALIGN(4096);
    >
    > Do not use magic numbers like this.
    > Please replace 4096 with PAGE_SIZE - page.h is already included so it is
    > available.
    > This servers two purposes:
    > 1) This make it more self documenting
    > 2) It makes it more portable should we decide to do this in a general
    > way for all arch's.
    >
    I'm happy to do this if that is the consensus, but if you look at the rest of
    the file, I'm simply following the standard that is currently in the file. Are
    you sure you want me to go and revamp that through the entire script?

    > And then maybe a comment why it is desireable to waste a lot of RAM
    > in some cases. For the embedded people wasting up to 4088 bytes
    > of RAM is not desireable.
    >
    Again, its the standard of the script. All other sections are page aligned (or
    rather were). some recent changes have added a section data (a __tracedata
    section I think), which is 4 byte aligned, which shifted the subsequent section
    to be 4 byte aligned (the following note section as described in the PHDRS
    section is prefixed with a simmilar ALIGN macro to bring it back into page
    alignment.

    Again, I'll repost with the adjustments you request, but first I'd like you please to
    look at the file and make sure thats best. Currently, I'm more comfortable with
    the above, as it reflects the current standard of the script.

    Regards
    Neil

    >
    > Sam

    --
    /***************************************************
    *Neil Horman
    *Software Engineer
    *gpg keyid: 1024D / 0x92A74FA1 - http://pgp.mit.edu
    ***************************************************/
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2006-09-29 01:29    [W:0.023 / U:0.308 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site