Commit:     4c4915627f94a81a834a7a65dee83acdfb45788c
Parent:     0c2bd5a5e3b9469c6ab2b96c403980f192db0bf7
Author:     Eric Dumazet <[EMAIL PROTECTED]>
AuthorDate: Wed Jan 30 13:32:50 2008 +0100
Committer:  Ingo Molnar <[EMAIL PROTECTED]>
CommitDate: Wed Jan 30 13:32:50 2008 +0100

    x86: make arch/x86/kernel/acpi/wakeup_32.S use a separate
    While examining vmlinux namelist on i386 (nm -v vmlinux) I noticed :
    c01021d0 t es7000_rename_gsi
    c010221a T es7000_start_cpu
    <Big Hole>
    c0103000 T thread_saved_pc
    c0113218 T acpi_restore_state_mem
    c0113219 T acpi_save_state_mem
    <Big Hole>
    c0114000 t wakeup_code
    This is because arch/x86/kernel/acpi/wakeup_32.S forces a .text alignment
    of 4096 bytes. (I have no idea if it is really needed, since
    arch/x86/kernel/acpi/wakeup_64.S uses a 16 bytes alignment *only*)
    So arch/x86/kernel/built-in.o also has this alignment
    arch/x86/kernel/built-in.o:     file format elf32-i386
    Idx Name          Size      VMA       LMA       File off  Algn
      0 .text         00018c94  00000000  00000000  00001000  2**12
    But as arch/x86/kernel/acpi/wakeup_32.o is not the first object linked
    into arch/x86/kernel/built-in.o, linker had to build several holes to meet
    alignement requirements, because of .o nestings in the kbuild process.
    This can be solved by using a special section, .text.page_aligned, so that
    no holes are needed.
    # size vmlinux.before vmlinux.after
       text    data     bss     dec     hex filename
    4619942  422838  458752 5501532  53f25c vmlinux.before
    4610534  422838  458752 5492124  53cd9c vmlinux.after
    This saves 9408 bytes
    Signed-off-by: Eric Dumazet <[EMAIL PROTECTED]>
    Signed-off-by: Ingo Molnar <[EMAIL PROTECTED]>
    Signed-off-by: Thomas Gleixner <[EMAIL PROTECTED]>
 arch/x86/kernel/acpi/wakeup_32.S |    2 +-
 arch/x86/kernel/ |    2 ++
 2 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/arch/x86/kernel/acpi/wakeup_32.S b/arch/x86/kernel/acpi/wakeup_32.S
index 1e931aa..f53e327 100644
--- a/arch/x86/kernel/acpi/wakeup_32.S
+++ b/arch/x86/kernel/acpi/wakeup_32.S
@@ -1,4 +1,4 @@
+       .section .text.page_aligned
 #include <linux/linkage.h>
 #include <asm/segment.h>
 #include <asm/page.h>
diff --git a/arch/x86/kernel/ b/arch/x86/kernel/
index ec07258..f1148ac 100644
--- a/arch/x86/kernel/
+++ b/arch/x86/kernel/
@@ -38,6 +38,8 @@ SECTIONS
   /* read-only */
   .text : AT(ADDR(.text) - LOAD_OFFSET) {
+       . = ALIGN(4096); /* not really needed, already page aligned */
+       *(.text.page_aligned)
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at

Reply via email to