[PATCH] efi_loader: Do not enable by default on armv7
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.
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
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.
Best regards
Heinrich
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
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
On Sun, Jun 27, 2021 at 11:08:04PM +0200, Mark Kettenis wrote:
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.
OK, thanks guys. I missed the Fedora link and I thought *BSD was still using a custom loader on v7.
On Sun, Jun 27, 2021 at 10:22 PM Tom Rini trini@konsulko.com wrote:
On Sun, Jun 27, 2021 at 11:08:04PM +0200, Mark Kettenis wrote:
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.
OK, thanks guys. I missed the Fedora link and I thought *BSD was still using a custom loader on v7.
Yes, so Fedora uses UEFI by default for across all arm devices now. Linaro also uses it for a bunch of their reference platforms but Ilias will be able to better provide details there.
Peter
On Mon, Jun 28, 2021 at 08:45:13AM +0100, Peter Robinson wrote:
On Sun, Jun 27, 2021 at 10:22 PM Tom Rini trini@konsulko.com wrote:
On Sun, Jun 27, 2021 at 11:08:04PM +0200, Mark Kettenis wrote:
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.
OK, thanks guys. I missed the Fedora link and I thought *BSD was still using a custom loader on v7.
Yes, so Fedora uses UEFI by default for across all arm devices now. Linaro also uses it for a bunch of their reference platforms but Ilias will be able to better provide details there.
Yes we are also booting every platform with EFI. And tbh it's not only a matter of distros that already use it. Arm itself has the SystemReady specification [1] which mandates the use of EFI in most of it's flavors (SR, ES and IR). Keeping in mind U-Boot's EFI support is very close to be 100% compliant, I think it's beneficial overall to have pre-canned board configs that support that.
[1] https://developer.arm.com/architectures/system-architectures/arm-systemready
Cheers /Ilias
On 6/27/21 11:03 PM, Heinrich Schuchardt wrote:
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.
OpenBSD and FreeBSD rely on UEFI for booting.
Best regards
Heinrich
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
participants (5)
-
Heinrich Schuchardt -
Ilias Apalodimas -
Mark Kettenis -
Peter Robinson -
Tom Rini