Patchworkβ [15/18] kbuild: check if linker supports the -X option

login
register
about
Submitter Andrew Morton
Date 2009-09-18 19:49:35
Message ID <200909181949.n8IJnZ7g019298@imap1.linux-foundation.org>
Download mbox | patch
Permalink /patch/48628/
State New
Headers show

Comments

Andrew Morton - 2009-09-18 19:49:35
From: Andi Kleen <andi@firstfloor.org>

The new alternative `gold' linker in recent binutils doesn't support the
-X option.  This breaks allyesconfig builds that have
CONFIG_STRIP_ASM_SYMS enabled.  Check if the linker really supports the
option using ld-option.

Signed-off-by: Andi Kleen <ak@linux.intel.com>
Cc: Sam Ravnborg <sam@ravnborg.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---

 Makefile |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Andi Kleen - 2009-09-19 01:53:39
akpm@linux-foundation.org wrote:
> From: Andi Kleen <andi@firstfloor.org>
> 
> The new alternative `gold' linker in recent binutils doesn't support the
> -X option.  This breaks allyesconfig builds that have
> CONFIG_STRIP_ASM_SYMS enabled.  Check if the linker really supports the
> option using ld-option.

I sent Sam an updated version of that patch and the "Fix ld-option to really work"
patch.

-Andi
--
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

diff -puN Makefile~kbuild-check-if-linker-supports-the-x-option Makefile
--- a/Makefile~kbuild-check-if-linker-supports-the-x-option
+++ a/Makefile
@@ -598,7 +598,7 @@  LDFLAGS_MODULE += $(LDFLAGS_BUILD_ID)
 LDFLAGS_vmlinux += $(LDFLAGS_BUILD_ID)
 
 ifeq ($(CONFIG_STRIP_ASM_SYMS),y)
-LDFLAGS_vmlinux	+= -X
+LDFLAGS_vmlinux	+= $(call ld-option, -X,)
 endif
 
 # Default kernel image to build when no specific target is given.