Re: [PATCH v2] Documentation: Fix x86_64 UEFI outdated references to elilo

[Nir Lichtman <nir@lichtman.org>]

On Mon, Jan 13, 2025 at 04:59:00PM +0700, Bagas Sanjaya wrote:
> On Wed, Jan 08, 2025 at 11:35:22AM +0000, Nir Lichtman wrote:
> > diff --git a/Documentation/arch/x86/x86_64/uefi.rst b/Documentation/arch/x86/x86_64/uefi.rst
> > index fbc30c9a071d..3949d83a9915 100644
> > --- a/Documentation/arch/x86/x86_64/uefi.rst
> > +++ b/Documentation/arch/x86/x86_64/uefi.rst
> > @@ -12,14 +12,19 @@ with EFI firmware and specifications are listed below.
> >  
> >  1. UEFI specification:  http://www.uefi.org
> >  
> > -2. Booting Linux kernel on UEFI x86_64 platform requires bootloader
> > -   support. Elilo with x86_64 support can be used.
> > +2. Booting Linux kernel on UEFI x86_64 platform can either be
> > +   done using the <Documentation/admin-guide/efi-stub.rst> or using a separate bootloader.
> >  
> >  3. x86_64 platform with EFI/UEFI firmware.
> >  
[..]
> >  - If some or all EFI runtime services don't work, you can try following
> >    kernel command line parameters to turn off some or all EFI runtime
> >    services.
> 
> Looks good, thanks!
> 
> Reviewed-by: Bagas Sanjaya <bagasdotme@gmail.com>

Thanks for the review! Small question:
I noticed that there is also a script in the kernel (genimage) which enables
automation of building the image, including an image utilizing the EFI stub.
The EFI stub image can be built by enabling the relavent EFI stub config options
and then running "make hdimage", thing is that I couldn't get the script the to
work yet (stuck on an error about missing EDK2/OMVF), but I could look deeper into
this method as well and add it also to this doc, do you think it is also relavent?

Thanks,
Nir