On 09.12.2022 01:52, Demi Marie Obenour wrote:
> This was missed in the initial patchset.
> 
> Signed-off-by: Demi Marie Obenour <d...@invisiblethingslab.com>

It looks as if, besides ...

> --- a/xen/arch/x86/efi/efi-boot.h
> +++ b/xen/arch/x86/efi/efi-boot.h
> @@ -818,6 +818,8 @@ void __init efi_multiboot2(EFI_HANDLE ImageHandle, 
> EFI_SYSTEM_TABLE *SystemTable
>      if ( gop )
>          efi_set_gop_mode(gop, gop_mode);
>  
> +    efi_relocate_esrt(SystemTable);
> +
>      efi_exit_boot(ImageHandle, SystemTable);
>  }
>  

... this core piece, the rest is merely pure code movement, with the goal
of avoiding a forward declaration. If such is the case, please say so in
your description. I'll add a sentence to this effect when committing, so
the remark if for future patches. Provided of course it is true (i.e. I'm
not overlooking any tiny change) - please confirm.

Reviewed-by: Jan Beulich <jbeul...@suse.com>

Jan

Reply via email to