Re: [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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



Relevant Pages

  • Re: [kde] Suspend Issues, or soft kernel locks + no networking, which is worse?
    ... And thats kernel 3.0 essentially) ... I don't claim to be a laptop or wireless guru, by a long shot, but I do ... at least try loading that ubuntu/mint kernel on fedora and see if it ... custom sleep script for ehci-hcd that worked for them. ...
    (KDE)
  • systemtap 1.2 release notes
    ... The systemtap team announces release 1.2. ... a script if any warnings are produced. ... where the kernel supports the ... Probe points may refer to low-level ...
    (Linux-Kernel)
  • Re: cx88 totally fried in 2.6.15-rcX -was- Re: HD3000 - no NTSC via tuner
    ... On Monday 28 November 2005 20:17, Michael Krufky wrote: ... as it would apply to a working 2.6.14.x kernel. ... that I comment/uncomment stuff in the buildit (thats another script I ... message by Gene Heskett are: ...
    (Linux-Kernel)
  • systemtap release 1.4
    ... The systemtap team announces release 1.4. ... prototype script remote-execution via ssh, ... More kernel tracepoints are accessible to the kernel.tracemechanism, ... Some kernel crashes continue to be reported when a script probes ...
    (Linux-Kernel)
  • Re: 2.6.22-rc5 regression
    ... Bisecting: 128 revisions left to test after this ... of the fact that I use the same directory tree to build the kernel ... [torvalds@woody linux]$ git bisect bad v2.6.22-rc5 ... -# Simple script to generate a deb package for a Linux kernel. ...
    (Linux-Kernel)