On 08/10/2020 14:02, Heinrich Schuchardt wrote:
Hi,
On 25.08.20 18:57, Simon Glass wrote:
On Mon, 24 Aug 2020 at 22:15, Heinrich Schuchardt xypron.glpk@gmx.de wrote:
On 7/25/20 8:18 PM, Heinrich Schuchardt wrote:
The current default of 0x400 for SYS_MALLOC_F_LEN is too small if any additional drivers marked as DM_FLAG_PRE_RELOC are loaded before relocation.
CONFIG_RSA=y which is needed for UEFI secure boot or for FIT image verification loads the driver mod_exp_sw which has DM_FLAG_PRE_RELOC.
CONFIG_LOG=Y is another setting requiring additional early malloc area, cf. log_init().
When running pine64-lts_defconfig with CONFIG_RSA=y and debug UART enabled we see as output in main U-Boot
alloc_simple() alloc space exhaustedWith this patch the default values of SYS_MALLOC_F_LEN and SPL_SYS_MALLOC_F_LEN on ARCH_SUNXI are raised to 0x2000.
Signed-off-by: Heinrich Schuchardt xypron.glpk@gmx.de Reviewed-by: Jagan Teki jagan@amarulasolutions.com
With current pine64-lts_defconfig we get a warning:
"alloc space exhausted"
So, please, merge the patch.
Reviewed-by: Simon Glass sjg@chromium.org
Best regards
Heinrich
Hello Jagan, hello Maxime,
it is merge window but I could not find this patch in
So apparently since commit 4ab3817ff16a ("clk: fixed-rate: Enable DM_FLAG_PRE_RELOC flag") even the pine64-lts_defconfig is now broken, because more malloc space is needed for U-Boot proper before relocation. This patches fixes is.
So can someone PLEASE merge this patch?
To answer this one lingering question: Apparently the malloc memory for SPL is taken from DRAM (similar to BSS), so we have plenty. The U-Boot proper early malloc is taken from SRAM A1 (why?), but we don't need it for anything else any more, so have enough of it. So the 8KB are safe in both worlds.
Thanks! Andre