mbox series

[kvm-unit-tests,v4,0/2] UEFI Improvements

Message ID 20240329131522.806983-1-papaluri@amd.com (mailing list archive)
Headers show
Series UEFI Improvements | expand

Message

Pavan Kumar Paluri March 29, 2024, 1:15 p.m. UTC
Patch-1: Introduces a fix for x86 arch which is ACPI-based to not get
	 into the path of fdt.

Patch-2: KUT UEFI-based guest may sometimes fail to exit boot services
	 due to a possible memory map update that might have taken place
	 between efi_get_memory_map() call and efi_exit_boot_services()
	call. As per UEFI specification (2.10), we need to try and keep
	updating the memory map as long as we get Invalid key failure.

=========
Changelog
=========
v3 -> v4:
    * Dropped patches 3 & 4 from the series as they are not relevant to 
      UEFI improvements introduced in this patchset. This would aid in 
      easier review and upstreaming.
    * Addressed feedback (Andrew)
    * Included R-b tag from Andrew Jones.

v2 -> v3:
    * Included R-b tag from Andrew for Patch-1.
    * Updated patch-2 to not leak memory map information during 
      re-trials to efi_get_memory_map().

v1 -> v2:
    * Incorporated feedback (Andrew, Mike, Tom)
    * Updated patch-2 to keep trying to update memory map and calls to 
      exit boot services as long as there is a failure.
    * Split Page allocation and GHCB page attributes patch into two 
      patches.
 
Pavan Kumar Paluri (2):
  x86 EFI: Bypass call to fdt_check_header()
  x86/efi: Retry call to efi exit boot services

 lib/efi.c | 39 +++++++++++++++++++++++++--------------
 1 file changed, 25 insertions(+), 14 deletions(-)

Comments

Andrew Jones April 3, 2024, 11:27 a.m. UTC | #1
On Fri, Mar 29, 2024 at 08:15:20AM -0500, Pavan Kumar Paluri wrote:
> Patch-1: Introduces a fix for x86 arch which is ACPI-based to not get
> 	 into the path of fdt.
> 
> Patch-2: KUT UEFI-based guest may sometimes fail to exit boot services
> 	 due to a possible memory map update that might have taken place
> 	 between efi_get_memory_map() call and efi_exit_boot_services()
> 	call. As per UEFI specification (2.10), we need to try and keep
> 	updating the memory map as long as we get Invalid key failure.
> 
> =========
> Changelog
> =========
> v3 -> v4:
>     * Dropped patches 3 & 4 from the series as they are not relevant to 
>       UEFI improvements introduced in this patchset. This would aid in 
>       easier review and upstreaming.
>     * Addressed feedback (Andrew)
>     * Included R-b tag from Andrew Jones.
> 
> v2 -> v3:
>     * Included R-b tag from Andrew for Patch-1.
>     * Updated patch-2 to not leak memory map information during 
>       re-trials to efi_get_memory_map().
> 
> v1 -> v2:
>     * Incorporated feedback (Andrew, Mike, Tom)
>     * Updated patch-2 to keep trying to update memory map and calls to 
>       exit boot services as long as there is a failure.
>     * Split Page allocation and GHCB page attributes patch into two 
>       patches.
>  
> Pavan Kumar Paluri (2):
>   x86 EFI: Bypass call to fdt_check_header()
>   x86/efi: Retry call to efi exit boot services
> 
>  lib/efi.c | 39 +++++++++++++++++++++++++--------------
>  1 file changed, 25 insertions(+), 14 deletions(-)
> 
> -- 
> 2.34.1
>

Merged.

Thanks,
drew