Skip to content

Commit b30d028

Browse files
okiasRussell King (Oracle)
authored andcommitted
ARM: 9105/1: atags_to_fdt: don't warn about stack size
The merge_fdt_bootargs() function by definition consumes more than 1024 bytes of stack because it has a 1024 byte command line on the stack, meaning that we always get a warning when building this file: arch/arm/boot/compressed/atags_to_fdt.c: In function 'merge_fdt_bootargs': arch/arm/boot/compressed/atags_to_fdt.c:98:1: warning: the frame size of 1032 bytes is larger than 1024 bytes [-Wframe-larger-than=] However, as this is the decompressor and we know that it has a very shallow call chain, and we do not actually risk overflowing the kernel stack at runtime here. This just shuts up the warning by disabling the warning flag for this file. Tested on Nexus 7 2012 builds. Acked-by: Nicolas Pitre <[email protected]> Signed-off-by: David Heidelberg <[email protected]> Signed-off-by: Arnd Bergmann <[email protected]> Cc: <[email protected]> Signed-off-by: Russell King (Oracle) <[email protected]>
1 parent 6fec92d commit b30d028

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

arch/arm/boot/compressed/Makefile

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,8 @@ compress-$(CONFIG_KERNEL_LZ4) = lz4
8585
libfdt_objs := fdt_rw.o fdt_ro.o fdt_wip.o fdt.o
8686

8787
ifeq ($(CONFIG_ARM_ATAG_DTB_COMPAT),y)
88+
CFLAGS_REMOVE_atags_to_fdt.o += -Wframe-larger-than=${CONFIG_FRAME_WARN}
89+
CFLAGS_atags_to_fdt.o += -Wframe-larger-than=1280
8890
OBJS += $(libfdt_objs) atags_to_fdt.o
8991
endif
9092
ifeq ($(CONFIG_USE_OF),y)

0 commit comments

Comments
 (0)