mbox series

[0/9] powerpc: Fix ppc64le configurations

Message ID 20220729000904.1295295-1-joel@jms.id.au
Headers show
Series powerpc: Fix ppc64le configurations | expand

Message

Joel Stanley July 29, 2022, 12:08 a.m. UTC 
The autobuilder has detected a bunch of powerpc build failures in recent
months, leading to workarounds applied to various packages. Many of these
failures were caused by invalid GCC configurations: building GCC in
little endian or 64-bit mode for CPUs that don't support that
configuration.

The first few patches clarify the menuconfig options for powerpc.

The third patch "arch/powerpc: Enable powerpc64le only on CPUs that
support it" should stop a user from selecting a ppc64le config for a CPU
that doesn't support it. The list of supported configurations can be
cross referenced from the kernel Kconfig:

 https://elixir.bootlin.com/linux/v5.18/source/arch/powerpc/platforms/Kconfig.cputype#L157

Patches 4-5 add new Power server CPUs from IBM.

The final three patches remove workarounds applied for power8, which
are unnecessary now. This should enable packages to be built with
optimisations for p9/p10 where applicable.

I've boot tested all of the in-tree configs, and tested the p9 and p10
cpu types.

Joel Stanley (9):
  arch/powerpc: Hide SPE ABI behind CPU type
  arch/powerpc: Clarify generic CPUs
  arch/powerpc: Enable powerpc64le only on CPUs that support it
  package/glibc: Use powerpc64le instead of power8
  arch/powerpc: Add Power9 CPU type
  arch/powerpc: Add Power10 CPU type
  package/mesa3d: Remove force disable of power8
  package/zlib-ng: Revert Power8 special casing
  package/openblas: Add Power9 and Power10 targets

 arch/Config.in.powerpc     | 40 +++++++++++++++++++++++++++++++-------
 package/glibc/Config.in    |  2 +-
 package/mesa3d/mesa3d.mk   |  3 +--
 package/openblas/Config.in |  2 ++
 package/zlib-ng/zlib-ng.mk |  6 ------
 5 files changed, 37 insertions(+), 16 deletions(-)

Comments

Yann E. MORIN Aug. 3, 2022, 9:07 p.m. UTC | #1 
Joel, All,

On 2022-07-29 09:38 +0930, Joel Stanley spake thusly:
> The autobuilder has detected a bunch of powerpc build failures in recent
> months, leading to workarounds applied to various packages. Many of these
> failures were caused by invalid GCC configurations: building GCC in
> little endian or 64-bit mode for CPUs that don't support that
> configuration.

Thanks for tackling the issue! 👍

There has been some feedback on some patches in the series, which will
require either some of the patches to be reworked, and/or the commit
logs to be expanded with some more detailed explanations.

So I've marked this series as Changes-requested in patchwork.

Looking forward to seeing he next iteration to eventually fix the
issues!

Regards,
Yann E. MORIN.

> The first few patches clarify the menuconfig options for powerpc.
> 
> The third patch "arch/powerpc: Enable powerpc64le only on CPUs that
> support it" should stop a user from selecting a ppc64le config for a CPU
> that doesn't support it. The list of supported configurations can be
> cross referenced from the kernel Kconfig:
> 
>  https://elixir.bootlin.com/linux/v5.18/source/arch/powerpc/platforms/Kconfig.cputype#L157
> 
> Patches 4-5 add new Power server CPUs from IBM.
> 
> The final three patches remove workarounds applied for power8, which
> are unnecessary now. This should enable packages to be built with
> optimisations for p9/p10 where applicable.
> 
> I've boot tested all of the in-tree configs, and tested the p9 and p10
> cpu types.
> 
> Joel Stanley (9):
>   arch/powerpc: Hide SPE ABI behind CPU type
>   arch/powerpc: Clarify generic CPUs
>   arch/powerpc: Enable powerpc64le only on CPUs that support it
>   package/glibc: Use powerpc64le instead of power8
>   arch/powerpc: Add Power9 CPU type
>   arch/powerpc: Add Power10 CPU type
>   package/mesa3d: Remove force disable of power8
>   package/zlib-ng: Revert Power8 special casing
>   package/openblas: Add Power9 and Power10 targets
> 
>  arch/Config.in.powerpc     | 40 +++++++++++++++++++++++++++++++-------
>  package/glibc/Config.in    |  2 +-
>  package/mesa3d/mesa3d.mk   |  3 +--
>  package/openblas/Config.in |  2 ++
>  package/zlib-ng/zlib-ng.mk |  6 ------
>  5 files changed, 37 insertions(+), 16 deletions(-)
> 
> -- 
> 2.35.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot