mbox series

[v2,00/12] exec: Rework around CPUState user fields

Message ID 20240428214915.10339-1-philmd@linaro.org
Headers show
Series exec: Rework around CPUState user fields | expand

Message

Philippe Mathieu-Daudé April 28, 2024, 9:49 p.m. UTC 
Missing review: 8 & 9 (new)

v2:
- Addressed Richard review comments
- Renamed page-prot-common.h -> page-protection.h
- Removed wrong/unuseful patches
- New patch moving cpu_loop_exit_requested()

Hi,

First batch of patches (I expect them to be non
controversial) related to extracting user specific
fields from CPUState.

Regards,

Phil.

Philippe Mathieu-Daudé (12):
  plugins: Update stale comment
  plugins/api: Only include 'exec/ram_addr.h' with system emulation
  exec: Include missing license in 'exec/cpu-common.h'
  exec/cpu: Indent TARGET_PAGE_foo definitions
  exec/cpu: Remove obsolete PAGE_RESERVED definition
  exec/cpu: Remove duplicated PAGE_PASSTHROUGH definition
  exec/cpu: Extract page-protection definitions to page-protection.h
  accel/tcg: Use cpu_loop_exit_requested() in cpu_loop_exec_tb()
  accel/tcg: Restrict cpu_loop_exit_requested() to TCG
  accel/tcg: Remove pointless initialization of cflags_next_tb
  accel/tcg: Reset TCG specific fields in tcg_cpu_reset_hold()
  accel/tcg: Access tcg_cflags with getter / setter

 MAINTAINERS                             |  1 +
 accel/tcg/internal-common.h             |  3 +-
 bsd-user/bsd-mem.h                      |  1 +
 bsd-user/qemu.h                         |  1 +
 include/exec/cpu-all.h                  | 36 ++++++++-----------
 include/exec/cpu-common.h               | 47 ++++++++-----------------
 include/exec/exec-all.h                 | 20 -----------
 include/exec/page-protection.h          | 41 +++++++++++++++++++++
 include/exec/translate-all.h            | 20 +++++++++++
 include/semihosting/uaccess.h           |  1 +
 target/arm/cpu.h                        |  1 +
 target/ppc/internal.h                   |  1 +
 target/ppc/mmu-radix64.h                |  2 ++
 accel/tcg/cpu-exec.c                    | 17 ++++++---
 accel/tcg/cputlb.c                      |  1 +
 accel/tcg/tb-maint.c                    |  1 +
 accel/tcg/tcg-accel-ops.c               |  5 ++-
 accel/tcg/user-exec.c                   |  1 +
 bsd-user/mmap.c                         |  1 +
 bsd-user/signal.c                       |  1 +
 cpu-target.c                            |  1 +
 hw/core/cpu-common.c                    |  3 --
 hw/ppc/ppc440_bamboo.c                  |  1 +
 hw/ppc/sam460ex.c                       |  1 +
 hw/ppc/virtex_ml507.c                   |  1 +
 linux-user/arm/cpu_loop.c               |  1 +
 linux-user/elfload.c                    |  1 +
 linux-user/mmap.c                       |  9 ++---
 linux-user/signal.c                     |  1 +
 linux-user/syscall.c                    |  5 +--
 plugins/api.c                           |  2 +-
 plugins/core.c                          |  2 +-
 system/physmem.c                        |  1 +
 target/alpha/helper.c                   |  1 +
 target/arm/cpu.c                        |  2 +-
 target/arm/ptw.c                        |  1 +
 target/arm/tcg/helper-a64.c             |  1 +
 target/arm/tcg/m_helper.c               |  1 +
 target/arm/tcg/mte_helper.c             |  1 +
 target/arm/tcg/sve_helper.c             |  1 +
 target/avr/cpu.c                        |  2 +-
 target/avr/helper.c                     |  1 +
 target/cris/mmu.c                       |  1 +
 target/hexagon/cpu.c                    |  2 +-
 target/hppa/cpu.c                       |  2 +-
 target/hppa/mem_helper.c                |  1 +
 target/hppa/translate.c                 |  1 +
 target/i386/cpu.c                       |  2 +-
 target/i386/helper.c                    |  2 +-
 target/i386/tcg/sysemu/excp_helper.c    |  1 +
 target/loongarch/cpu.c                  |  2 +-
 target/loongarch/tcg/tlb_helper.c       |  1 +
 target/m68k/helper.c                    |  1 +
 target/microblaze/cpu.c                 |  2 +-
 target/microblaze/helper.c              |  1 +
 target/microblaze/mmu.c                 |  1 +
 target/mips/sysemu/physaddr.c           |  1 +
 target/mips/tcg/exception.c             |  2 +-
 target/mips/tcg/sysemu/special_helper.c |  2 +-
 target/mips/tcg/sysemu/tlb_helper.c     |  1 +
 target/openrisc/cpu.c                   |  2 +-
 target/openrisc/mmu.c                   |  1 +
 target/ppc/mmu-hash32.c                 |  1 +
 target/ppc/mmu-hash64.c                 |  1 +
 target/ppc/mmu-radix64.c                |  1 +
 target/ppc/mmu_common.c                 |  1 +
 target/ppc/mmu_helper.c                 |  1 +
 target/riscv/cpu_helper.c               |  1 +
 target/riscv/pmp.c                      |  1 +
 target/riscv/tcg/tcg-cpu.c              |  4 +--
 target/riscv/vector_helper.c            |  1 +
 target/rx/cpu.c                         |  3 +-
 target/s390x/mmu_helper.c               |  1 +
 target/s390x/tcg/mem_helper.c           |  2 ++
 target/sh4/cpu.c                        |  4 +--
 target/sh4/helper.c                     |  1 +
 target/sparc/cpu.c                      |  2 +-
 target/sparc/ldst_helper.c              |  1 +
 target/sparc/mmu_helper.c               |  1 +
 target/tricore/cpu.c                    |  2 +-
 target/tricore/helper.c                 |  1 +
 target/xtensa/mmu_helper.c              |  1 +
 target/xtensa/op_helper.c               |  1 +
 83 files changed, 193 insertions(+), 110 deletions(-)
 create mode 100644 include/exec/page-protection.h

Comments

Philippe Mathieu-Daudé April 29, 2024, 12:02 p.m. UTC | #1 
On 28/4/24 23:49, Philippe Mathieu-Daudé wrote:

> Philippe Mathieu-Daudé (12):
>    plugins: Update stale comment
>    plugins/api: Only include 'exec/ram_addr.h' with system emulation
>    exec: Include missing license in 'exec/cpu-common.h'
>    exec/cpu: Indent TARGET_PAGE_foo definitions
>    exec/cpu: Remove obsolete PAGE_RESERVED definition
>    exec/cpu: Remove duplicated PAGE_PASSTHROUGH definition
>    exec/cpu: Extract page-protection definitions to page-protection.h
>    accel/tcg: Use cpu_loop_exit_requested() in cpu_loop_exec_tb()
>    accel/tcg: Restrict cpu_loop_exit_requested() to TCG
>    accel/tcg: Remove pointless initialization of cflags_next_tb
>    accel/tcg: Reset TCG specific fields in tcg_cpu_reset_hold()
>    accel/tcg: Access tcg_cflags with getter / setter

Thanks, series queued.