Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-11 Thread Albert ARIBAUD
On Fri, 5 Apr 2013 00:13:53 +0200, Albert ARIBAUD albert.u.b...@aribaud.net wrote: Commit 3ebd1cbc introduced compiler-generated __bss_start and __bss_end__ and commit c23561e7 rewrote all __bss_end__ as __bss_end. Their merge caused silent and harmless but potentially bug-inducing clashes

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-11 Thread Tom Warren
Albert, -Original Message- From: Stephen Warren [mailto:swar...@wwwdotorg.org] Sent: Friday, April 05, 2013 8:59 AM To: Tom Rini Cc: Albert ARIBAUD; Scott Wood; u-boot@lists.denx.de; step...@theia.denx.de; Tom Warren; Simon Glass; Allen Martin Subject: Re: [U-Boot] [PATCH] ARM: Fix

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-11 Thread Albert ARIBAUD
...@theia.denx.de; Tom Warren; Simon Glass; Allen Martin Subject: Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts I just fetched merged TOT u-boot-arm (getting ready for a PR from u-boot-tegra - u-boot-arm), and did a test build of all Tegra boards. I get

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-11 Thread Tom Warren
05, 2013 8:59 AM To: Tom Rini Cc: Albert ARIBAUD; Scott Wood; u-boot@lists.denx.de; step...@theia.denx.de; Tom Warren; Simon Glass; Allen Martin Subject: Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts I just fetched merged TOT u-boot-arm (getting ready

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-08 Thread Albert ARIBAUD
Hi Tom, On Fri, 5 Apr 2013 16:23:30 -0400, Tom Rini tr...@ti.com wrote: On Fri, Apr 05, 2013 at 10:04:02PM +0200, Albert ARIBAUD wrote: The two general ideas of my proposal are: 1) to separate testing the image (text,data,rodata,lists) size on the one hand and the image BSS size

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-08 Thread Tom Rini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/08/2013 03:03 PM, Albert ARIBAUD wrote: Hi Tom, On Fri, 5 Apr 2013 16:23:30 -0400, Tom Rini tr...@ti.com wrote: On Fri, Apr 05, 2013 at 10:04:02PM +0200, Albert ARIBAUD wrote: The two general ideas of my proposal are: 1) to separate

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-08 Thread Albert ARIBAUD
Hi Tom, On Mon, 8 Apr 2013 15:56:12 -0400, Tom Rini tr...@ti.com wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/08/2013 03:03 PM, Albert ARIBAUD wrote: Hi Tom, On Fri, 5 Apr 2013 16:23:30 -0400, Tom Rini tr...@ti.com wrote: On Fri, Apr 05, 2013 at 10:04:02PM +0200,

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-05 Thread Albert ARIBAUD
Hi Benoît, CC:ing Stephen Warren who wrote commit 2b7818d4 which git blame tells me added the ASSERT() to arch/arm/cpu/u-boot.lds, and Tom Rini to help decide what to do. On Fri, 5 Apr 2013 05:44:55 +0200 (CEST), Benoît Thébaudeau benoit.thebaud...@advansee.com wrote: Hi Albert, On Friday,

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-05 Thread Tom Rini
On Fri, Apr 05, 2013 at 08:00:43AM +0200, Albert ARIBAUD wrote: Hi Beno??t, CC:ing Stephen Warren who wrote commit 2b7818d4 which git blame tells me added the ASSERT() to arch/arm/cpu/u-boot.lds, and Tom Rini to help decide what to do. [snip] Looks good, but what about the __bss_end in

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-05 Thread Benoît Thébaudeau
Hi Albert, On Friday, April 5, 2013 8:00:43 AM, Albert ARIBAUD wrote: Hi Benoît, CC:ing Stephen Warren who wrote commit 2b7818d4 which git blame tells me added the ASSERT() to arch/arm/cpu/u-boot.lds, and Tom Rini to help decide what to do. On Fri, 5 Apr 2013 05:44:55 +0200 (CEST),

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-05 Thread Stephen Warren
On 04/05/2013 07:53 AM, Tom Rini wrote: On Fri, Apr 05, 2013 at 08:00:43AM +0200, Albert ARIBAUD wrote: Hi Beno??t, CC:ing Stephen Warren who wrote commit 2b7818d4 which git blame tells me added the ASSERT() to arch/arm/cpu/u-boot.lds, and Tom Rini to help decide what to do. [snip] Looks

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-05 Thread Tom Rini
On Fri, Apr 05, 2013 at 03:56:46PM +0200, Beno??t Th??baudeau wrote: Hi Albert, On Friday, April 5, 2013 8:00:43 AM, Albert ARIBAUD wrote: Hi Beno??t, [snip] IIUC, this future patch would increase the limit for SPL run-time size, as the constant against which the ASS tests __bss_end for

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-05 Thread Benoît Thébaudeau
Hi Tom, On Friday, April 5, 2013 6:00:30 PM, Tom Rini wrote: On Fri, Apr 05, 2013 at 03:56:46PM +0200, Beno??t Th??baudeau wrote: Hi Albert, On Friday, April 5, 2013 8:00:43 AM, Albert ARIBAUD wrote: Hi Beno??t, [snip] IIUC, this future patch would increase the limit for SPL

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-05 Thread Tom Rini
On Fri, Apr 05, 2013 at 07:32:54PM +0200, Beno??t Th??baudeau wrote: Hi Tom, On Friday, April 5, 2013 6:00:30 PM, Tom Rini wrote: On Fri, Apr 05, 2013 at 03:56:46PM +0200, Beno??t Th??baudeau wrote: Hi Albert, On Friday, April 5, 2013 8:00:43 AM, Albert ARIBAUD wrote: Hi

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-05 Thread Albert ARIBAUD
Hi Tom, On Fri, 5 Apr 2013 13:55:21 -0400, Tom Rini tr...@ti.com wrote: On Fri, Apr 05, 2013 at 07:32:54PM +0200, Beno??t Th??baudeau wrote: Hi Tom, On Friday, April 5, 2013 6:00:30 PM, Tom Rini wrote: On Fri, Apr 05, 2013 at 03:56:46PM +0200, Beno??t Th??baudeau wrote: Hi Albert,

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-05 Thread Albert ARIBAUD
On Fri, 5 Apr 2013 21:17:40 +0200, Albert ARIBAUD albert.u.b...@aribaud.net wrote: Hi Tom, On Fri, 5 Apr 2013 13:55:21 -0400, Tom Rini tr...@ti.com wrote: On Fri, Apr 05, 2013 at 07:32:54PM +0200, Beno??t Th??baudeau wrote: Hi Tom, On Friday, April 5, 2013 6:00:30 PM, Tom Rini

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-05 Thread Tom Rini
On Fri, Apr 05, 2013 at 09:17:40PM +0200, Albert ARIBAUD wrote: Hi Tom, On Fri, 5 Apr 2013 13:55:21 -0400, Tom Rini tr...@ti.com wrote: On Fri, Apr 05, 2013 at 07:32:54PM +0200, Beno??t Th??baudeau wrote: Hi Tom, On Friday, April 5, 2013 6:00:30 PM, Tom Rini wrote: On Fri,

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-05 Thread Albert ARIBAUD
Hi Tom, On Fri, 5 Apr 2013 15:44:08 -0400, Tom Rini tr...@ti.com wrote: On Fri, Apr 05, 2013 at 09:17:40PM +0200, Albert ARIBAUD wrote: Hi Tom, On Fri, 5 Apr 2013 13:55:21 -0400, Tom Rini tr...@ti.com wrote: On Fri, Apr 05, 2013 at 07:32:54PM +0200, Beno??t Th??baudeau wrote:

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-05 Thread Tom Rini
On Fri, Apr 05, 2013 at 10:04:02PM +0200, Albert ARIBAUD wrote: The two general ideas of my proposal are: 1) to separate testing the image (text,data,rodata,lists) size on the one hand and the image BSS size on the other hand, and 2) to consider that if a target defines an image max

[U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-04 Thread Albert ARIBAUD
Commit 3ebd1cbc introduced compiler-generated __bss_start and __bss_end__ and commit c23561e7 rewrote all __bss_end__ as __bss_end. Their merge caused silent and harmless but potentially bug-inducing clashes between compiler- and linker- enerated __bss_end symbols. Make __bss_end and __bss_start

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-04 Thread Benoît Thébaudeau
Hi Albert, On Friday, April 5, 2013 12:13:53 AM, Albert ARIBAUD wrote: Subject: [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts Commit 3ebd1cbc introduced compiler-generated __bss_start and __bss_end__ and commit c23561e7 rewrote all __bss_end__ as __bss_end. Their merge caused

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-04 Thread Albert ARIBAUD
Hi Albert, On Fri, 5 Apr 2013 00:13:53 +0200, Albert ARIBAUD albert.u.b...@aribaud.net wrote: Commit 3ebd1cbc introduced compiler-generated __bss_start and __bss_end__ and commit c23561e7 rewrote all __bss_end__ as __bss_end. Their merge caused silent and harmless but potentially

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-04 Thread Benoît Thébaudeau
On Friday, April 5, 2013 1:05:53 AM, Benoît Thébaudeau wrote: Hi Albert, On Friday, April 5, 2013 12:13:53 AM, Albert ARIBAUD wrote: Subject: [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts Commit 3ebd1cbc introduced compiler-generated __bss_start and __bss_end__ and

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-04 Thread Albert ARIBAUD
Hi Benoît, On Fri, 5 Apr 2013 01:13:51 +0200 (CEST), Benoît Thébaudeau benoit.thebaud...@advansee.com wrote: On Friday, April 5, 2013 1:05:53 AM, Benoît Thébaudeau wrote: Hi Albert, On Friday, April 5, 2013 12:13:53 AM, Albert ARIBAUD wrote: Subject: [PATCH] ARM: Fix __bss_start and

Re: [U-Boot] [PATCH] ARM: Fix __bss_start and __bss_end in linker scripts

2013-04-04 Thread Benoît Thébaudeau
Hi Albert, On Friday, April 5, 2013 1:54:38 AM, Albert ARIBAUD wrote: Hi Benoît, On Fri, 5 Apr 2013 01:13:51 +0200 (CEST), Benoît Thébaudeau benoit.thebaud...@advansee.com wrote: On Friday, April 5, 2013 1:05:53 AM, Benoît Thébaudeau wrote: Hi Albert, On Friday, April 5, 2013