Re: [PATCH] efi_loader: Do not enable by default on armv7

[Mark Kettenis <mark.kettenis@xs4all.nl>]

> From: Heinrich Schuchardt <xypron.glpk@gmx.de>
> Date: Sun, 27 Jun 2021 23:03:58 +0200
> 
> On 6/27/21 9:44 PM, Tom Rini wrote:
> > We have had EFI loader support enabled by default on armv7 platforms for
> > a long time now.  However, there has been little uptake in the wider
> > community to use EFI loader support as the step to start a generic OS
> > distribution, on this architecture family and the focus has been on
> > armv8 instead.  Disable this by default for armv7 only.
> 
> NAK
> 
> Please, read https://fedoraproject.org/wiki/Changes/uEFIforARMv7.

+1

OpenBSD/armv7 relies on EFI loader support as well.

Cheers.

> > Cc: Alexander Graf <agraf@csgraf.de>
> > Cc: Heinrich Schuchardt <xypron.glpk@gmx.de>
> > Cc: Ilias Apalodimas <ilias.apalodimas@linaro.org>
> > Cc: Mark Kettenis <kettenis@openbsd.org>
> > Cc: Masahisa Kojima <masahisa.kojima@linaro.org>
> > Cc: Peter Robinson <pbrobinson@gmail.com>
> > Cc: Sughosh Ganu <sughosh.ganu@linaro.org>
> > Cc: Vagrant Cascadian <vagrant@debian.org>
> > Signed-off-by: Tom Rini <trini@konsulko.com>
> > ---
> > I'm very much looking for feedback here that this is the wrong approach
> > and that I've missed the off the shelf distributions that make use of
> > this.
> > ---
> >   lib/efi_loader/Kconfig | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/lib/efi_loader/Kconfig b/lib/efi_loader/Kconfig
> > index 6242caceb7f9..8a22fe9951a3 100644
> > --- a/lib/efi_loader/Kconfig
> > +++ b/lib/efi_loader/Kconfig
> > @@ -10,7 +10,7 @@ config EFI_LOADER
> >   	depends on !EFI_STUB || !X86_64 || EFI_STUB_64BIT
> >   	# We need EFI_STUB_32BIT to be set on x86_32 with EFI_STUB
> >   	depends on !EFI_STUB || !X86 || X86_64 || EFI_STUB_32BIT
> > -	default y if !ARM || SYS_CPU = armv7 || SYS_CPU = armv8
> > +	default y if !ARM || SYS_CPU = armv8
> >   	select LIB_UUID
> >   	select HAVE_BLOCK_DEVICE
> >   	select REGEX
> >
> 
>