ARM: zImage: Use $(CROSS_COMPILE)size to get .bss size

PATCH Superseded Thomas Weber 3 participants 4 messages Oct 13, 2011 – Nov 2, 2011 Arm (32-bit)

lore

Share & tools

lore

Thomas Weber

author

2011-10-13 06:50 UTC #0 lore raw

PATCH v2 1/1 ARM: zImage: Use $(CROSS_COMPILE)size to get .bss size

Use $(CROSS_COMPILE)size to determine the bss size from vmlinux.

The problem is introduced in:

commit 5ffb04f6690d71fab241b3562ebf52b893ac4ff1
ARM: zImage: make sure appended DTB doesn't get overwritten by kernel .bss

This fixes following error message:

size: arch/arm/boot/compressed/../../../../vmlinux:
File format is ambiguous

size: Matching formats:
elf32-littlearm
elf32-littlearm-symbian
elf32-littlearm-vxworks

Acked-by Nicolas Pitre <[email protected]>

Acked-by Tony Lindgren <[email protected]>

Signed-off-by Thomas Weber <[email protected]>

---
v2:
- Fixed commit hash and rebased to rmk/devel-stable branch.
- Added received Acked-by

 arch/arm/boot/compressed/Makefile |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

arch/arm/boot/compressed/Makefile +1 -1

index e4f32a8..3103632 100644
@@ -126,7 +126,7 @@ ccflags-y := -fpic -fno-builtin -I$(obj)
 asflags-y := -Wa,-march=all
 
 # Supply kernel BSS size to the decompressor via a linker symbol.
-KBSS_SZ = $(shell size $(obj)/../../../../vmlinux | awk 'END{print $$3}')
+KBSS_SZ = $(shell $(CROSS_COMPILE)size $(obj)/../../../../vmlinux | awk 'END{print $$3}')
 LDFLAGS_vmlinux = --defsym _kernel_bss_size=$(KBSS_SZ)
 # Supply ZRELADDR to the decompressor via a linker symbol.
 ifneq ($(CONFIG_AUTO_ZRELADDR),y)

Signature · 1 line

1.7.7

Russell King - ARM Linux

2011-10-13 08:38 UTC #1 lore raw

PATCH v2 1/1 ARM: zImage: Use $(CROSS_COMPILE)size to get .bss size

Quoted 14 lines · Thomas Weber

Use $(CROSS_COMPILE)size to determine the bss size from vmlinux.
The problem is introduced in:
commit 5ffb04f6690d71fab241b3562ebf52b893ac4ff1
ARM: zImage: make sure appended DTB doesn't get overwritten by kernel .bss
This fixes following error message:
size: arch/arm/boot/compressed/../../../../vmlinux:
File format is ambiguous

Quoted 8 lines

size: Matching formats:
elf32-littlearm
elf32-littlearm-symbian
elf32-littlearm-vxworks
Acked-by Nicolas Pitre <[email protected]>
Acked-by Tony Lindgren <[email protected]>
Signed-off-by Thomas Weber <[email protected]>

This is going to totally break my ability to build any kernels at the moment - for some reason I don't have an arm-linux-size with all the other bits of binutils, so I've been relying on the x86 version so far. I'll add it to the list of things to look at...

Nicolas Pitre

2011-10-13 18:56 UTC #2 lore raw

PATCH v2 1/1 ARM: zImage: Use $(CROSS_COMPILE)size to get .bss size

Quoted 20 lines · Russell King - ARM Linux

Quoted 19 lines · Thomas Weber
Use $(CROSS_COMPILE)size to determine the bss size from vmlinux.
The problem is introduced in:
commit 5ffb04f6690d71fab241b3562ebf52b893ac4ff1
ARM: zImage: make sure appended DTB doesn't get overwritten by kernel .bss
This fixes following error message:
size: arch/arm/boot/compressed/../../../../vmlinux:
File format is ambiguous
size: Matching formats:
elf32-littlearm
elf32-littlearm-symbian
elf32-littlearm-vxworks

Quoted 7 lines

Quoted 3 lines
Acked-by Nicolas Pitre <[email protected]>
Acked-by Tony Lindgren <[email protected]>
Signed-off-by Thomas Weber <[email protected]>
This is going to totally break my ability to build any kernels at the moment - for some reason I don't have an arm-linux-size with all the other bits of binutils, so I've been relying on the x86 version so far.

Naive suggestion: ln -s /usr/bin/size arm-linux-size

Nicolas

Thomas Weber

author

2011-11-02 12:35 UTC #3 lore raw

PATCH v2 1/1 ARM: zImage: Use $(CROSS_COMPILE)size to get .bss size

Am 13.10.2011 20:56, schrieb Nicolas Pitre:

Quoted 25 lines · Russell King - ARM Linux

Quoted 24 lines · Thomas Weber
Quoted 22 lines
Use $(CROSS_COMPILE)size to determine the bss size from vmlinux.
The problem is introduced in:
commit 5ffb04f6690d71fab241b3562ebf52b893ac4ff1
ARM: zImage: make sure appended DTB doesn't get overwritten by kernel .bss
This fixes following error message:
size: arch/arm/boot/compressed/../../../../vmlinux:
File format is ambiguous
size: Matching formats:
elf32-littlearm
elf32-littlearm-symbian
elf32-littlearm-vxworks
Acked-by Nicolas Pitre <[email protected]>
Acked-by Tony Lindgren <[email protected]>
Signed-off-by Thomas Weber <[email protected]>

Quoted 9 lines

Quoted 3 lines
This is going to totally break my ability to build any kernels at the moment - for some reason I don't have an arm-linux-size with all the other bits of binutils, so I've been relying on the x86 version so far.
Naive suggestion: ln -s /usr/bin/size arm-linux-size
Nicolas

Hello Russell,

what is the status of this patch?

Thomas