From: Allen Martin <amartin@nvidia.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 2/2] arm: work around assembler bug
Date: Tue, 10 Jul 2012 10:00:17 -0700 [thread overview]
Message-ID: <1341939617-26450-2-git-send-email-amartin@nvidia.com> (raw)
In-Reply-To: <1341939617-26450-1-git-send-email-amartin@nvidia.com>
Disable sibling call optimization based on binutils version. This is
to work around a bug in the assember in binutils versions < 2.22.
Branches to weak symbols can be incorrectly optimized in thumb mode to
a short branch (b.n instruction) that won't reach when the symbol gets
preempted.
http://sourceware.org/bugzilla/show_bug.cgi?id=12532
Signed-off-by: Allen Martin <amartin@nvidia.com>
---
changes for v2:
-changed GAS_BUG_12532 from yes/no to y/n to be consistent
-added additional warning about code size increase
---
arch/arm/config.mk | 21 +++++++++++++++++++++
1 file changed, 21 insertions(+)
diff --git a/arch/arm/config.mk b/arch/arm/config.mk
index 3f4453a..bcabf17 100644
--- a/arch/arm/config.mk
+++ b/arch/arm/config.mk
@@ -87,3 +87,24 @@ endif
ifndef CONFIG_NAND_SPL
LDFLAGS_u-boot += -pie
endif
+
+#
+# binutils versions < 2.22 have a bug in the assembler where branches
+# to weak symbols can be incorrectly optimized in thumb mode to a
+# short branch (b.n instruction) that won't reach when the symbol
+# gets preempted
+#
+# http://sourceware.org/bugzilla/show_bug.cgi?id=12532
+#
+ifeq ($(CONFIG_SYS_THUMB_BUILD),y)
+ifeq ($(GAS_BUG_12532),)
+export GAS_BUG_12532:=$(shell if [ $(call binutils-version) -lt 0222 ] ; then echo y; else echo n; fi)
+ifeq ($(GAS_BUG_12532),y)
+$(warning *** disabling sibling call optimzation because binutils version < 2.22)
+$(warning *** code size may be slightly larger)
+endif
+endif
+ifeq ($(GAS_BUG_12532),y)
+PLATFORM_RELFLAGS += -fno-optimize-sibling-calls
+endif
+endif
--
1.7.9.5
next prev parent reply other threads:[~2012-07-10 17:00 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-10 17:00 [U-Boot] [PATCH v2 1/2] tools, config.mk: add binutils-version Allen Martin
2012-07-10 17:00 ` Allen Martin [this message]
2012-07-10 17:25 ` [U-Boot] [PATCH v2 2/2] arm: work around assembler bug Tom Rini
2012-07-10 17:44 ` Allen Martin
2012-07-10 17:50 ` Tom Rini
2012-07-10 18:04 ` Allen Martin
2012-07-10 18:11 ` Tom Rini
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=1341939617-26450-2-git-send-email-amartin@nvidia.com \
--to=amartin@nvidia.com \
--cc=u-boot@lists.denx.de \
/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.