* [U-Boot] [PATCH v2 1/2] tools, config.mk: add binutils-version @ 2012-07-10 17:00 Allen Martin 2012-07-10 17:00 ` [U-Boot] [PATCH v2 2/2] arm: work around assembler bug Allen Martin 0 siblings, 1 reply; 7+ messages in thread From: Allen Martin @ 2012-07-10 17:00 UTC (permalink / raw) To: u-boot Modeled after gcc-version, add function to get binutils version. Signed-off-by: Allen Martin <amartin@nvidia.com> --- config.mk | 1 + tools/binutils-version.sh | 20 ++++++++++++++++++++ 2 files changed, 21 insertions(+) create mode 100755 tools/binutils-version.sh diff --git a/config.mk b/config.mk index c239f23..fa5e626 100644 --- a/config.mk +++ b/config.mk @@ -128,6 +128,7 @@ endif # cc-version # Usage gcc-ver := $(call cc-version) cc-version = $(shell $(SHELL) $(SRCTREE)/tools/gcc-version.sh $(CC)) +binutils-version = $(shell $(SHELL) $(SRCTREE)/tools/binutils-version.sh $(AS)) # # Include the make variables (CC, etc...) diff --git a/tools/binutils-version.sh b/tools/binutils-version.sh new file mode 100755 index 0000000..d4d9eb4 --- /dev/null +++ b/tools/binutils-version.sh @@ -0,0 +1,20 @@ +#!/bin/sh +# +# binutils-version [-p] gas-command +# +# Prints the binutils version of `gas-command' in a canonical 4-digit form +# such as `0222' for binutils 2.22 +# + +gas="$*" + +if [ ${#gas} -eq 0 ]; then + echo "Error: No assembler specified." + printf "Usage:\n\t$0 <gas-command>\n" + exit 1 +fi + +MAJOR=$($gas --version | head -1 | awk '{print $NF}' | cut -d . -f 1) +MINOR=$($gas --version | head -1 | awk '{print $NF}' | cut -d . -f 2) + +printf "%02d%02d\\n" $MAJOR $MINOR -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v2 2/2] arm: work around assembler bug 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 2012-07-10 17:25 ` Tom Rini 0 siblings, 1 reply; 7+ messages in thread From: Allen Martin @ 2012-07-10 17:00 UTC (permalink / raw) To: u-boot 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 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v2 2/2] arm: work around assembler bug 2012-07-10 17:00 ` [U-Boot] [PATCH v2 2/2] arm: work around assembler bug Allen Martin @ 2012-07-10 17:25 ` Tom Rini 2012-07-10 17:44 ` Allen Martin 0 siblings, 1 reply; 7+ messages in thread From: Tom Rini @ 2012-07-10 17:25 UTC (permalink / raw) To: u-boot On Tue, Jul 10, 2012 at 10:00:17AM -0700, Allen Martin wrote: > 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) Please don't squash out whitespace, just break this out into a multi-line test ala the others in the toplevel config.mk. > +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 This seems a bit longer than needed. Just check that it's y once and do both things. Also, I _think_ we should be able to squish this down to: ifeq ($(CONFIG_SYS_THUMB_BUILD),y) GAS_BUG_12532 := $(shell if [ $(call binutils-version) -lt 0222 ] ; \ then echo y; else echo n; fi) ifeq ($(GAS_BUG_125320),y) $(warning ...) $(warning ...) PLATFORM_RELFLAGS += ... endif endif But I haven't tested it. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20120710/2f841858/attachment.pgp> ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v2 2/2] arm: work around assembler bug 2012-07-10 17:25 ` Tom Rini @ 2012-07-10 17:44 ` Allen Martin 2012-07-10 17:50 ` Tom Rini 0 siblings, 1 reply; 7+ messages in thread From: Allen Martin @ 2012-07-10 17:44 UTC (permalink / raw) To: u-boot On Tue, Jul 10, 2012 at 10:25:05AM -0700, Tom Rini wrote: > * PGP Signed by an unknown key > > On Tue, Jul 10, 2012 at 10:00:17AM -0700, Allen Martin wrote: > > 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) > > Please don't squash out whitespace, just break this out into a > multi-line test ala the others in the toplevel config.mk. Ok > > +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 > > This seems a bit longer than needed. Just check that it's y once and do > both things. Also, I _think_ we should be able to squish this down to: > ifeq ($(CONFIG_SYS_THUMB_BUILD),y) > GAS_BUG_12532 := $(shell if [ $(call binutils-version) -lt 0222 ] ; \ > then echo y; else echo n; fi) > ifeq ($(GAS_BUG_125320),y) > $(warning ...) > $(warning ...) > PLATFORM_RELFLAGS += ... > endif > endif > > But I haven't tested it. I wrote it the way I did because I only want the expensive shell/binutils-version to get evaluated once per "make". I think what you proposed would cause it to get re-evaluated every time the build goes down into a subdir. -Allen -- nvpublic ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v2 2/2] arm: work around assembler bug 2012-07-10 17:44 ` Allen Martin @ 2012-07-10 17:50 ` Tom Rini 2012-07-10 18:04 ` Allen Martin 0 siblings, 1 reply; 7+ messages in thread From: Tom Rini @ 2012-07-10 17:50 UTC (permalink / raw) To: u-boot On Tue, Jul 10, 2012 at 10:44:33AM -0700, Allen Martin wrote: > On Tue, Jul 10, 2012 at 10:25:05AM -0700, Tom Rini wrote: > > * PGP Signed by an unknown key > > > > On Tue, Jul 10, 2012 at 10:00:17AM -0700, Allen Martin wrote: > > > 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) > > > > Please don't squash out whitespace, just break this out into a > > multi-line test ala the others in the toplevel config.mk. > > Ok > > > > +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 > > > > This seems a bit longer than needed. Just check that it's y once and do > > both things. Also, I _think_ we should be able to squish this down to: > > ifeq ($(CONFIG_SYS_THUMB_BUILD),y) > > GAS_BUG_12532 := $(shell if [ $(call binutils-version) -lt 0222 ] ; \ > > then echo y; else echo n; fi) > > ifeq ($(GAS_BUG_125320),y) > > $(warning ...) > > $(warning ...) > > PLATFORM_RELFLAGS += ... > > endif > > endif > > > > But I haven't tested it. > > I wrote it the way I did because I only want the expensive > shell/binutils-version to get evaluated once per "make". I think what > you proposed would cause it to get re-evaluated every time the build > goes down into a subdir. It should still get evaluated once, I think. Kluding the binutils-version script to echo 1 >> /tmp/test every time it's run would tell tho for sure :) -- Tom ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v2 2/2] arm: work around assembler bug 2012-07-10 17:50 ` Tom Rini @ 2012-07-10 18:04 ` Allen Martin 2012-07-10 18:11 ` Tom Rini 0 siblings, 1 reply; 7+ messages in thread From: Allen Martin @ 2012-07-10 18:04 UTC (permalink / raw) To: u-boot On Tue, Jul 10, 2012 at 10:50:34AM -0700, Tom Rini wrote: > On Tue, Jul 10, 2012 at 10:44:33AM -0700, Allen Martin wrote: > > On Tue, Jul 10, 2012 at 10:25:05AM -0700, Tom Rini wrote: > > > * PGP Signed by an unknown key > > > > > > On Tue, Jul 10, 2012 at 10:00:17AM -0700, Allen Martin wrote: > > > > 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) > > > > > > Please don't squash out whitespace, just break this out into a > > > multi-line test ala the others in the toplevel config.mk. > > > > Ok > > > > > > +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 > > > > > > This seems a bit longer than needed. Just check that it's y once and do > > > both things. Also, I _think_ we should be able to squish this down to: > > > ifeq ($(CONFIG_SYS_THUMB_BUILD),y) > > > GAS_BUG_12532 := $(shell if [ $(call binutils-version) -lt 0222 ] ; \ > > > then echo y; else echo n; fi) > > > ifeq ($(GAS_BUG_125320),y) > > > $(warning ...) > > > $(warning ...) > > > PLATFORM_RELFLAGS += ... > > > endif > > > endif > > > > > > But I haven't tested it. > > > > I wrote it the way I did because I only want the expensive > > shell/binutils-version to get evaluated once per "make". I think what > > you proposed would cause it to get re-evaluated every time the build > > goes down into a subdir. > > It should still get evaluated once, I think. Kluding the > binutils-version script to echo 1 >> /tmp/test every time it's run would > tell tho for sure :) I tried it out, it's obvious from the $(warning) it's getting reevaluated: $ ./MAKEALL -s tegra20 /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22 /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger Configuring for harmony board... text data bss dec hex filename 170334 4284 278352 452970 6e96a ./u-boot 11676 161 36 11873 2e61 ./spl/u-boot-spl /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22 /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22 /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22 /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22 /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22 /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22 /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger ... and on and on It makes sense since unless GAS_BUG_125320 is "export"ed to the new make, it's going to be unset again each the build starts a new "make -C" -Allen -- nvpublic ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v2 2/2] arm: work around assembler bug 2012-07-10 18:04 ` Allen Martin @ 2012-07-10 18:11 ` Tom Rini 0 siblings, 0 replies; 7+ messages in thread From: Tom Rini @ 2012-07-10 18:11 UTC (permalink / raw) To: u-boot On Tue, Jul 10, 2012 at 11:04:50AM -0700, Allen Martin wrote: > On Tue, Jul 10, 2012 at 10:50:34AM -0700, Tom Rini wrote: > > On Tue, Jul 10, 2012 at 10:44:33AM -0700, Allen Martin wrote: > > > On Tue, Jul 10, 2012 at 10:25:05AM -0700, Tom Rini wrote: > > > > * PGP Signed by an unknown key > > > > > > > > On Tue, Jul 10, 2012 at 10:00:17AM -0700, Allen Martin wrote: > > > > > 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) > > > > > > > > Please don't squash out whitespace, just break this out into a > > > > multi-line test ala the others in the toplevel config.mk. > > > > > > Ok > > > > > > > > +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 > > > > > > > > This seems a bit longer than needed. Just check that it's y once and do > > > > both things. Also, I _think_ we should be able to squish this down to: > > > > ifeq ($(CONFIG_SYS_THUMB_BUILD),y) > > > > GAS_BUG_12532 := $(shell if [ $(call binutils-version) -lt 0222 ] ; \ > > > > then echo y; else echo n; fi) > > > > ifeq ($(GAS_BUG_125320),y) > > > > $(warning ...) > > > > $(warning ...) > > > > PLATFORM_RELFLAGS += ... > > > > endif > > > > endif > > > > > > > > But I haven't tested it. > > > > > > I wrote it the way I did because I only want the expensive > > > shell/binutils-version to get evaluated once per "make". I think what > > > you proposed would cause it to get re-evaluated every time the build > > > goes down into a subdir. > > > > It should still get evaluated once, I think. Kluding the > > binutils-version script to echo 1 >> /tmp/test every time it's run would > > tell tho for sure :) > > I tried it out, it's obvious from the $(warning) it's getting > reevaluated: > > $ ./MAKEALL -s tegra20 > /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22 > /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger > Configuring for harmony board... > text data bss dec hex filename > 170334 4284 278352 452970 6e96a ./u-boot > 11676 161 36 11873 2e61 ./spl/u-boot-spl > /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22 > /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger > /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22 > /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger > /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22 > /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger > /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22 > /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger > /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22 > /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger > /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22 > /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger > ... and on and on > > > It makes sense since unless GAS_BUG_125320 is "export"ed to the new > make, it's going to be unset again each the build starts a new > "make -C" Thanks for proving me wrong. Just the white-space / multi line thing please, thanks :) -- Tom ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2012-07-10 18:11 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2012-07-10 17:00 [U-Boot] [PATCH v2 1/2] tools, config.mk: add binutils-version Allen Martin 2012-07-10 17:00 ` [U-Boot] [PATCH v2 2/2] arm: work around assembler bug Allen Martin 2012-07-10 17:25 ` 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
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.