[v2] target/hppa: Use explicit big-endian LD/ST API

[PATCH v2 0/5] target/hppa: Use explicit big-endian LD/ST API

Philippe Mathieu-Daudé posted 5 patches 1 month, 1 week ago

Diff against v1
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20251229225517.45078-1-philmd@linaro.org

Maintainers: Laurent Vivier <laurent@vivier.eu>, Richard Henderson <richard.henderson@linaro.org>, Helge Deller <deller@gmx.de>

configs/targets/hppa-linux-user.mak |  1 +
configs/targets/hppa-softmmu.mak    |  1 +
target/hppa/gdbstub.c               |  8 ++----
target/hppa/int_helper.c            |  2 +-
target/hppa/op_helper.c             | 44 ++++++++++++++---------------
target/hppa/translate.c             |  3 +-
6 files changed, 30 insertions(+), 29 deletions(-)

Expand all Fold all

[PATCH v2 0/5] target/hppa: Use explicit big-endian LD/ST API

Posted by Philippe Mathieu-Daudé 1 month, 1 week ago

Missing review: #2 and #3

Since v1:
- Rebased on commit 83202ca7227 (hppa-updates-qemu-v11-pull-request)

PA-RISC is big-endian. Use the explicit 'big'
endianness instead of the 'native' one.
Forbid further uses of legacy APIs.

tag: https://gitlab.com/philmd/qemu/-/tags/endian_hppa-v1
CI: https://gitlab.com/philmd/qemu/-/pipelines/2231223066

Philippe Mathieu-Daudé (5):
  target/hppa: Use big-endian variant of cpu_ld/st_data*()
  target/hppa: Remove unnecessary @cpu variable by using cpu_env()
  target/hppa: Use explicit big-endian LD/ST API
  target/hppa: Inline translator_ldl()
  configs/targets: Forbid HPPA to use legacy native endianness APIs

 configs/targets/hppa-linux-user.mak |  1 +
 configs/targets/hppa-softmmu.mak    |  1 +
 target/hppa/gdbstub.c               |  8 ++----
 target/hppa/int_helper.c            |  2 +-
 target/hppa/op_helper.c             | 44 ++++++++++++++---------------
 target/hppa/translate.c             |  3 +-
 6 files changed, 30 insertions(+), 29 deletions(-)

-- 
2.52.0

Re: [PATCH v2 0/5] target/hppa: Use explicit big-endian LD/ST API

Posted by Philippe Mathieu-Daudé 1 month, 1 week ago

On 29/12/25 23:55, Philippe Mathieu-Daudé wrote:
> Missing review: #2 and #3
> 
> Since v1:
> - Rebased on commit 83202ca7227 (hppa-updates-qemu-v11-pull-request)
> 
> PA-RISC is big-endian. Use the explicit 'big'
> endianness instead of the 'native' one.
> Forbid further uses of legacy APIs.
> 
> tag: https://gitlab.com/philmd/qemu/-/tags/endian_hppa-v1

   https://gitlab.com/philmd/qemu/-/tags/endian_hppa-v2 ;)

> CI: https://gitlab.com/philmd/qemu/-/pipelines/2231223066
> 
> Philippe Mathieu-Daudé (5):
>    target/hppa: Use big-endian variant of cpu_ld/st_data*()
>    target/hppa: Remove unnecessary @cpu variable by using cpu_env()
>    target/hppa: Use explicit big-endian LD/ST API
>    target/hppa: Inline translator_ldl()
>    configs/targets: Forbid HPPA to use legacy native endianness APIs
> 
>   configs/targets/hppa-linux-user.mak |  1 +
>   configs/targets/hppa-softmmu.mak    |  1 +
>   target/hppa/gdbstub.c               |  8 ++----
>   target/hppa/int_helper.c            |  2 +-
>   target/hppa/op_helper.c             | 44 ++++++++++++++---------------
>   target/hppa/translate.c             |  3 +-
>   6 files changed, 30 insertions(+), 29 deletions(-)
>