On Sat, Jun 07, 2025 at 01:49:04PM +0200, Heinrich Schuchardt wrote:
> To implement the EFI_SYSTEM_TABLE_POINTER we need 4 MiB aligned
> memory.
>
> On the sandbox LMB uses addresses relative to the start of a page aligned
> RAM buffer allocated with mmap(). This leads to a mismatch of alignment
> between EFI which uses pointers and LMB which uses phys_addr_t.
>
> Ensure that the RAM buffer used for LMB is 4 MiB aligned.
>
> Signed-off-by: Heinrich Schuchardt <heinrich.schucha...@canonical.com>
> ---
> arch/sandbox/cpu/state.c | 6 ++++--
> arch/sandbox/include/asm/state.h | 1 +
> include/configs/sandbox.h | 2 ++
> 3 files changed, 7 insertions(+), 2 deletions(-)
[snip]
> diff --git a/include/configs/sandbox.h b/include/configs/sandbox.h
> index db2ac7f83bb..ee1ad21fd8d 100644
> --- a/include/configs/sandbox.h
> +++ b/include/configs/sandbox.h
> @@ -14,6 +14,8 @@
> #define CFG_SYS_SDRAM_BASE 0
> #define CFG_SYS_SDRAM_SIZE \
> (SB_TO_UL(CONFIG_SANDBOX_RAM_SIZE_MB) << 20)
> +/** define CFG_SYS_SDRAM_ALIGN - Alignment of emulated RAM */
> +#define CFG_SYS_SDRAM_ALIGN 0x400000
>
> #define CFG_SYS_BAUDRATE_TABLE {4800, 9600, 19200, 38400, 57600,\
> 115200}Can we avoid adding something to the CFG_SYS namespace? Since this is sandbox centric is there something under arch/sandbox/include/asm we can drop this in to instead? Thanks. > -- > 2.48.1 > -- Tom
signature.asc
Description: PGP signature
