Re: [v2][PATCH 1/3] efi_loader: Add check for event log passed from firmware

2021-11-24 Thread Masahisa Kojima
Hi Ruchika, On Wed, 24 Nov 2021 at 18:12, Ruchika Gupta wrote: > > Hi Kojima-san, > > On Wed, 24 Nov 2021 at 13:08, Masahisa Kojima > wrote: >> >> Hi Ruchika, Ilias, >> >> On Tue, 23 Nov 2021 at 20:53, Ruchika Gupta wrote: >> > >> > Platforms may have support to measure their initial firmware

Re: [v2][PATCH 1/3] efi_loader: Add check for event log passed from firmware

2021-11-24 Thread Ruchika Gupta
Hi Kojima-san, On Wed, 24 Nov 2021 at 13:08, Masahisa Kojima wrote: > Hi Ruchika, Ilias, > > On Tue, 23 Nov 2021 at 20:53, Ruchika Gupta > wrote: > > > > Platforms may have support to measure their initial firmware components > > and pass the event log to u-boot. The event log address can be pa

Re: [v2][PATCH 1/3] efi_loader: Add check for event log passed from firmware

2021-11-24 Thread Ruchika Gupta
Hi Ilias, On Wed, 24 Nov 2021 at 12:34, Ilias Apalodimas wrote: > Hi Ruchika, > > + > > [...] > > > + ret = platform_get_eventlog(dev, &base, &sz); > > + if (ret == EFI_SUCCESS) { > > Can we invert the logic here? > if (ret != EFI_SUCCESS) > return ret; > > etc... > Change posted

Re: [v2][PATCH 1/3] efi_loader: Add check for event log passed from firmware

2021-11-23 Thread Masahisa Kojima
Hi Ruchika, Ilias, On Tue, 23 Nov 2021 at 20:53, Ruchika Gupta wrote: > > Platforms may have support to measure their initial firmware components > and pass the event log to u-boot. The event log address can be passed > in property tpm_event_log_addr and tpm_event_log_size of the tpm node. > Plat

Re: [v2][PATCH 1/3] efi_loader: Add check for event log passed from firmware

2021-11-23 Thread Ilias Apalodimas
Hi Ruchika, > + [...] > + ret = platform_get_eventlog(dev, &base, &sz); > + if (ret == EFI_SUCCESS) { Can we invert the logic here? if (ret != EFI_SUCCESS) return ret; etc... > + void *buffer = (void *)base; > + > + if (sz > TPM2_EVENT_LOG_SIZE) > +

[v2][PATCH 1/3] efi_loader: Add check for event log passed from firmware

2021-11-23 Thread Ruchika Gupta
Platforms may have support to measure their initial firmware components and pass the event log to u-boot. The event log address can be passed in property tpm_event_log_addr and tpm_event_log_size of the tpm node. Platforms may choose their own specific mechanism to do so. A weak function is added t

[v2][PATCH 1/3] efi_loader: Add check for event log passed from firmware

2021-11-23 Thread Ruchika Gupta
Platforms may have support to measure their initial firmware components and pass the event log to u-boot. The event log address can be passed in property tpm_event_log_addr and tpm_event_log_size of the tpm node. Platforms may choose their own specific mechanism to do so. A weak function is added t