Re: [PATCH 3/3] x86/boot: Rewrite EFI start part in C

2024-09-24 Thread Jan Beulich
On 24.09.2024 17:21, Frediano Ziglio wrote: > On Mon, Sep 23, 2024 at 3:26 PM Jan Beulich wrote: >> > > ... omissis ... > >> > --- a/xen/arch/x86/efi/stub.c > +++ b/xen/arch/x86/efi/stub.c > @@ -17,7 +17,8 @@ > */ > > void __init noreturn efi_multiboot2(EFI_HANDLE Imag

Re: [PATCH 3/3] x86/boot: Rewrite EFI start part in C

2024-09-24 Thread Frediano Ziglio
On Mon, Sep 23, 2024 at 3:26 PM Jan Beulich wrote: > ... omissis ... > > >>> --- a/xen/arch/x86/efi/stub.c > >>> +++ b/xen/arch/x86/efi/stub.c > >>> @@ -17,7 +17,8 @@ > >>> */ > >>> > >>> void __init noreturn efi_multiboot2(EFI_HANDLE ImageHandle, > >>> -EF

Re: [PATCH 3/3] x86/boot: Rewrite EFI start part in C

2024-09-23 Thread Jan Beulich
On 16.09.2024 10:25, Frediano Ziglio wrote: > On Sun, Sep 15, 2024 at 8:00 AM Jan Beulich wrote: >> On 10.09.2024 18:16, Frediano Ziglio wrote: >>> -.Lefi_multiboot2_proto: >>> -/* Zero EFI SystemTable, EFI ImageHandle addresses and cmdline. */ >>> -xor %esi,%esi >>> -x

Re: [PATCH 3/3] x86/boot: Rewrite EFI start part in C

2024-09-16 Thread Frediano Ziglio
On Sun, Sep 15, 2024 at 8:00 AM Jan Beulich wrote: > > On 10.09.2024 18:16, Frediano Ziglio wrote: > > No need to have it coded in assembly. > > As to the title: It's the EFI/MB2 case you re-write. That wants reflecting > there, as the "normal" EFI start part is all C already anyway. I also > thin

Re: [PATCH 3/3] x86/boot: Rewrite EFI start part in C

2024-09-15 Thread Jan Beulich
On 10.09.2024 18:16, Frediano Ziglio wrote: > No need to have it coded in assembly. As to the title: It's the EFI/MB2 case you re-write. That wants reflecting there, as the "normal" EFI start part is all C already anyway. I also think you mean "partly"? > @@ -255,34 +246,29 @@ __efi64_mb2_start:

[PATCH 3/3] x86/boot: Rewrite EFI start part in C

2024-09-10 Thread Frediano Ziglio
No need to have it coded in assembly. Signed-off-by: Frediano Ziglio --- xen/arch/x86/boot/head.S | 131 ++ xen/arch/x86/efi/Makefile | 1 + xen/arch/x86/efi/parse-mbi2.c | 54 ++ xen/arch/x86/efi/stub.c | 3 +- 4 files changed, 80