Missing review: #1 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) Since v2: - added 'dump' patch - collected R-b tags Philippe Mathieu-Daudé (5): dump: Include missing "cpu.h" header for tswap32/tswap64() declarations 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/cpu: Restrict do_transaction_failed() handlers to sysemu accel/kvm/kvm-all.c | 4 ++-- accel/kvm/kvm-cpus.h | 4 ++-- accel/tcg/tcg-accel-ops.c | 4 ++-- dump/dump.c | 1 + 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/arm/cpu.h | 2 +- target/arm/internals.h | 2 ++ target/cris/cpu.h | 3 +-- target/hppa/cpu.h | 2 +- target/i386/cpu.h | 5 ++--- target/m68k/cpu.h | 4 +++- target/microblaze/cpu.h | 4 ++-- target/nios2/cpu.h | 2 +- target/openrisc/cpu.h | 3 ++- target/ppc/cpu.h | 2 +- target/riscv/cpu.h | 12 ++++++------ target/rx/cpu.h | 2 +- target/rx/helper.c | 4 ++-- target/sh4/cpu.h | 2 +- target/sparc/cpu.h | 3 ++- target/xtensa/cpu.h | 2 +- 27 files changed, 52 insertions(+), 46 deletions(-) -- 2.38.1