mbox series

[0/2] efi/x86: Move runtime map sysfs code into arch/x86

Message ID 20221107083134.234918-1-ardb@kernel.org
Headers show
Series efi/x86: Move runtime map sysfs code into arch/x86 | expand

Message

Ard Biesheuvel Nov. 7, 2022, 8:31 a.m. UTC
The EFI runtime map code is only used on x86, so let's move it out of
the generic code and into arch/x86. While at it, clarify the Kconfig
description so that it gets enabled automatically when required (i.e.,
when kexec is enabled), and can only be disabled when CONFIG_EXPERT=y

Ard Biesheuvel (2):
  efi: runtime-maps: Clarify purpose and enable by default for kexec
  efi: x86: Move EFI runtime map sysfs code to arch/x86

 arch/x86/Kconfig                                          | 11 ++++++++
 arch/x86/include/asm/efi.h                                | 22 +++++++++++++++
 arch/x86/platform/efi/Makefile                            |  1 +
 {drivers/firmware => arch/x86/platform}/efi/runtime-map.c |  6 ++---
 drivers/firmware/efi/Kconfig                              | 11 --------
 drivers/firmware/efi/Makefile                             |  1 -
 drivers/firmware/efi/efi.c                                |  5 +---
 include/linux/efi.h                                       | 28 --------------------
 8 files changed, 38 insertions(+), 47 deletions(-)
 rename {drivers/firmware => arch/x86/platform}/efi/runtime-map.c (97%)

Comments

Dave Young Nov. 8, 2022, 3:46 a.m. UTC | #1
Hi Ard,

On Mon, 7 Nov 2022 at 16:31, Ard Biesheuvel <ardb@kernel.org> wrote:
>
> The EFI runtime map code is only used on x86, so let's move it out of
> the generic code and into arch/x86. While at it, clarify the Kconfig
> description so that it gets enabled automatically when required (i.e.,
> when kexec is enabled), and can only be disabled when CONFIG_EXPERT=y
>
> Ard Biesheuvel (2):
>   efi: runtime-maps: Clarify purpose and enable by default for kexec
>   efi: x86: Move EFI runtime map sysfs code to arch/x86

For the series:

Reviewed-by: Dave Young <dyoung@redhat.com>

>
>  arch/x86/Kconfig                                          | 11 ++++++++
>  arch/x86/include/asm/efi.h                                | 22 +++++++++++++++
>  arch/x86/platform/efi/Makefile                            |  1 +
>  {drivers/firmware => arch/x86/platform}/efi/runtime-map.c |  6 ++---
>  drivers/firmware/efi/Kconfig                              | 11 --------
>  drivers/firmware/efi/Makefile                             |  1 -
>  drivers/firmware/efi/efi.c                                |  5 +---
>  include/linux/efi.h                                       | 28 --------------------
>  8 files changed, 38 insertions(+), 47 deletions(-)
>  rename {drivers/firmware => arch/x86/platform}/efi/runtime-map.c (97%)
>
> --
> 2.35.1
>
Ard Biesheuvel Nov. 9, 2022, 8:56 a.m. UTC | #2
On Tue, 8 Nov 2022 at 04:47, Dave Young <dyoung@redhat.com> wrote:
>
> Hi Ard,
>
> On Mon, 7 Nov 2022 at 16:31, Ard Biesheuvel <ardb@kernel.org> wrote:
> >
> > The EFI runtime map code is only used on x86, so let's move it out of
> > the generic code and into arch/x86. While at it, clarify the Kconfig
> > description so that it gets enabled automatically when required (i.e.,
> > when kexec is enabled), and can only be disabled when CONFIG_EXPERT=y
> >
> > Ard Biesheuvel (2):
> >   efi: runtime-maps: Clarify purpose and enable by default for kexec
> >   efi: x86: Move EFI runtime map sysfs code to arch/x86
>
> For the series:
>
> Reviewed-by: Dave Young <dyoung@redhat.com>
>

Thanks Dave, I've queued these up now.

> >
> >  arch/x86/Kconfig                                          | 11 ++++++++
> >  arch/x86/include/asm/efi.h                                | 22 +++++++++++++++
> >  arch/x86/platform/efi/Makefile                            |  1 +
> >  {drivers/firmware => arch/x86/platform}/efi/runtime-map.c |  6 ++---
> >  drivers/firmware/efi/Kconfig                              | 11 --------
> >  drivers/firmware/efi/Makefile                             |  1 -
> >  drivers/firmware/efi/efi.c                                |  5 +---
> >  include/linux/efi.h                                       | 28 --------------------
> >  8 files changed, 38 insertions(+), 47 deletions(-)
> >  rename {drivers/firmware => arch/x86/platform}/efi/runtime-map.c (97%)
> >
> > --
> > 2.35.1
> >
>