smallsolar/u-boot
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

sandbox: Avoid mon_len being larger than available RAM

Browse Source 
With the shared library the size can be quite large. Normally sandbox
pretends that its code is mapped into emulated RAM, even though it
actually isn't.

This is fine in most cases, but when mon_len exceeds the RAM size the
reservation top starts off at a negative address (i.e. very near the top
of the address space), which immediately segfaults.

Add a special case for this, for sandbox only.

We might consider dropping this mapping altogether, as is down with
RISC-V, but that is left for further discussion.

With this, the test app boots to a prompt.

Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Simon Glass
2025-09-03 12:34:03 -06:00
parent 80bda07c54
commit 88b810e241
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
common/board_f.c
View File

@@ -460,7 +460,11 @@ static int reserve_uboot(void)
* reserve memory for U-Boot code, data & bss
* round down to next 4 kB limit
*/
gd->relocaddr -= gd->mon_len;
if (IS_ENABLED(CONFIG_SANDBOX) && gd->mon_len > gd->relocaddr)
log_debug("Cannot reserve space for U-Boot\n");
else
gd->relocaddr -= gd->mon_len;
gd->relocaddr &= ~(4096 - 1);
#if defined(CONFIG_E500) || defined(CONFIG_MIPS)
/* round down to next 64 kB limit so that IVPR stays aligned */