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 <[email protected]> > > --- > > 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 <[email protected]> > > #ifdef CONFIG_ARM > /* > _______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
