[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [PATCH] efi: discover ESRT table on Xen PV too
On Mon, Aug 17, 2020 at 10:16:07AM +0200, Ard Biesheuvel wrote: > > @@ -331,7 +333,8 @@ void __init efi_esrt_init(void) > > > > end = esrt_data + size; > > pr_info("Reserving ESRT space from %pa to %pa.\n", &esrt_data, > > &end); > > - if (md.type == EFI_BOOT_SERVICES_DATA) > > + > > + if (efi_enabled(EFI_MEMMAP) && md.type == EFI_BOOT_SERVICES_DATA) > > efi_mem_reserve(esrt_data, esrt_data_size); > > > > This does not look correct to me. Why doesn't the region need to be > reserved on a Xen boot? The OS may overwrite it otherwise. In case of Xen, it is Xen responsibility to do that. Otherwise even if dom0 would not use it, Xen could allocate that physical memory to another guest. -- Best Regards, Marek Marczykowski-Górecki Invisible Things Lab A: Because it messes up the order in which people normally read text. Q: Why is top-posting such a bad thing? Attachment:
signature.asc
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |