From: Peter Korsgaard <peter@korsgaard.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v4 1/2] ts4800-mbrboot: new package
Date: Sun, 03 Jul 2016 17:44:51 +0200 [thread overview]
Message-ID: <87oa6eu3nw.fsf@dell.be.48ers.dk> (raw)
In-Reply-To: <1465498542-7333-2-git-send-email-patrick.keroulas@savoirfairelinux.com> (Patrick Keroulas's message of "Thu, 9 Jun 2016 14:55:41 -0400")
>>>>> "Patrick" == Patrick Keroulas <patrick.keroulas@savoirfairelinux.com> writes:
One more issue:
> +mbrboot.elf : mbrboot.o mbrboot.ld
> + $(CC) -Wl,--gc-sections -Wl,-static -o mbrboot.elf -nostdlib -Tmbrboot.ld -fno-pic mbrboot.o
> + size mbrboot.elf
With at least our current Linaro external toolchain, this ends up with a
2.4G big file.
It reminds me of a similar problem in the Linux kernel from some years
ago:
commit 1e621a8e3752367d4aae78a8ab00a18fb2793f34
Author: Lennert Buytenhek <buytenh@wantstofly.org>
Date: Fri Oct 12 14:38:54 2007 +0100
[ARM] 4600/1: fix kernel build failure with build-id-supporting binutils
Newer versions of binutils support --build-id, which adds an ELF
note section called ".note.gnu.build-id" to the output. On the ARM
kernel build, because there is no explicit mention of this section
in the shipped ld script, this section is placed at vaddr 0x00000000
(whereas the normal kernel text/data typically starts at vaddr
0xc0008000), causing the output of objcopy (Image) to produce a 3G+
file.
This patch makes objcopy strip the .note.gnu.build-id section from
the Image file along with all other note sections, which fixes the
build.
Signed-off-by: Lennert Buytenhek <buytenh@wantstofly.org>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=1e621a8e3752367d4aae78a8ab00a18fb2793f34
And indeed, passing -R .note.gnu.build-id to objcopy fixes it - So that
might be worth including.
--
Bye, Peter Korsgaard
next prev parent reply other threads:[~2016-07-03 15:44 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-09 18:55 [Buildroot] [PATCH v4 0/2] ARM: ts4800: add basic board support Patrick Keroulas
2016-06-09 18:55 ` [Buildroot] [PATCH v4 1/2] ts4800-mbrboot: new package Patrick Keroulas
2016-07-03 15:03 ` Peter Korsgaard
2016-07-03 15:44 ` Peter Korsgaard [this message]
2016-06-09 18:55 ` [Buildroot] [PATCH v4 2/2] configs: add defconfig for TS-4800 Patrick Keroulas
2016-07-03 15:56 ` Peter Korsgaard
2016-07-08 16:06 ` Patrick Keroulas
2016-06-09 19:12 ` [Buildroot] [PATCH v4 0/2] ARM: ts4800: add basic board support Patrick Keroulas
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87oa6eu3nw.fsf@dell.be.48ers.dk \
--to=peter@korsgaard.com \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.