[v4] target/sparc: Use explicit big-endian LD/ST API

[PATCH v4 5/5] configs/targets: Forbid SPARC to use legacy native endianness APIs

Posted by Philippe Mathieu-Daudé 5 days, 21 hours ago

All SPARC-related binaries are buildable without a single
use of the legacy "native endian" API. Set the transitional
TARGET_NOT_USING_LEGACY_NATIVE_ENDIAN_API definition to forbid
further uses of the legacy API.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 configs/targets/sparc-linux-user.mak       | 1 +
 configs/targets/sparc-softmmu.mak          | 1 +
 configs/targets/sparc32plus-linux-user.mak | 1 +
 configs/targets/sparc64-linux-user.mak     | 1 +
 configs/targets/sparc64-softmmu.mak        | 1 +
 5 files changed, 5 insertions(+)

diff --git a/configs/targets/sparc-linux-user.mak b/configs/targets/sparc-linux-user.mak
index 4ff4b7287d2..d3f0716ca2d 100644
--- a/configs/targets/sparc-linux-user.mak
+++ b/configs/targets/sparc-linux-user.mak
@@ -3,3 +3,4 @@ TARGET_SYSTBL_ABI=common,32
 TARGET_SYSTBL=syscall.tbl
 TARGET_BIG_ENDIAN=y
 TARGET_LONG_BITS=32
+TARGET_NOT_USING_LEGACY_NATIVE_ENDIAN_API=y
diff --git a/configs/targets/sparc-softmmu.mak b/configs/targets/sparc-softmmu.mak
index 57801faf1fc..272fd46f6db 100644
--- a/configs/targets/sparc-softmmu.mak
+++ b/configs/targets/sparc-softmmu.mak
@@ -1,3 +1,4 @@
 TARGET_ARCH=sparc
 TARGET_BIG_ENDIAN=y
 TARGET_LONG_BITS=32
+TARGET_NOT_USING_LEGACY_NATIVE_ENDIAN_API=y
diff --git a/configs/targets/sparc32plus-linux-user.mak b/configs/targets/sparc32plus-linux-user.mak
index 7a16934fd17..3e6c72e793e 100644
--- a/configs/targets/sparc32plus-linux-user.mak
+++ b/configs/targets/sparc32plus-linux-user.mak
@@ -6,3 +6,4 @@ TARGET_SYSTBL_ABI=common,32
 TARGET_SYSTBL=syscall.tbl
 TARGET_BIG_ENDIAN=y
 TARGET_LONG_BITS=64
+TARGET_NOT_USING_LEGACY_NATIVE_ENDIAN_API=y
diff --git a/configs/targets/sparc64-linux-user.mak b/configs/targets/sparc64-linux-user.mak
index 7c2ecb7be06..3bbd8495210 100644
--- a/configs/targets/sparc64-linux-user.mak
+++ b/configs/targets/sparc64-linux-user.mak
@@ -6,3 +6,4 @@ TARGET_SYSTBL=syscall.tbl
 TARGET_BIG_ENDIAN=y
 TARGET_XML_FILES=gdb-xml/sparc64-core.xml
 TARGET_LONG_BITS=64
+TARGET_NOT_USING_LEGACY_NATIVE_ENDIAN_API=y
diff --git a/configs/targets/sparc64-softmmu.mak b/configs/targets/sparc64-softmmu.mak
index 8ee6d057682..8a0290c2093 100644
--- a/configs/targets/sparc64-softmmu.mak
+++ b/configs/targets/sparc64-softmmu.mak
@@ -4,3 +4,4 @@ TARGET_BIG_ENDIAN=y
 TARGET_XML_FILES=gdb-xml/sparc64-core.xml
 TARGET_LONG_BITS=64
 TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
+TARGET_NOT_USING_LEGACY_NATIVE_ENDIAN_API=y
-- 
2.52.0

Re: [PATCH v4 5/5] configs/targets: Forbid SPARC to use legacy native endianness APIs

Posted by Richard Henderson 5 days, 18 hours ago

On 2/4/26 09:00, Philippe Mathieu-Daudé wrote:
> All SPARC-related binaries are buildable without a single
> use of the legacy "native endian" API. Set the transitional
> TARGET_NOT_USING_LEGACY_NATIVE_ENDIAN_API definition to forbid
> further uses of the legacy API.
> 
> Signed-off-by: Philippe Mathieu-Daudé<philmd@linaro.org>
> Reviewed-by: Manos Pitsidianakis<manos.pitsidianakis@linaro.org>
> ---
>   configs/targets/sparc-linux-user.mak       | 1 +
>   configs/targets/sparc-softmmu.mak          | 1 +
>   configs/targets/sparc32plus-linux-user.mak | 1 +
>   configs/targets/sparc64-linux-user.mak     | 1 +
>   configs/targets/sparc64-softmmu.mak        | 1 +
>   5 files changed, 5 insertions(+)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~

[RFC PATCH v4 1/5] qemu/int128: Add int128_signextend()
[RFC PATCH v4 2/5] accel/tcg: Have cpu_ld*_mmu() methods consider the MO_SIGN flag
[PATCH v4 3/5] target/sparc: Remove dubious swapping in LD_code() helper
[PATCH v4 4/5] target/sparc: Replace MO_TE -> MO_BE
[PATCH v4 5/5] configs/targets: Forbid SPARC to use legacy native endianness APIs