[PATCH] bloblist: adjust default bloblist size after reloc

[Ilias Apalodimas]

On Mon, 1 Sept 2025 at 20:03, Heinrich Schuchardt
<heinrich.schuchardt at canonical.com> wrote:
>
> If neither CONFIG_BLOBLIST_FIXED NOR CONFIG_BLOBLIST_ALLOC is set,
> currently CONFIG_BLOBLIST_SIZE_RELOC defaults to 0 except if
> * CONFIG_ARM=y && CONFIG_EFI_LOADER=y && GENERATE_ACPI_TABLE=y.
>
> A size of zero never makes sense for a bloblist.
>
> When using QFW we need more than 64 KiB to host the ACPI table.
> In this case CONFIG_BLOBLIST_ALLOC is used.
>
> Set a reasonable default.
>
> Remove the CONFIG_BLOBLIST_SIZE_RELOC in ARM QEMU defconfigs which are
> not compatible with ACPI tables passed from QEMU.
>
> Reported-by: Emil Renner Berthing <emil.renner.berthing at canonical.com>
> Fixes: 6f9b015c138b ("common: Enable BLOBLIST_TABLES on arm")
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
> ---
>  common/Kconfig               | 4 ++--
>  configs/qemu_arm64_defconfig | 1 -
>  configs/qemu_arm_defconfig   | 1 -
>  3 files changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/common/Kconfig b/common/Kconfig
> index a2f653f7e72..7af438a91e7 100644
> --- a/common/Kconfig
> +++ b/common/Kconfig
> @@ -1121,8 +1121,8 @@ config BLOBLIST_SIZE
>
>  config BLOBLIST_SIZE_RELOC
>         hex "Size of bloblist after relocation"
> -       default BLOBLIST_SIZE if BLOBLIST_FIXED || BLOBLIST_ALLOC
> -       default 0x20000 if (ARM && EFI_LOADER && GENERATE_ACPI_TABLE)
> +       default BLOBLIST_SIZE if BLOBLIST_FIXED
> +       default 0x20000
>         help
>           Sets the size of the bloblist in bytes after relocation. Since U-Boot
>           has a lot more memory available then, it is possible to use a larger
> diff --git a/configs/qemu_arm64_defconfig b/configs/qemu_arm64_defconfig
> index 358bb1aeeb9..12ed6b61239 100644
> --- a/configs/qemu_arm64_defconfig
> +++ b/configs/qemu_arm64_defconfig
> @@ -2,7 +2,6 @@ CONFIG_ARM=y
>  CONFIG_KVM_VIRT_INS=y
>  CONFIG_ARCH_QEMU=y
>  CONFIG_SYS_MALLOC_LEN=0x1000000
> -CONFIG_BLOBLIST_SIZE_RELOC=0x2000
>  CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x40200000
>  CONFIG_ENV_SIZE=0x40000
>  CONFIG_ENV_SECT_SIZE=0x40000
> diff --git a/configs/qemu_arm_defconfig b/configs/qemu_arm_defconfig
> index d5890bf87fb..e6d4414cf0d 100644
> --- a/configs/qemu_arm_defconfig
> +++ b/configs/qemu_arm_defconfig
> @@ -3,7 +3,6 @@ CONFIG_KVM_VIRT_INS=y
>  CONFIG_ARM_SMCCC=y
>  CONFIG_ARCH_QEMU=y
>  CONFIG_SYS_MALLOC_LEN=0x1000000
> -CONFIG_BLOBLIST_SIZE_RELOC=0x2000
>  CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x40200000
>  CONFIG_ENV_SIZE=0x40000
>  CONFIG_ENV_SECT_SIZE=0x40000
> --
> 2.50.1
>

Acked-by: Ilias Apalodimas <ilias.apalodimas at linaro.org>