Hi Ilias, I will add both TPM_EVLOG and TPM_CRB_BASE.
Regards, Raymond On Wed, 6 Dec 2023 at 05:54, Ilias Apalodimas <ilias.apalodi...@linaro.org> wrote: > On Mon, 4 Dec 2023 at 20:55, Raymond Mao <raymond....@linaro.org> wrote: > > > > Hi Ilias, > > > > What is the difference between the new added XFERLIST_EVLOG and the > existing BLOBLISTT_TPM2_TCG_LOG and BLOBLISTT_TCPA_LOG in U-Boot? > > > > I am not really sure what the existing options are supposed to mean. > Having discrete options for v1 and v2 makes little sense since the > EvenLog format already contains that info. > > The newly added options is supposed to > - Hand you over an EventLog from a previous stage boot loader, so you > can continue extending that instead of creating a new one. We do that > already parsing for tpm_event_log_addr, which is what TF-A fills in. > In the future we need to add the bloblist option. > - Tell you whether you need to replay it or not. > > Thanks > /Ilias > > > Thanks and regards, > > Raymond > > > > On Mon, 4 Dec 2023 at 12:52, Ilias Apalodimas < > ilias.apalodi...@linaro.org> wrote: > >> > >> On Mon, 4 Dec 2023 at 18:25, Raymond Mao <raymond....@linaro.org> > wrote: > >> > > >> > Hi Ilias, > >> > > >> > BLOBLISTT_AREA_ARM is now holding the ones we already defined in the > FW Handoff spec for TF-A project only. > >> > The TPM eventlog related ones are undefined in the spec yet, they > stay in the group BLOBLISTT_AREA_FIRMWARE. > >> > >> We did define them past 0.9 [0]. So I think we should add them > regardless. > >> They are part of the main doc now [1] > >> > >> [0] https://github.com/FirmwareHandoff/firmware_handoff/pull/16/files > >> [1] > https://github.com/FirmwareHandoff/firmware_handoff/blob/main/source/transfer_list.rst#tpm-event-log-table-entry-layout-xferlist_evlog > >> > >> Regards > >> /Ilias > >> > >> > > >> > > /* BLOBLISTT_AREA_FIRMWARE */ > >> > >- { BLOBLISTT_ACPI_GNVS, "ACPI GNVS" }, > >> > >- { BLOBLISTT_INTEL_VBT, "Intel Video-BIOS table" }, > >> > > { BLOBLISTT_TPM2_TCG_LOG, "TPM v2 log space" }, > >> > > { BLOBLISTT_TCPA_LOG, "TPM log space" }, > >> > >- { BLOBLISTT_ACPI_TABLES, "ACPI tables for x86" }, > >> > >+ { BLOBLISTT_ACPI_GNVS, "ACPI GNVS" }, > >> > > >> > Thanks and regards, > >> > Raymond > >> > > >> > On Mon, 4 Dec 2023 at 03:25, Ilias Apalodimas < > ilias.apalodi...@linaro.org> wrote: > >> >> > >> >> Hi all, > >> >> > >> >> [...] > >> >> > >> >> > common/bloblist.c | 16 +++++++++--- > >> >> > include/bloblist.h | 65 > ++++++++++++++++++++++++---------------------- > >> >> > test/bloblist.c | 4 +-- > >> >> > 3 files changed, 48 insertions(+), 37 deletions(-) > >> >> > > >> >> > diff --git a/common/bloblist.c b/common/bloblist.c > >> >> > index a22f6c12b0..349ceddea5 100644 > >> >> > --- a/common/bloblist.c > >> >> > +++ b/common/bloblist.c > >> >> > @@ -36,16 +36,24 @@ static struct tag_name { > >> >> > enum bloblist_tag_t tag; > >> >> > const char *name; > >> >> > } tag_name[] = { > >> >> > - { BLOBLISTT_NONE, "(none)" }, > >> >> > + { BLOBLISTT_VOID, "(void)" }, > >> >> > > >> >> > /* BLOBLISTT_AREA_FIRMWARE_TOP */ > >> >> > + { BLOBLISTT_CONTROL_FDT, "Control FDT" }, > >> >> > + { BLOBLISTT_HOB_BLOCK, "HOB block" }, > >> >> > + { BLOBLISTT_HOB_LIST, "HOB list" }, > >> >> > + { BLOBLISTT_ACPI_TABLES, "ACPI tables for x86" }, > >> >> > > >> >> > /* BLOBLISTT_AREA_FIRMWARE */ > >> >> > - { BLOBLISTT_ACPI_GNVS, "ACPI GNVS" }, > >> >> > - { BLOBLISTT_INTEL_VBT, "Intel Video-BIOS table" }, > >> >> > { BLOBLISTT_TPM2_TCG_LOG, "TPM v2 log space" }, > >> >> > { BLOBLISTT_TCPA_LOG, "TPM log space" }, > >> >> > - { BLOBLISTT_ACPI_TABLES, "ACPI tables for x86" }, > >> >> > >> >> There are some TPM Eventlog related entries that are missing here. > >> >> Can we add them? > >> >> > >> >> > + { BLOBLISTT_ACPI_GNVS, "ACPI GNVS" }, > >> >> > + > >> >> > + /* BLOBLISTT_AREA_ARM */ > >> >> > + { BLOBLISTT_OPTEE_PAGABLE_PART, "OP-TEE pagable part" }, > >> >> > + > >> >> > + /* BLOBLISTT_AREA_OTHER */ > >> >> > + { BLOBLISTT_INTEL_VBT, "Intel Video-BIOS table" }, > >> >> [...] > >> >> > >> >> Thanks > >> >> /Ilias >