Hi Masahisa, On Wed, 7 Jul 2021 at 20:21, Masahisa Kojima <masahisa.koj...@linaro.org> wrote: > > On Wed, 7 Jul 2021 at 22:47, Heinrich Schuchardt <xypron.g...@gmx.de> wrote: > > > > > > > > On 7/7/21 3:36 PM, Masahisa Kojima wrote: > > > This is a preperation to add eventlog support > > > described in TCG PC Client PFP spec. > > > > > > Signed-off-by: Masahisa Kojima <masahisa.koj...@linaro.org> > > > --- > > > lib/efi_loader/Kconfig | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/lib/efi_loader/Kconfig b/lib/efi_loader/Kconfig > > > index b2ab48a048..a87bf3cc98 100644 > > > --- a/lib/efi_loader/Kconfig > > > +++ b/lib/efi_loader/Kconfig > > > @@ -327,7 +327,7 @@ config EFI_TCG2_PROTOCOL > > > config EFI_TCG2_PROTOCOL_EVENTLOG_SIZE > > > int "EFI_TCG2_PROTOCOL EventLog size" > > > depends on EFI_TCG2_PROTOCOL > > > - default 4096 > > > + default 16384 > > > > I found this text in EDK II: > > > > Minimum length(in bytes) of the system preboot TCG event log area(LAML) > > ----------------------------------------------------------------------- > > > > For PC Client Implementation spec up to and including 1.2 the minimum > > log size is 64KB. (SecurityPkg/SecurityPkg.dec) > > Thank you for your feedback. > I have not checked this. > TCG spec also says "The Log Area Minimum Length for the TCG event log > MUST be at least 64KB." in ACPI chapter. > I will update to set 64KB as default. >
Is this the same as the BLOBLISTT_TPM2_TCG_LOG thing? If so, can we put this in the bloblist? We want to avoid adding code in EFI which is in U-Boot. - Simon > Thanks, > Masahisa Kojima > > > > > Why should ours be smaller? > > > > Best regards > > > > Heinrich > > > > > help > > > Define the size of the EventLog for EFI_TCG2_PROTOCOL. Note > > > that > > > this is going to be allocated twice. One for the eventlog > > > it self > > >