mbox series

[0/2] tests/tcg: Fix new cross-compiler warnings

Message ID 20240724010733.22129-1-richard.henderson@linaro.org (mailing list archive)
Headers show
Series tests/tcg: Fix new cross-compiler warnings | expand

Message

Richard Henderson July 24, 2024, 1:07 a.m. UTC
Recent updates to debian cross-compilers have enabled some
new warnings by default, which encourage good behaviour for
userland binaries.

Neither are applicable for our system mode kernel images,
but we do have a few userland tests that are affected.

The --noexecstack assembler flag takes care of adding the
appropriate .note.GNU-stack elf note for us, rather than
writing a dozen lines of assembly template, and so is a
true fix rather than merely disabling the linker warning.


r~


Richard Henderson (2):
  tests/tcg: Use --noexecstack with assembler files
  tests/tcg/loongarch64: Use --no-warn-rwx-segments to link system tests

 tests/tcg/Makefile.target                     | 2 +-
 tests/tcg/aarch64/Makefile.softmmu-target     | 2 +-
 tests/tcg/alpha/Makefile.softmmu-target       | 2 +-
 tests/tcg/arm/Makefile.softmmu-target         | 2 +-
 tests/tcg/arm/Makefile.target                 | 2 +-
 tests/tcg/i386/Makefile.softmmu-target        | 2 +-
 tests/tcg/loongarch64/Makefile.softmmu-target | 4 ++--
 tests/tcg/riscv64/Makefile.softmmu-target     | 2 +-
 tests/tcg/s390x/Makefile.softmmu-target       | 2 +-
 tests/tcg/x86_64/Makefile.softmmu-target      | 2 +-
 10 files changed, 11 insertions(+), 11 deletions(-)

Comments

Alex Bennée July 24, 2024, 9:54 a.m. UTC | #1
Richard Henderson <richard.henderson@linaro.org> writes:

> Recent updates to debian cross-compilers have enabled some
> new warnings by default, which encourage good behaviour for
> userland binaries.
>
> Neither are applicable for our system mode kernel images,
> but we do have a few userland tests that are affected.
>
> The --noexecstack assembler flag takes care of adding the
> appropriate .note.GNU-stack elf note for us, rather than
> writing a dozen lines of assembly template, and so is a
> true fix rather than merely disabling the linker warning.

Queued to maintainer/for-9.1, thanks.
>  10 files changed, 11 insertions(+), 11 deletions(-)
Alex Bennée July 24, 2024, 10:50 a.m. UTC | #2
Richard Henderson <richard.henderson@linaro.org> writes:

> Recent updates to debian cross-compilers have enabled some
> new warnings by default, which encourage good behaviour for
> userland binaries.
>
> Neither are applicable for our system mode kernel images,
> but we do have a few userland tests that are affected.
>
> The --noexecstack assembler flag takes care of adding the
> appropriate .note.GNU-stack elf note for us, rather than
> writing a dozen lines of assembly template, and so is a
> true fix rather than merely disabling the linker warning.

Queued to maintainer/for-9.1, thanks.