diff options
author | Ingo Molnar <mingo@elte.hu> | 2009-10-15 02:08:12 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-10-15 02:09:55 -0400 |
commit | db8590f5043f3436a65b24155a3a7af2604df876 (patch) | |
tree | 5a2bafc2102c37a9cee1d2dea6176bc75b89ef8e | |
parent | a0738a688d1105cbf8d71868a1e020c6cdf42d4c (diff) |
Revert "x86: linker script syntax nits"
This reverts commit e9a63a4e559fbdc522072281d05e6b13c1022f4b.
This breaks older binutils, where sink-less asserts are broken.
See this commit for further details:
d2ba8b2: x86: Fix assert syntax in vmlinux.lds.S
Acked-by: "H. Peter Anvin" <hpa@zytor.com>
Acked-by: Sam Ravnborg <sam@ravnborg.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
LKML-Reference: <4AD6523D.5030909@zytor.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r-- | arch/x86/kernel/acpi/realmode/wakeup.lds.S | 4 | ||||
-rw-r--r-- | arch/x86/kernel/vmlinux.lds.S | 17 |
2 files changed, 11 insertions, 10 deletions
diff --git a/arch/x86/kernel/acpi/realmode/wakeup.lds.S b/arch/x86/kernel/acpi/realmode/wakeup.lds.S index 0e50e1e5c573..7da00b799cda 100644 --- a/arch/x86/kernel/acpi/realmode/wakeup.lds.S +++ b/arch/x86/kernel/acpi/realmode/wakeup.lds.S | |||
@@ -56,6 +56,6 @@ SECTIONS | |||
56 | /DISCARD/ : { | 56 | /DISCARD/ : { |
57 | *(.note*) | 57 | *(.note*) |
58 | } | 58 | } |
59 | } | ||
60 | 59 | ||
61 | ASSERT(_end <= WAKEUP_SIZE, "Wakeup too big!"); | 60 | . = ASSERT(_end <= WAKEUP_SIZE, "Wakeup too big!"); |
61 | } | ||
diff --git a/arch/x86/kernel/vmlinux.lds.S b/arch/x86/kernel/vmlinux.lds.S index 8d6001ad8d8d..92929fb3f9fa 100644 --- a/arch/x86/kernel/vmlinux.lds.S +++ b/arch/x86/kernel/vmlinux.lds.S | |||
@@ -305,8 +305,8 @@ SECTIONS | |||
305 | 305 | ||
306 | 306 | ||
307 | #ifdef CONFIG_X86_32 | 307 | #ifdef CONFIG_X86_32 |
308 | ASSERT((_end - LOAD_OFFSET <= KERNEL_IMAGE_SIZE), | 308 | . = ASSERT((_end - LOAD_OFFSET <= KERNEL_IMAGE_SIZE), |
309 | "kernel image bigger than KERNEL_IMAGE_SIZE"); | 309 | "kernel image bigger than KERNEL_IMAGE_SIZE"); |
310 | #else | 310 | #else |
311 | /* | 311 | /* |
312 | * Per-cpu symbols which need to be offset from __per_cpu_load | 312 | * Per-cpu symbols which need to be offset from __per_cpu_load |
@@ -319,12 +319,12 @@ INIT_PER_CPU(irq_stack_union); | |||
319 | /* | 319 | /* |
320 | * Build-time check on the image size: | 320 | * Build-time check on the image size: |
321 | */ | 321 | */ |
322 | ASSERT((_end - _text <= KERNEL_IMAGE_SIZE), | 322 | . = ASSERT((_end - _text <= KERNEL_IMAGE_SIZE), |
323 | "kernel image bigger than KERNEL_IMAGE_SIZE"); | 323 | "kernel image bigger than KERNEL_IMAGE_SIZE"); |
324 | 324 | ||
325 | #ifdef CONFIG_SMP | 325 | #ifdef CONFIG_SMP |
326 | ASSERT((per_cpu__irq_stack_union == 0), | 326 | . = ASSERT((per_cpu__irq_stack_union == 0), |
327 | "irq_stack_union is not at start of per-cpu area"); | 327 | "irq_stack_union is not at start of per-cpu area"); |
328 | #endif | 328 | #endif |
329 | 329 | ||
330 | #endif /* CONFIG_X86_32 */ | 330 | #endif /* CONFIG_X86_32 */ |
@@ -332,6 +332,7 @@ ASSERT((per_cpu__irq_stack_union == 0), | |||
332 | #ifdef CONFIG_KEXEC | 332 | #ifdef CONFIG_KEXEC |
333 | #include <asm/kexec.h> | 333 | #include <asm/kexec.h> |
334 | 334 | ||
335 | ASSERT(kexec_control_code_size <= KEXEC_CONTROL_CODE_MAX_SIZE, | 335 | . = ASSERT(kexec_control_code_size <= KEXEC_CONTROL_CODE_MAX_SIZE, |
336 | "kexec control code size is too big"); | 336 | "kexec control code size is too big"); |
337 | #endif | 337 | #endif |
338 | |||