[v2] target/cpu: System/User cleanups around hwaddr/vaddr

[PATCH-for-8.0 v2 0/4] target/cpu: System/User cleanups around hwaddr/vaddr

Philippe Mathieu-Daudé posted 4 patches 1 year, 4 months ago

Diff against v1 v3
Download series mbox

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

Maintainers: Paolo Bonzini <pbonzini@redhat.com>, Richard Henderson <richard.henderson@linaro.org>, "Alex Bennée" <alex.bennee@linaro.org>, "Philippe Mathieu-Daudé" <philmd@linaro.org>, Eduardo Habkost <eduardo@habkost.net>, Marcel Apfelbaum <marcel.apfelbaum@gmail.com>, Yanan Wang <wangyanan55@huawei.com>, "Edgar E. Iglesias" <edgar.iglesias@gmail.com>, Laurent Vivier <laurent@vivier.eu>, Chris Wulff <crwulff@gmail.com>, Marek Vasut <marex@denx.de>, Stafford Horne <shorne@gmail.com>, Daniel Henrique Barboza <danielhb413@gmail.com>, "Cédric Le Goater" <clg@kaod.org>, David Gibson <david@gibson.dropbear.id.au>, Greg Kurz <groug@kaod.org>, Yoshinori Sato <ysato@users.sourceforge.jp>, Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>, Artyom Tarasenko <atar4qemu@gmail.com>, Max Filippov <jcmvbkbc@gmail.com>

There is a newer version of this series

accel/kvm/kvm-all.c        | 4 ++--
accel/kvm/kvm-cpus.h       | 4 ++--
accel/tcg/tcg-accel-ops.c  | 4 ++--
gdbstub/gdbstub.c          | 1 -
gdbstub/internals.h        | 6 ++++--
gdbstub/softmmu.c          | 5 ++---
gdbstub/user.c             | 5 ++---
include/hw/core/cpu.h      | 6 ++++--
include/sysemu/accel-ops.h | 6 +++---
target/alpha/cpu.h         | 2 +-
target/cris/cpu.h          | 3 +--
target/hppa/cpu.h          | 2 +-
target/m68k/cpu.h          | 2 +-
target/nios2/cpu.h         | 2 +-
target/openrisc/cpu.h      | 3 ++-
target/ppc/cpu.h           | 2 +-
target/rx/cpu.h            | 2 +-
target/rx/helper.c         | 4 ++--
target/sh4/cpu.h           | 2 +-
target/sparc/cpu.h         | 3 ++-
target/sparc/mmu_helper.c  | 2 --
target/xtensa/cpu.h        | 2 +-
22 files changed, 36 insertions(+), 36 deletions(-)

Expand all Fold all

[PATCH-for-8.0 v2 0/4] target/cpu: System/User cleanups around hwaddr/vaddr

Posted by Philippe Mathieu-Daudé 1 year, 4 months ago

We are not supposed to use the 'hwaddr' type on user emulation.

This series is a preparatory cleanup before few refactors to
isolate further System vs User code.

Since v1:
- only restrict SavedIOTLB in header (Alex)
- convert insert/remove_breakpoint implementations (Peter)

Philippe Mathieu-Daudé (4):
  cputlb: Restrict SavedIOTLB to system emulation
  gdbstub: Use vaddr type for generic insert/remove_breakpoint() API
  target/cpu: Restrict cpu_get_phys_page_debug() handlers to sysemu
  target/sparc/sysemu: Remove pointless CONFIG_USER_ONLY guard

 accel/kvm/kvm-all.c        | 4 ++--
 accel/kvm/kvm-cpus.h       | 4 ++--
 accel/tcg/tcg-accel-ops.c  | 4 ++--
 gdbstub/gdbstub.c          | 1 -
 gdbstub/internals.h        | 6 ++++--
 gdbstub/softmmu.c          | 5 ++---
 gdbstub/user.c             | 5 ++---
 include/hw/core/cpu.h      | 6 ++++--
 include/sysemu/accel-ops.h | 6 +++---
 target/alpha/cpu.h         | 2 +-
 target/cris/cpu.h          | 3 +--
 target/hppa/cpu.h          | 2 +-
 target/m68k/cpu.h          | 2 +-
 target/nios2/cpu.h         | 2 +-
 target/openrisc/cpu.h      | 3 ++-
 target/ppc/cpu.h           | 2 +-
 target/rx/cpu.h            | 2 +-
 target/rx/helper.c         | 4 ++--
 target/sh4/cpu.h           | 2 +-
 target/sparc/cpu.h         | 3 ++-
 target/sparc/mmu_helper.c  | 2 --
 target/xtensa/cpu.h        | 2 +-
 22 files changed, 36 insertions(+), 36 deletions(-)

-- 
2.38.1

Re: [PATCH-for-8.0 v2 0/4] target/cpu: System/User cleanups around hwaddr/vaddr

Posted by Richard Henderson 1 year, 4 months ago

On 12/8/22 09:35, Philippe Mathieu-Daudé wrote:
> Philippe Mathieu-Daudé (4):
>    cputlb: Restrict SavedIOTLB to system emulation
>    gdbstub: Use vaddr type for generic insert/remove_breakpoint() API
>    target/cpu: Restrict cpu_get_phys_page_debug() handlers to sysemu
>    target/sparc/sysemu: Remove pointless CONFIG_USER_ONLY guard

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


r~