Re: [PATCH v4] Preserve the EFI System Resource Table for dom0

2022-05-17 Thread Demi Marie Obenour
On Tue, May 17, 2022 at 03:16:55PM +0200, Jan Beulich wrote: > On 07.05.2022 06:26, Demi Marie Obenour wrote: > > This would mean the allocation would need to be unconditional. Right > > now, I avoid the allocation if it is not necessary. > > Hmm, I guess I don't see (taking into account also my

Re: [PATCH v4] Preserve the EFI System Resource Table for dom0

2022-05-17 Thread Jan Beulich
On 07.05.2022 06:26, Demi Marie Obenour wrote: > On Fri, May 06, 2022 at 12:59:05PM +0200, Jan Beulich wrote: >> On 05.05.2022 07:38, Demi Marie Obenour wrote: >>> @@ -1077,6 +1110,35 @@ static void __init efi_exit_boot(EFI_HANDLE >>> ImageHandle, EFI_SYSTEM_TABLE *Syste >>> if ( EFI_ERRO

Re: [PATCH v4] Preserve the EFI System Resource Table for dom0

2022-05-06 Thread Demi Marie Obenour
On Fri, May 06, 2022 at 12:59:05PM +0200, Jan Beulich wrote: > On 05.05.2022 07:38, Demi Marie Obenour wrote: > > @@ -1056,13 +1091,11 @@ static void __init efi_exit_boot(EFI_HANDLE > > ImageHandle, EFI_SYSTEM_TABLE *Syste > > EFI_STATUS status; > > UINTN info_size = 0, map_key; > >

Re: [PATCH v4] Preserve the EFI System Resource Table for dom0

2022-05-06 Thread Jan Beulich
On 06.05.2022 12:59, Jan Beulich wrote: > On 05.05.2022 07:38, Demi Marie Obenour wrote: >> @@ -1077,6 +1110,35 @@ static void __init efi_exit_boot(EFI_HANDLE >> ImageHandle, EFI_SYSTEM_TABLE *Syste >> if ( EFI_ERROR(status) ) >> PrintErrMesg(L"Cannot obtain memory map", stat

Re: [PATCH v4] Preserve the EFI System Resource Table for dom0

2022-05-06 Thread Jan Beulich
On 05.05.2022 07:38, Demi Marie Obenour wrote: > @@ -1056,13 +1091,11 @@ static void __init efi_exit_boot(EFI_HANDLE > ImageHandle, EFI_SYSTEM_TABLE *Syste > EFI_STATUS status; > UINTN info_size = 0, map_key; > bool retry; > -#ifdef CONFIG_EFI_SET_VIRTUAL_ADDRESS_MAP > unsigned

[PATCH v4] Preserve the EFI System Resource Table for dom0

2022-05-04 Thread Demi Marie Obenour
The EFI System Resource Table (ESRT) is necessary for fwupd to identify firmware updates to install. According to the UEFI specification ยง23.4, the ESRT shall be stored in memory of type EfiBootServicesData. However, memory of type EfiBootServicesData is considered general-purpose memory by Xen,