On Fri, Jun 30, 2023 at 11:23:11AM -0300, Fabio Estevam wrote: > From: Fabio Estevam <[email protected]> > > Currently, spl_end points to the __bss_end address, which > is an external RAM address instead of the end of the SPL text > section in the internal RAM. > > This causes boot failures on imx6-colibri, for example: > > ``` > Trying to boot from MMC1 > SPL: Image overlaps SPL > resetting ... > ``` > Fix this problem by assigning spl_end to the _image_binary_end, as this > symbol properly represents the end of the SPL text section. > > From u-boot-spl.map: > > .end > *(.__end) > 0x00000000009121a4 _image_binary_end = . > > Fixes: 77aed22b48ab ("spl: spl_legacy: Add extra address checks") > Reported-by: Francesco Dolcini <[email protected]> > Signed-off-by: Fabio Estevam <[email protected]> > --- > Tom, > > As 2023.07 is coming in a few days, maybe it is safer to revert 77aed22b48ab > in master and then squash 77aed22b48ab + this fix to next. > > 77aed22b48ab has been applied to 2023.07-rc5, which is too late in the cycle.
I want to see what Marek says about which changes are the most critical for the security issue aspect of this and then figure out what's next. -- Tom
signature.asc
Description: PGP signature
