[U-Boot] [PATCH v2 2/6] efi_loader: Increase number of configuration tables to 16

[Heinrich Schuchardt]

On 06/27/2018 12:16 PM, Bin Meng wrote:
> At present the number of configuration tables is set to 2. By
> looking at which tables the Linux EFI stub or iPXE can process,
> it looks 16 is a reasonable number.
> 
> Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
> 
> ---
> 
> Changes in v2:
> - new patch "increase number of configuration tables to 16"
> 
>  lib/efi_loader/efi_boottime.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/efi_loader/efi_boottime.c b/lib/efi_loader/efi_boottime.c
> index 50d3115..261d66d 100644
> --- a/lib/efi_loader/efi_boottime.c
> +++ b/lib/efi_loader/efi_boottime.c
> @@ -43,7 +43,7 @@ static bool efi_is_direct_boot = true;
>   * In most cases we want to pass an FDT to the payload, so reserve one slot of
>   * config table space for it. The pointer gets populated by do_bootefi_exec().
>   */
> -static struct efi_configuration_table __efi_runtime_data efi_conf_table[2];
> +static struct efi_configuration_table __efi_runtime_data efi_conf_table[16];

Alex suggested to make this a pointer to be allocated at runtime. But
this will involve further changes like updating the crc32 in the header.

I prefer to change this value to 16 in this patch series to cut down the
size of your patch series. Later we can provide a separate patch series
that allocates the space from pool, updates crc32, and provides a test
case for the table management including a test if the correct event
group is being notified.

Reviewed-by: Heinrich Schuchardt <xypron.glpk at gmx.de>

>  
>  #ifdef CONFIG_ARM
>  /*
>