[U-Boot] [PATCH 3/4] ARM: tegra: check for SPL size overflow in makefile

Stephen Warren
2012-10-18T20:45:51Z

On 10/18/2012 10:27 AM, Tom Rini wrote:
> On Tue, Oct 16, 2012 at 03:50:08PM -0600, Stephen Warren wrote:
>
>> From: Stephen Warren <swarren@nvidia.com>
>>
>> If the SPL extends beyond CONFIG_SYS_TEXT_BASE, then it will
>> likely corrupt the main U-Boot binary during execution, causing
>> the main U-Boot binary to fail. Check for this situation during
>> the build to avoid extremely annoying and hard-to-find bugs. Note
>> that checking the size of u-boot-spl.bin is not enough, since BSS
>> size doesn't affect the size of u-boot-spl.bin.
>>
>> Signed-off-by: Stephen Warren <swarren@nvidia.com>
>
> Can't you do this in the linker script like we do for other SPL
> size constraints? Or am I just mis-reading how this is unique and
> that link-time check can't be used? Thanks!

Ah, there aren't any such checks in the linker script I looked at, so
I wasn't aware of this capability. I found the following in a
different linker script:

ASSERT(__bss_end__ <= 0xfff01000, "NAND bootstrap too big");

I agree using that technique would make sense; I'll try it out.