2023-01-19 20:45:06

by Demi Marie Obenour

[permalink] [raw]
Subject: [PATCH v3 0/5] efi: Support ESRT under Xen

This patch series fixes handling of EFI tables when running under Xen.
These fixes allow the ESRT to be loaded when running paravirtualized in
dom0, making the use of EFI capsule updates possible.

Demi Marie Obenour (5):
efi: memmap: Disregard bogus entries instead of returning them
efi: xen: Implement memory descriptor lookup based on hypercall
efi: Apply allowlist to EFI configuration tables when running under
Xen
efi: Actually enable the ESRT under Xen
efi: Warn if trying to reserve memory under Xen

drivers/firmware/efi/efi.c | 22 ++++++++++++-
drivers/firmware/efi/esrt.c | 15 +++------
drivers/xen/efi.c | 61 +++++++++++++++++++++++++++++++++++++
include/linux/efi.h | 3 ++
4 files changed, 90 insertions(+), 11 deletions(-)

--
Sincerely,
Demi Marie Obenour (she/her/hers)
Invisible Things Lab


2023-01-23 07:30:30

by Ard Biesheuvel

[permalink] [raw]
Subject: Re: [PATCH v3 0/5] efi: Support ESRT under Xen

On Thu, 19 Jan 2023 at 20:04, Demi Marie Obenour
<[email protected]> wrote:
>
> This patch series fixes handling of EFI tables when running under Xen.
> These fixes allow the ESRT to be loaded when running paravirtualized in
> dom0, making the use of EFI capsule updates possible.
>
> Demi Marie Obenour (5):
> efi: memmap: Disregard bogus entries instead of returning them
> efi: xen: Implement memory descriptor lookup based on hypercall
> efi: Apply allowlist to EFI configuration tables when running under
> Xen
> efi: Actually enable the ESRT under Xen
> efi: Warn if trying to reserve memory under Xen
>

I have given these a spin on a system with a dodgy ESRT (the region in
question is not covered by the memory map at all), and things are
exactly as broken before, which is good.

I have queued these up in efi/next now, they should appear in -next tomorrow.


> drivers/firmware/efi/efi.c | 22 ++++++++++++-
> drivers/firmware/efi/esrt.c | 15 +++------
> drivers/xen/efi.c | 61 +++++++++++++++++++++++++++++++++++++
> include/linux/efi.h | 3 ++
> 4 files changed, 90 insertions(+), 11 deletions(-)
>
> --
> Sincerely,
> Demi Marie Obenour (she/her/hers)
> Invisible Things Lab