According to the UEFI spec, the UEFI OS Loader (aka the stub) should
transition from UEFI to the OS by getting the current memory map from UEFI,
then calling ExitBootServices. The spec states that ExitBootServices may
return EFI_INVALID_PARAMETER if the memory map reference provided by the stub
is not current, ie UEFI handled some event prior to ExitBootServices which
modified the map. The spec states that to handle this scenario, the stub shall
get the updated map, and invoke ExitBootServices again. The spec also states
that once ExitBootServices is invoked, even if it returns error, the only
APIs the stub is allowed to invoke is GetMemoryMap and ExitBootServices.
The EFI_INVALID_PARAMETER scenario has been seen in the wild previously in
x86 but the fix - d3768d885c6c ("x86, efi: retry ExitBootServices() on failure")
still violates the spec. The FDT code does not handle this scenario, but
instances of it are now observed.
This patch series aims to provide a spec complaint solution that can be reused
in all stubs, thus preventing each arch or variant from reinventing the wheel
and likely getting it wrong.
[V5]
-Vertically align struct definition and initializations per Ingo Molnar
-Remove struct typedef per Ingo Molnar
-Clarify commit texts per Ingo Molnar
-Clean up comment placement and format per Ingo Molnar
-Misc spelling/phrasing cleanups per Ingo Molnar
-Add inline refernce to relevant UEFI spec section per Timur Tabi
[V4]
-Fix x86 boot error reported by Matt Fleming
-Update change logs to indicate real world occurances per Matt Fleming
-Make the callback function non-optional per Matt Fleming
-Reduce parameter list of efi_get_memory_map() per Matt Fleming
-Reduce parameter list of efi_exit_boot_services per Matt Fleming
-Define callback function signature with a typedef per Matt Fleming
[V3]
-Remove old extra headroom allocation per Matt Fleming
-Create and use headroom check wrapper per Matt Fleming
[V2]
-Define EFI_MMAP_NR_SLACK_SLOTS per Mark Rutland
-Use desc_size as firmware may exceed the defined struct size per Ard Biesheuvel
[V1]
-Allocate headspace on the memory map buffer for reuse per Ard Biesheuvel
-Create a shared helper address the issue universally per Matt Fleming
Jeffrey Hugo (4):
efi/libstub: Allocate headspace in efi_get_memory_map()
efi/libstub: Introduce ExitBootServices helper
efi/libstub: Use efi_exit_boot_services() in FDT
x86/efi: Use efi_exit_boot_services()
arch/x86/boot/compressed/eboot.c | 132 +++++++++++++------------
drivers/firmware/efi/libstub/efi-stub-helper.c | 129 ++++++++++++++++++++++--
drivers/firmware/efi/libstub/fdt.c | 52 +++++++---
drivers/firmware/efi/libstub/random.c | 3 +-
include/linux/efi.h | 22 ++++-
5 files changed, 249 insertions(+), 89 deletions(-)
--
Qualcomm Datacenter Technologies as an affiliate of Qualcomm Technologies, Inc.
Qualcomm Technologies, Inc. is a member of the
Code Aurora Forum, a Linux Foundation Collaborative Project.
--
To unsubscribe from this list: send the line "unsubscribe linux-efi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
