[PATCH 9/9] configs/targets: Mark targets not using legacy ldst_phys() API

Philippe Mathieu-Daudé posted 9 patches 1 month, 2 weeks ago
Maintainers: "Philippe Mathieu-Daudé" <philmd@linaro.org>, Jiaxun Yang <jiaxun.yang@flygoat.com>, Palmer Dabbelt <palmer@dabbelt.com>, Alistair Francis <alistair.francis@wdc.com>, Weiwei Li <liwei1518@gmail.com>, Daniel Henrique Barboza <dbarboza@ventanamicro.com>, Liu Zhiwei <zhiwei_liu@linux.alibaba.com>, "Cédric Le Goater" <clg@kaod.org>, Peter Maydell <peter.maydell@linaro.org>, Steven Lee <steven_lee@aspeedtech.com>, Troy Lee <leetroy@gmail.com>, Jamin Lin <jamin_lin@aspeedtech.com>, Andrew Jeffery <andrew@codeconstruct.com.au>, Joel Stanley <joel@jms.id.au>, "Michael S. Tsirkin" <mst@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Peter Xu <peterx@redhat.com>, David Hildenbrand <david@kernel.org>, Zhao Liu <zhao1.liu@intel.com>, Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>, Artyom Tarasenko <atar4qemu@gmail.com>
[PATCH 9/9] configs/targets: Mark targets not using legacy ldst_phys() API
Posted by Philippe Mathieu-Daudé 1 month, 2 weeks ago
These targets were only using the legacy ldst_phys() API
via the virtio load/store helpers, which got cleaned up.

Mark them not using the legacy ldst_phys() API to avoid
further use, allowing to eventually remove it.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 configs/targets/mips-softmmu.mak     | 1 +
 configs/targets/mips64-softmmu.mak   | 1 +
 configs/targets/mips64el-softmmu.mak | 1 +
 configs/targets/mipsel-softmmu.mak   | 1 +
 configs/targets/or1k-softmmu.mak     | 1 +
 configs/targets/riscv32-softmmu.mak  | 1 +
 configs/targets/riscv64-softmmu.mak  | 1 +
 configs/targets/sh4-softmmu.mak      | 1 +
 configs/targets/sh4eb-softmmu.mak    | 1 +
 configs/targets/sparc64-softmmu.mak  | 1 +
 configs/targets/xtensa-softmmu.mak   | 1 +
 configs/targets/xtensaeb-softmmu.mak | 1 +
 12 files changed, 12 insertions(+)

diff --git a/configs/targets/mips-softmmu.mak b/configs/targets/mips-softmmu.mak
index c9588066b8d..95397af9442 100644
--- a/configs/targets/mips-softmmu.mak
+++ b/configs/targets/mips-softmmu.mak
@@ -1,3 +1,4 @@
 TARGET_ARCH=mips
 TARGET_BIG_ENDIAN=y
 TARGET_LONG_BITS=32
+TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
diff --git a/configs/targets/mips64-softmmu.mak b/configs/targets/mips64-softmmu.mak
index 7202655fcac..f9cc41b9ed4 100644
--- a/configs/targets/mips64-softmmu.mak
+++ b/configs/targets/mips64-softmmu.mak
@@ -2,3 +2,4 @@ TARGET_ARCH=mips64
 TARGET_BASE_ARCH=mips
 TARGET_BIG_ENDIAN=y
 TARGET_LONG_BITS=64
+TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
diff --git a/configs/targets/mips64el-softmmu.mak b/configs/targets/mips64el-softmmu.mak
index 3ebeadb29ea..4e7dedc5f30 100644
--- a/configs/targets/mips64el-softmmu.mak
+++ b/configs/targets/mips64el-softmmu.mak
@@ -1,3 +1,4 @@
 TARGET_ARCH=mips64
 TARGET_BASE_ARCH=mips
 TARGET_LONG_BITS=64
+TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
diff --git a/configs/targets/mipsel-softmmu.mak b/configs/targets/mipsel-softmmu.mak
index 90e09bdc3e5..b0fba8a9d0a 100644
--- a/configs/targets/mipsel-softmmu.mak
+++ b/configs/targets/mipsel-softmmu.mak
@@ -1,2 +1,3 @@
 TARGET_ARCH=mips
 TARGET_LONG_BITS=32
+TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
diff --git a/configs/targets/or1k-softmmu.mak b/configs/targets/or1k-softmmu.mak
index 0e47d9878b0..89f81b75bf7 100644
--- a/configs/targets/or1k-softmmu.mak
+++ b/configs/targets/or1k-softmmu.mak
@@ -3,3 +3,4 @@ TARGET_BIG_ENDIAN=y
 # needed by boot.c and all boards
 TARGET_NEED_FDT=y
 TARGET_LONG_BITS=32
+TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
diff --git a/configs/targets/riscv32-softmmu.mak b/configs/targets/riscv32-softmmu.mak
index db55275b868..7476e596082 100644
--- a/configs/targets/riscv32-softmmu.mak
+++ b/configs/targets/riscv32-softmmu.mak
@@ -4,3 +4,4 @@ TARGET_XML_FILES= gdb-xml/riscv-32bit-cpu.xml gdb-xml/riscv-32bit-fpu.xml gdb-xm
 # needed by boot.c
 TARGET_NEED_FDT=y
 TARGET_LONG_BITS=32
+TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
diff --git a/configs/targets/riscv64-softmmu.mak b/configs/targets/riscv64-softmmu.mak
index 2bdd4a62cd2..f836f08c5b9 100644
--- a/configs/targets/riscv64-softmmu.mak
+++ b/configs/targets/riscv64-softmmu.mak
@@ -5,3 +5,4 @@ TARGET_XML_FILES= gdb-xml/riscv-64bit-cpu.xml gdb-xml/riscv-32bit-fpu.xml gdb-xm
 # needed by boot.c
 TARGET_NEED_FDT=y
 TARGET_LONG_BITS=64
+TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
diff --git a/configs/targets/sh4-softmmu.mak b/configs/targets/sh4-softmmu.mak
index 787d349b501..e0b51f8108d 100644
--- a/configs/targets/sh4-softmmu.mak
+++ b/configs/targets/sh4-softmmu.mak
@@ -1,2 +1,3 @@
 TARGET_ARCH=sh4
 TARGET_LONG_BITS=32
+TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
diff --git a/configs/targets/sh4eb-softmmu.mak b/configs/targets/sh4eb-softmmu.mak
index cdea2c61c58..7850dac8191 100644
--- a/configs/targets/sh4eb-softmmu.mak
+++ b/configs/targets/sh4eb-softmmu.mak
@@ -1,3 +1,4 @@
 TARGET_ARCH=sh4
 TARGET_BIG_ENDIAN=y
 TARGET_LONG_BITS=32
+TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
diff --git a/configs/targets/sparc64-softmmu.mak b/configs/targets/sparc64-softmmu.mak
index d9d51d21e59..8ee6d057682 100644
--- a/configs/targets/sparc64-softmmu.mak
+++ b/configs/targets/sparc64-softmmu.mak
@@ -3,3 +3,4 @@ TARGET_BASE_ARCH=sparc
 TARGET_BIG_ENDIAN=y
 TARGET_XML_FILES=gdb-xml/sparc64-core.xml
 TARGET_LONG_BITS=64
+TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
diff --git a/configs/targets/xtensa-softmmu.mak b/configs/targets/xtensa-softmmu.mak
index 2a9797338a6..6aad751e2ba 100644
--- a/configs/targets/xtensa-softmmu.mak
+++ b/configs/targets/xtensa-softmmu.mak
@@ -1,2 +1,3 @@
 TARGET_ARCH=xtensa
 TARGET_LONG_BITS=32
+TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
diff --git a/configs/targets/xtensaeb-softmmu.mak b/configs/targets/xtensaeb-softmmu.mak
index 5204729af8b..537407e11ae 100644
--- a/configs/targets/xtensaeb-softmmu.mak
+++ b/configs/targets/xtensaeb-softmmu.mak
@@ -1,3 +1,4 @@
 TARGET_ARCH=xtensa
 TARGET_BIG_ENDIAN=y
 TARGET_LONG_BITS=32
+TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
-- 
2.52.0


Re: [PATCH 9/9] configs/targets: Mark targets not using legacy ldst_phys() API
Posted by Richard Henderson 1 month, 1 week ago
On 12/25/25 02:13, Philippe Mathieu-Daudé wrote:
> These targets were only using the legacy ldst_phys() API
> via the virtio load/store helpers, which got cleaned up.
> 
> Mark them not using the legacy ldst_phys() API to avoid
> further use, allowing to eventually remove it.
> 
> Signed-off-by: Philippe Mathieu-Daudé<philmd@linaro.org>
> ---
>   configs/targets/mips-softmmu.mak     | 1 +
>   configs/targets/mips64-softmmu.mak   | 1 +
>   configs/targets/mips64el-softmmu.mak | 1 +
>   configs/targets/mipsel-softmmu.mak   | 1 +
>   configs/targets/or1k-softmmu.mak     | 1 +
>   configs/targets/riscv32-softmmu.mak  | 1 +
>   configs/targets/riscv64-softmmu.mak  | 1 +
>   configs/targets/sh4-softmmu.mak      | 1 +
>   configs/targets/sh4eb-softmmu.mak    | 1 +
>   configs/targets/sparc64-softmmu.mak  | 1 +
>   configs/targets/xtensa-softmmu.mak   | 1 +
>   configs/targets/xtensaeb-softmmu.mak | 1 +
>   12 files changed, 12 insertions(+)

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

r~

Re: [PATCH 9/9] configs/targets: Mark targets not using legacy ldst_phys() API
Posted by Manos Pitsidianakis 1 month, 2 weeks ago
On Wed, Dec 24, 2025 at 5:15 PM Philippe Mathieu-Daudé
<philmd@linaro.org> wrote:
>
> These targets were only using the legacy ldst_phys() API
> via the virtio load/store helpers, which got cleaned up.
>
> Mark them not using the legacy ldst_phys() API to avoid
> further use, allowing to eventually remove it.
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---

Reviewed-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>

Nice cleanup!

>  configs/targets/mips-softmmu.mak     | 1 +
>  configs/targets/mips64-softmmu.mak   | 1 +
>  configs/targets/mips64el-softmmu.mak | 1 +
>  configs/targets/mipsel-softmmu.mak   | 1 +
>  configs/targets/or1k-softmmu.mak     | 1 +
>  configs/targets/riscv32-softmmu.mak  | 1 +
>  configs/targets/riscv64-softmmu.mak  | 1 +
>  configs/targets/sh4-softmmu.mak      | 1 +
>  configs/targets/sh4eb-softmmu.mak    | 1 +
>  configs/targets/sparc64-softmmu.mak  | 1 +
>  configs/targets/xtensa-softmmu.mak   | 1 +
>  configs/targets/xtensaeb-softmmu.mak | 1 +
>  12 files changed, 12 insertions(+)
>
> diff --git a/configs/targets/mips-softmmu.mak b/configs/targets/mips-softmmu.mak
> index c9588066b8d..95397af9442 100644
> --- a/configs/targets/mips-softmmu.mak
> +++ b/configs/targets/mips-softmmu.mak
> @@ -1,3 +1,4 @@
>  TARGET_ARCH=mips
>  TARGET_BIG_ENDIAN=y
>  TARGET_LONG_BITS=32
> +TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
> diff --git a/configs/targets/mips64-softmmu.mak b/configs/targets/mips64-softmmu.mak
> index 7202655fcac..f9cc41b9ed4 100644
> --- a/configs/targets/mips64-softmmu.mak
> +++ b/configs/targets/mips64-softmmu.mak
> @@ -2,3 +2,4 @@ TARGET_ARCH=mips64
>  TARGET_BASE_ARCH=mips
>  TARGET_BIG_ENDIAN=y
>  TARGET_LONG_BITS=64
> +TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
> diff --git a/configs/targets/mips64el-softmmu.mak b/configs/targets/mips64el-softmmu.mak
> index 3ebeadb29ea..4e7dedc5f30 100644
> --- a/configs/targets/mips64el-softmmu.mak
> +++ b/configs/targets/mips64el-softmmu.mak
> @@ -1,3 +1,4 @@
>  TARGET_ARCH=mips64
>  TARGET_BASE_ARCH=mips
>  TARGET_LONG_BITS=64
> +TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
> diff --git a/configs/targets/mipsel-softmmu.mak b/configs/targets/mipsel-softmmu.mak
> index 90e09bdc3e5..b0fba8a9d0a 100644
> --- a/configs/targets/mipsel-softmmu.mak
> +++ b/configs/targets/mipsel-softmmu.mak
> @@ -1,2 +1,3 @@
>  TARGET_ARCH=mips
>  TARGET_LONG_BITS=32
> +TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
> diff --git a/configs/targets/or1k-softmmu.mak b/configs/targets/or1k-softmmu.mak
> index 0e47d9878b0..89f81b75bf7 100644
> --- a/configs/targets/or1k-softmmu.mak
> +++ b/configs/targets/or1k-softmmu.mak
> @@ -3,3 +3,4 @@ TARGET_BIG_ENDIAN=y
>  # needed by boot.c and all boards
>  TARGET_NEED_FDT=y
>  TARGET_LONG_BITS=32
> +TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
> diff --git a/configs/targets/riscv32-softmmu.mak b/configs/targets/riscv32-softmmu.mak
> index db55275b868..7476e596082 100644
> --- a/configs/targets/riscv32-softmmu.mak
> +++ b/configs/targets/riscv32-softmmu.mak
> @@ -4,3 +4,4 @@ TARGET_XML_FILES= gdb-xml/riscv-32bit-cpu.xml gdb-xml/riscv-32bit-fpu.xml gdb-xm
>  # needed by boot.c
>  TARGET_NEED_FDT=y
>  TARGET_LONG_BITS=32
> +TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
> diff --git a/configs/targets/riscv64-softmmu.mak b/configs/targets/riscv64-softmmu.mak
> index 2bdd4a62cd2..f836f08c5b9 100644
> --- a/configs/targets/riscv64-softmmu.mak
> +++ b/configs/targets/riscv64-softmmu.mak
> @@ -5,3 +5,4 @@ TARGET_XML_FILES= gdb-xml/riscv-64bit-cpu.xml gdb-xml/riscv-32bit-fpu.xml gdb-xm
>  # needed by boot.c
>  TARGET_NEED_FDT=y
>  TARGET_LONG_BITS=64
> +TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
> diff --git a/configs/targets/sh4-softmmu.mak b/configs/targets/sh4-softmmu.mak
> index 787d349b501..e0b51f8108d 100644
> --- a/configs/targets/sh4-softmmu.mak
> +++ b/configs/targets/sh4-softmmu.mak
> @@ -1,2 +1,3 @@
>  TARGET_ARCH=sh4
>  TARGET_LONG_BITS=32
> +TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
> diff --git a/configs/targets/sh4eb-softmmu.mak b/configs/targets/sh4eb-softmmu.mak
> index cdea2c61c58..7850dac8191 100644
> --- a/configs/targets/sh4eb-softmmu.mak
> +++ b/configs/targets/sh4eb-softmmu.mak
> @@ -1,3 +1,4 @@
>  TARGET_ARCH=sh4
>  TARGET_BIG_ENDIAN=y
>  TARGET_LONG_BITS=32
> +TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
> diff --git a/configs/targets/sparc64-softmmu.mak b/configs/targets/sparc64-softmmu.mak
> index d9d51d21e59..8ee6d057682 100644
> --- a/configs/targets/sparc64-softmmu.mak
> +++ b/configs/targets/sparc64-softmmu.mak
> @@ -3,3 +3,4 @@ TARGET_BASE_ARCH=sparc
>  TARGET_BIG_ENDIAN=y
>  TARGET_XML_FILES=gdb-xml/sparc64-core.xml
>  TARGET_LONG_BITS=64
> +TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
> diff --git a/configs/targets/xtensa-softmmu.mak b/configs/targets/xtensa-softmmu.mak
> index 2a9797338a6..6aad751e2ba 100644
> --- a/configs/targets/xtensa-softmmu.mak
> +++ b/configs/targets/xtensa-softmmu.mak
> @@ -1,2 +1,3 @@
>  TARGET_ARCH=xtensa
>  TARGET_LONG_BITS=32
> +TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
> diff --git a/configs/targets/xtensaeb-softmmu.mak b/configs/targets/xtensaeb-softmmu.mak
> index 5204729af8b..537407e11ae 100644
> --- a/configs/targets/xtensaeb-softmmu.mak
> +++ b/configs/targets/xtensaeb-softmmu.mak
> @@ -1,3 +1,4 @@
>  TARGET_ARCH=xtensa
>  TARGET_BIG_ENDIAN=y
>  TARGET_LONG_BITS=32
> +TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
> --
> 2.52.0
>