[LEDE-DEV] [PATCH] include/kernel: Do not strip kernel's Elf

[Daniel Golle]

On Mon, May 16, 2016 at 05:42:43PM +0300, Alexey Brodkin wrote:
> If an image gets built as an Elf there's a chance
> it will be used by developers for debugging purposes.
> In that case it's very helpful to keep debugging info
> in that image.
> 
> I would think that most OWRT-powered devices in production
> will use some form of binary image for booting so Elf
> flavours could be left a bit bulkier with more debug info
> inside.

Well, some of the tiniest AR7 boxes out there come with a bootloader
expecting ELF binaries... I didn't have a close look, just assuming
that they might be affected. May Florian knows more...?

Cheers

Daniel

> 
> Signed-off-by: Alexey Brodkin <abrodkin at synopsys.com>
> ---
>  include/kernel-defaults.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/kernel-defaults.mk b/include/kernel-defaults.mk
> index 406fd46..0166dc5 100644
> --- a/include/kernel-defaults.mk
> +++ b/include/kernel-defaults.mk
> @@ -142,7 +142,7 @@ endif
>  define Kernel/CopyImage
>  	cmp -s $(LINUX_DIR)/vmlinux $(KERNEL_BUILD_DIR)/vmlinux$(1).debug || { \
>  		$(KERNEL_CROSS)objcopy -O binary $(OBJCOPY_STRIP) -S $(LINUX_DIR)/vmlinux $(LINUX_KERNEL)$(1); \
> -		$(KERNEL_CROSS)objcopy $(OBJCOPY_STRIP) -S $(LINUX_DIR)/vmlinux $(KERNEL_BUILD_DIR)/vmlinux$(1).elf; \
> +		$(KERNEL_CROSS)objcopy $(OBJCOPY_STRIP) $(LINUX_DIR)/vmlinux $(KERNEL_BUILD_DIR)/vmlinux$(1).elf; \
>  		$(CP) $(LINUX_DIR)/vmlinux $(KERNEL_BUILD_DIR)/vmlinux$(1).debug; \
>  		$(foreach k, \
>  			$(if $(KERNEL_IMAGES),$(KERNEL_IMAGES),$(filter-out dtbs,$(KERNELNAME))), \
> -- 
> 2.5.5
> 
> 
> _______________________________________________
> Lede-dev mailing list
> Lede-dev at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/lede-dev