mbox series

[v2,00/14] Another set of Bootlin toolchain integration improvements

Message ID 20220608071027.3384469-1-thomas.petazzoni@bootlin.com
Headers show
Series Another set of Bootlin toolchain integration improvements | expand

Message

Thomas Petazzoni June 8, 2022, 7:10 a.m. UTC
Hello,

Overall summary
===============

Patches 1 to 12 in this series are fixes for autobuilder
issues. Patches 13 and 14 are improvements.

Patches 1 and 2 fix the issue of RISC-V 64-bit toolchains being
available on MMU-less architectures, even though they are not
supported by the toolchain.

Patches 3 and 4 fix the issue of ARM little endian toolchains being
selectable for ARM big endian architectures.

Patches 5 and 6 fix the issue of i686 toolchains being used for < i686
platforms.

Patches 7, 8 and 9 fix the issue of an error when trying to copy
gdbserver, with toolchains that don't have gdbserver.

Patches 10, 11 and 12 fix the issue of the "mips64" Bootlin toolchains
being obsolete, but the gen-bootlin-toolchains still thinking they
were available. This caused some mips64 n64 configurations to be
thought of having Bootlin toolchains available, but in fact no such
toolchains were provided. We fix this by making sure we check if an
architecture has valid toolchains, and erroring out if not. And of
course, we remove the "mips64" toolchain config.

Patches 13 and 14 add support for the ARMv7 big-endian toolchains
provided by toolchains.bootlin.com.

Changes since v1
================

 - Patches 10, 11 and 12 added to fix the "mips64" issue.

Thomas

Thomas Petazzoni (14):
  support/scripts/gen-bootlin-toolchains: fix RISC-V 64-bit toolchain
    description
  support/testing,toolchain/toolchain-external/toolchain-external-bootlin:
    regenerate with latest gen-bootlin-toolchains script
  support/scripts/gen-bootlin-toolchains: improve dependencies of ARM
    toolchains
  toolchain/toolchain-external/toolchain-external-bootlin: regenerate
    with BR2_ARM dependency
  support/scripts/gen-bootlin-toolchains: adjust dependencies of i686
    toolchains
  toolchain/toolchain-external/toolchain-external-bootlin: regenerate
    after i686 toolchain dependency fixes
  toolchain/toolchain-external: add
    BR2_TOOLCHAIN_EXTERNAL_HAS_NO_GDBSERVER option
  support/scripts/gen-bootlin-toolchains: make use of
    BR2_TOOLCHAIN_EXTERNAL_HAS_NO_GDBSERVER
  toolchain/toolchain-external/toolchain-external-bootlin: regenerate
    with BR2_TOOLCHAIN_EXTERNAL_HAS_NO_GDBSERVER
  support/scripts/gen-bootlin-toolchains: check that toolchains exists
    for a certain arch
  support/scripts/gen-bootlin-toolchains: drop "mips64" architecture
    variant
  toolchain/toolchain-external/toolchain-external-bootlin: regenerate
    after mips64 toolchain removal
  support/scripts/gen-bootlin-toolchains: add support for ARMv7 big
    endian toolchain
  support/testing,toolchain/toolchain-external/toolchain-external-bootlin:
    regenerate with ARMv7 big endian toolchain

 support/scripts/gen-bootlin-toolchains        |  44 +++-
 .../tests/toolchain/test_external_bootlin.py  | 102 ++++++++
 toolchain/toolchain-external/Config.in        |   4 +
 .../Config.in.options                         | 234 +++++++++++++++++-
 .../toolchain-external-bootlin.hash           |  12 +
 .../toolchain-external-bootlin.mk             |  36 +++
 6 files changed, 412 insertions(+), 20 deletions(-)

Comments

Arnout Vandecappelle June 14, 2022, 3:43 p.m. UTC | #1
On 08/06/2022 09:10, Thomas Petazzoni via buildroot wrote:
> Hello,
> 
> Overall summary
> ===============
> 
> Patches 1 to 12 in this series are fixes for autobuilder
> issues. Patches 13 and 14 are improvements.
> 
> Patches 1 and 2 fix the issue of RISC-V 64-bit toolchains being
> available on MMU-less architectures, even though they are not
> supported by the toolchain.
> 
> Patches 3 and 4 fix the issue of ARM little endian toolchains being
> selectable for ARM big endian architectures.
> 
> Patches 5 and 6 fix the issue of i686 toolchains being used for < i686
> platforms.
> 
> Patches 7, 8 and 9 fix the issue of an error when trying to copy
> gdbserver, with toolchains that don't have gdbserver.
> 
> Patches 10, 11 and 12 fix the issue of the "mips64" Bootlin toolchains
> being obsolete, but the gen-bootlin-toolchains still thinking they
> were available. This caused some mips64 n64 configurations to be
> thought of having Bootlin toolchains available, but in fact no such
> toolchains were provided. We fix this by making sure we check if an
> architecture has valid toolchains, and erroring out if not. And of
> course, we remove the "mips64" toolchain config.
> 
> Patches 13 and 14 add support for the ARMv7 big-endian toolchains
> provided by toolchains.bootlin.com.
> 
> Changes since v1
> ================
> 
>   - Patches 10, 11 and 12 added to fix the "mips64" issue.
> 
> Thomas
> 
> Thomas Petazzoni (14):
>    support/scripts/gen-bootlin-toolchains: fix RISC-V 64-bit toolchain
>      description
>    support/testing,toolchain/toolchain-external/toolchain-external-bootlin:
>      regenerate with latest gen-bootlin-toolchains script
>    support/scripts/gen-bootlin-toolchains: improve dependencies of ARM
>      toolchains
>    toolchain/toolchain-external/toolchain-external-bootlin: regenerate
>      with BR2_ARM dependency
>    support/scripts/gen-bootlin-toolchains: adjust dependencies of i686
>      toolchains
>    toolchain/toolchain-external/toolchain-external-bootlin: regenerate
>      after i686 toolchain dependency fixes
>    toolchain/toolchain-external: add
>      BR2_TOOLCHAIN_EXTERNAL_HAS_NO_GDBSERVER option
>    support/scripts/gen-bootlin-toolchains: make use of
>      BR2_TOOLCHAIN_EXTERNAL_HAS_NO_GDBSERVER
>    toolchain/toolchain-external/toolchain-external-bootlin: regenerate
>      with BR2_TOOLCHAIN_EXTERNAL_HAS_NO_GDBSERVER
>    support/scripts/gen-bootlin-toolchains: check that toolchains exists
>      for a certain arch
>    support/scripts/gen-bootlin-toolchains: drop "mips64" architecture
>      variant
>    toolchain/toolchain-external/toolchain-external-bootlin: regenerate
>      after mips64 toolchain removal
>    support/scripts/gen-bootlin-toolchains: add support for ARMv7 big
>      endian toolchain
>    support/testing,toolchain/toolchain-external/toolchain-external-bootlin:
>      regenerate with ARMv7 big endian toolchain
> 
>   support/scripts/gen-bootlin-toolchains        |  44 +++-
>   .../tests/toolchain/test_external_bootlin.py  | 102 ++++++++
>   toolchain/toolchain-external/Config.in        |   4 +
>   .../Config.in.options                         | 234 +++++++++++++++++-
>   .../toolchain-external-bootlin.hash           |  12 +
>   .../toolchain-external-bootlin.mk             |  36 +++
>   6 files changed, 412 insertions(+), 20 deletions(-)
> 


  Series applied to master, thanks.

  Regards,
  Arnout