mbox series

[v6,0/4] Option to build big-endian with ELFv2 ABI

Message ID 20221128041539.1742489-1-npiggin@gmail.com (mailing list archive)
Headers show
Series Option to build big-endian with ELFv2 ABI | expand

Message

Nicholas Piggin Nov. 28, 2022, 4:15 a.m. UTC 
This is hopefully the final attempt. Luis was happy for the module
patch to go via the powerpc tree, so I've put the the ELFv2 for big
endian build patches into the series. Hopefully we can deprecate
the ELFv1 ABI 

Since v5, I cleaned up patch 2 as per Christophe's review. And patch
4 I removed the EXPERT depends so it's easier to test. It's marked as
experimental, but we should soon make it default and try to deprecate
the v1 ABI so we can eventually remove it.

Thanks,
Nick

Nicholas Piggin (4):
  module: add module_elf_check_arch for module-specific checks
  powerpc/64: Add module check for ELF ABI version
  powerpc/64: Add big-endian ELFv2 flavour to crypto VMX asm generation
  powerpc/64: Option to build big-endian with ELFv2 ABI

 arch/powerpc/Kconfig                   | 21 +++++++++++++++++++++
 arch/powerpc/kernel/module_64.c        | 10 ++++++++++
 arch/powerpc/platforms/Kconfig.cputype |  4 ++--
 drivers/crypto/vmx/Makefile            | 12 +++++++++++-
 drivers/crypto/vmx/ppc-xlate.pl        | 10 ++++++----
 include/linux/moduleloader.h           |  3 +++
 kernel/module/main.c                   | 10 ++++++++++
 7 files changed, 63 insertions(+), 7 deletions(-)

Comments

Joel Stanley Nov. 29, 2022, 3:02 a.m. UTC | #1 
On Mon, 28 Nov 2022 at 04:16, Nicholas Piggin <npiggin@gmail.com> wrote:
>
> This is hopefully the final attempt. Luis was happy for the module
> patch to go via the powerpc tree, so I've put the the ELFv2 for big
> endian build patches into the series. Hopefully we can deprecate
> the ELFv1 ABI
>
> Since v5, I cleaned up patch 2 as per Christophe's review. And patch
> 4 I removed the EXPERT depends so it's easier to test. It's marked as
> experimental, but we should soon make it default and try to deprecate
> the v1 ABI so we can eventually remove it.

Reviewed-by: Joel Stanley <joel@jms.id.au>

I did some builds and boot tested them in qemu fine.

>
> Thanks,
> Nick
>
> Nicholas Piggin (4):
>   module: add module_elf_check_arch for module-specific checks
>   powerpc/64: Add module check for ELF ABI version
>   powerpc/64: Add big-endian ELFv2 flavour to crypto VMX asm generation
>   powerpc/64: Option to build big-endian with ELFv2 ABI
>
>  arch/powerpc/Kconfig                   | 21 +++++++++++++++++++++
>  arch/powerpc/kernel/module_64.c        | 10 ++++++++++
>  arch/powerpc/platforms/Kconfig.cputype |  4 ++--
>  drivers/crypto/vmx/Makefile            | 12 +++++++++++-
>  drivers/crypto/vmx/ppc-xlate.pl        | 10 ++++++----
>  include/linux/moduleloader.h           |  3 +++
>  kernel/module/main.c                   | 10 ++++++++++
>  7 files changed, 63 insertions(+), 7 deletions(-)
>
> --
> 2.37.2
>
Michael Ellerman Dec. 8, 2022, 12:39 p.m. UTC | #2 
On Mon, 28 Nov 2022 14:15:35 +1000, Nicholas Piggin wrote:
> This is hopefully the final attempt. Luis was happy for the module
> patch to go via the powerpc tree, so I've put the the ELFv2 for big
> endian build patches into the series. Hopefully we can deprecate
> the ELFv1 ABI
> 
> Since v5, I cleaned up patch 2 as per Christophe's review. And patch
> 4 I removed the EXPERT depends so it's easier to test. It's marked as
> experimental, but we should soon make it default and try to deprecate
> the v1 ABI so we can eventually remove it.
> 
> [...]

Applied to powerpc/next.

[1/4] module: add module_elf_check_arch for module-specific checks
      https://git.kernel.org/powerpc/c/f9231a996e229c13d23f907352c2cea84bd1c30a
[2/4] powerpc/64: Add module check for ELF ABI version
      https://git.kernel.org/powerpc/c/de3d098dd1fc635535e3689c5d4aa0684242adde
[3/4] powerpc/64: Add big-endian ELFv2 flavour to crypto VMX asm generation
      https://git.kernel.org/powerpc/c/505ea33089dcfc3ee3201b0fcb94751165805413
[4/4] powerpc/64: Option to build big-endian with ELFv2 ABI
      https://git.kernel.org/powerpc/c/5017b45946722bdd20ac255c9ae7273b78d1f12e

cheers