v2023.07-rc5 regression: Image overlaps SPL

Tom Rini trini at konsulko.com
Fri Jun 30 16:05:25 CEST 2023


On Fri, Jun 30, 2023 at 10:51:43AM -0300, Fabio Estevam wrote:
> On Fri, Jun 30, 2023 at 10:22 AM Fabio Estevam <festevam at gmail.com> wrote:
> 
> > Should we fix spl_end like this?
> 
> Looking at u-boot-spl.map:
> 
>                 0x0000000000912264        0x0 common/spl/spl.o
>                 0x0000000000912264                . = ALIGN (0x4)
>                 0x0000000000912264                __image_copy_end = .
> 
> .end
>  *(.__end)
>                 0x0000000000912264                _image_binary_end = .
> 
> __image_copy_end is the last address of the SPL inside the internal RAM.
> 
> So shouldn't we do this instead?
> 
> --- a/common/spl/spl_legacy.c
> +++ b/common/spl/spl_legacy.c
> @@ -19,9 +19,17 @@
>  static void spl_parse_legacy_validate(uintptr_t start, uintptr_t size)
>  {
>         uintptr_t spl_start = (uintptr_t)_start;
> -       uintptr_t spl_end = (uintptr_t)__bss_end;
> +       uintptr_t spl_end = (uintptr_t)__image_copy_end;
>         uintptr_t end = start + size;

I think this breaks x86, without updating their linker scripts at least.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230630/31d96961/attachment.sig>


More information about the U-Boot mailing list