Re: Linking issue with ld.bfd 2.28.0.20170307

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi!

U-boot has fix for this issue:
http://git.denx.de/?p=u-boot.git;a=commit;h=e391b1e64b0bd65709a28a4764afe4f32d408243

With attached patch I was able to build barebox with new ld. Tested on imx6 arch only.


19.04.2017, 13:31, "Sascha Hauer" <s.hauer@xxxxxxxxxxxxxx>:
> Hi Holger,
>
> On Mon, Apr 17, 2017 at 07:35:19AM +0200, Holger Freyther wrote:
>>  > On 30. Mar 2017, at 20:49, Holger Freyther <holger@xxxxxxxxxxxxxxxx> wrote:
>>  >
>>  > Hi,
>>
>>  Hi!
>>
>>  > I am rebuilding the bootloader with gcc6 and the above version
>>  > of binutils and it is failing to link with .bfd but is working
>>  > with .gold.
>>  >
>>  > Is this known? Is anyone else seeing this issue? And no I did
>>  > not try linking with -N and I have no idea what ld thinks is
>>  > the program headers here.
>>
>>  my steps to reproduce this with Yocto's gcc-6.3 and ld.bfd
>>  2.28.0.20170307 are the following. CROSS_COMPILE points into
>>  the sysroot of a build on poky master.
>>
>>  git clone --branch=hfreyther/v2017.03-rebase git://git.sysmocom.de/barebox
>>  cd barebox/
>>  wget -O .config "http://git.sysmocom.de/poky/meta-sysmocom-bsp/plain/recipes-bsp/barebox/barebox-sysmobts/defconfig?id=39afcd35ae40b8468868f1e942706dec11e03961";
>>  make ARCH=arm
>>
>>  .../arm-poky-linux-gnueabi-ld: .tmp_barebox1: Not enough room for program headers, try linking with -N
>>  .../arm-poky-linux-gnueabi-ld: final link failed: Bad value
>
> I'm afraid nobody currently uses gcc6 for barebox. I have no idea what's
> going wrong here, sorry. Does it work when you disable
> CONFIG_RELOCATABLE?
>
> Sascha
>
> --
> Pengutronix e.K. | |
> Industrial Linux Solutions | http://www.pengutronix.de/ |
> Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
> Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
>
> _______________________________________________
> barebox mailing list
> barebox@xxxxxxxxxxxxxxxxxxx
> http://lists.infradead.org/mailman/listinfo/barebox

-- 
Андрей
From fc6dd69d33e3e3285d7efe1647af8f1ae3194846 Mon Sep 17 00:00:00 2001
From: Andrey Panov <rockford@xxxxxxxxx>
Date: Wed, 19 Apr 2017 22:25:17 +0300
Subject: [PATCH] Fix linking with new ld, based on u-boot

---
 Makefile               | 3 +++
 images/Makefile        | 3 +++
 scripts/Kbuild.include | 2 +-
 3 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/Makefile b/Makefile
index afa4bf802..c78f66f73 100644
--- a/Makefile
+++ b/Makefile
@@ -306,6 +306,9 @@ AFLAGS          := -D__ASSEMBLY__
 
 LDFLAGS_barebox	:= -Map barebox.map
 
+# Avoid 'Not enough room for program headers' error on binutils 2.28 onwards.
+LDFLAGS_barebox += $(call ld-option, --no-dynamic-linker)
+
 # Read KERNELRELEASE from include/config/kernel.release (if it exists)
 KERNELRELEASE = $(shell cat include/config/kernel.release 2> /dev/null)
 KERNELVERSION = $(VERSION).$(PATCHLEVEL).$(SUBLEVEL)$(EXTRAVERSION)
diff --git a/images/Makefile b/images/Makefile
index adf950aa9..8c5dac4bf 100644
--- a/images/Makefile
+++ b/images/Makefile
@@ -53,6 +53,9 @@ extra-y += $(pbl-lds)
 $(pbl-lds): $(obj)/../arch/$(ARCH)/lib/pbl.lds.S FORCE
 	$(call if_changed_dep,cpp_lds_S)
 
+# Avoid 'Not enough room for program headers' error on binutils 2.28 onwards.
+LDFLAGS += $(call ld-option, --no-dynamic-linker)
+
 quiet_cmd_elf__ ?= LD      $@
       cmd_elf__ ?= $(LD) $(LDFLAGS) --gc-sections -pie			\
 		-e $(2) -Map $@.map $(LDFLAGS_$(@F)) -o $@		\
diff --git a/scripts/Kbuild.include b/scripts/Kbuild.include
index c7faf67a2..310dcdc94 100644
--- a/scripts/Kbuild.include
+++ b/scripts/Kbuild.include
@@ -148,7 +148,7 @@ cc-ldoption = $(call try-run,\
 # ld-option
 # Usage: LDFLAGS += $(call ld-option, -X)
 ld-option = $(call try-run,\
-	$(CC) /dev/null -c -o "$$TMPO" ; $(LD) $(1) "$$TMPO" -o "$$TMP",$(1),$(2))
+	$(CC) -x c /dev/null -c -o "$$TMPO" ; $(LD) $(1) "$$TMPO" -o "$$TMP",$(1),$(2))
 
 # ar-option
 # Usage: KBUILD_ARFLAGS := $(call ar-option,D)
-- 
2.11.0

_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox

[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux