[v3] target/s390x: Prohibit target_ulong and TARGET_PAGE_SIZE uses

[PATCH v3 00/10] target/s390x: Prohibit target_ulong and TARGET_PAGE_SIZE uses

Philippe Mathieu-Daudé posted 10 patches 2 days, 9 hours ago

Diff against v1 v2
Download series mbox

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

Maintainers: Richard Henderson <richard.henderson@linaro.org>, Ilya Leoshkevich <iii@linux.ibm.com>, David Hildenbrand <david@kernel.org>, Thomas Huth <thuth@redhat.com>, Halil Pasic <pasic@linux.ibm.com>, Christian Borntraeger <borntraeger@linux.ibm.com>, Eric Farman <farman@linux.ibm.com>, Matthew Rosato <mjrosato@linux.ibm.com>

target/s390x/kvm/pv.h          |  27 ------
target/s390x/s390x-internal.h  |  14 +--
target/s390x/helper.c          |  10 +-
target/s390x/kvm/stubs.c       | 170 ++++++++++++++++++++++++++++++++-
target/s390x/mmu_helper.c      |  33 +++----
target/s390x/tcg/excp_helper.c |   3 +-
target/s390x/tcg/translate.c   |  66 ++++++-------
target/s390x/meson.build       |   8 +-
target/s390x/tcg/meson.build   |   2 +-
9 files changed, 240 insertions(+), 93 deletions(-)

Expand all Fold all

[PATCH v3 00/10] target/s390x: Prohibit target_ulong and TARGET_PAGE_SIZE uses

Posted by Philippe Mathieu-Daudé 2 days, 9 hours ago

Series fully reviewed (Thomas if you don't mind I'll merge it
via my single-binary tree).

Since v2:
- Added R-b/A-b tags
- Use hwaddr in mmu_translate_real (Eric)
- Add gen_features_h to s390x_common_system_ss[] (Pierrick)

Since v1:
- Drop merged patches
- Added R-b/A-b tags
- Use address_space_ldQ_be() in read_table_entry()
- One patch for vaddr replacement and another for hwaddr
- Do not remove the TARGET_PAGE_SIZE use in ioinst.c
- Expand TCG definitions for fixed 64-bit target

v1 cover:
- Restrict WatchPoint API to TCG
- Replace target_ulong uses by vaddr / hwaddr / uint64_t
- Remove the single TARGET_PAGE_SIZE
- Remove CONFIG_KVM from headers
- Compiles objects as target_common_system_arch, effectively
  forbidding further target_ulong / TARGET_PAGE_SIZE uses.

Philippe Mathieu-Daudé (10):
  target/s390x: Use address_space_ldl_be() in read_table_entry()
  target/s390x: Rename get_phys_page_debug() @vaddr argument as @v_addr
  target/s390x: Replace %target_ulong -> %vaddr where appropriate
  target/s390x: Replace %target_ulong -> %hwaddr where appropriate
  target/s390x: Un-inline KVM Protected Virtualization stubs
  target/s390x: Add more unreachable KVM stubs
  target/s390x: Build system units in common source set
  target/s390x: Expand TCGv type as 64-bit target
  target/s390x: Expand tcg_gen_qemu_ld/st_tl() as 64-bit target
  target/s390x: Expand tcg_global_mem_new() -> tcg_global_mem_new_i64()

 target/s390x/kvm/pv.h          |  27 ------
 target/s390x/s390x-internal.h  |  14 +--
 target/s390x/helper.c          |  10 +-
 target/s390x/kvm/stubs.c       | 170 ++++++++++++++++++++++++++++++++-
 target/s390x/mmu_helper.c      |  33 +++----
 target/s390x/tcg/excp_helper.c |   3 +-
 target/s390x/tcg/translate.c   |  66 ++++++-------
 target/s390x/meson.build       |   8 +-
 target/s390x/tcg/meson.build   |   2 +-
 9 files changed, 240 insertions(+), 93 deletions(-)

-- 
2.52.0

Re: [PATCH v3 00/10] target/s390x: Prohibit target_ulong and TARGET_PAGE_SIZE uses

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

On 6/2/26 19:19, Philippe Mathieu-Daudé wrote:

> Philippe Mathieu-Daudé (10):
>    target/s390x: Use address_space_ldl_be() in read_table_entry()
>    target/s390x: Rename get_phys_page_debug() @vaddr argument as @v_addr
>    target/s390x: Replace %target_ulong -> %vaddr where appropriate
>    target/s390x: Replace %target_ulong -> %hwaddr where appropriate
>    target/s390x: Un-inline KVM Protected Virtualization stubs
>    target/s390x: Add more unreachable KVM stubs
>    target/s390x: Build system units in common source set
>    target/s390x: Expand TCGv type as 64-bit target
>    target/s390x: Expand tcg_gen_qemu_ld/st_tl() as 64-bit target
>    target/s390x: Expand tcg_global_mem_new() -> tcg_global_mem_new_i64()

Series queued, thanks.