[Qemu-devel] [PATCH 00/15] v7M: cleanups and bugfixes prior to v8M

Peter Maydell posted 15 patches 6 years, 8 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1501692241-23310-1-git-send-email-peter.maydell@linaro.org
Test FreeBSD passed
Test checkpatch failed
Test docker passed
Test s390x passed
hw/intc/armv7m_nvic.c                  |  68 +++++++++++-------
include/hw/arm/armv7m.h                |   2 +-
include/hw/{arm => intc}/armv7m_nvic.h |   0
target/arm/cpu.c                       |   5 --
target/arm/cpu.h                       |  54 ++++++++++----
target/arm/helper.c                    | 124 ++++++++++++++++-----------------
target/arm/internals.h                 |   3 +-
target/arm/machine.c                   |  54 +++++++++++++-
target/arm/op_helper.c                 |   5 ++
target/arm/translate.c                 | 106 +++++++++++++++++++++-------
10 files changed, 286 insertions(+), 135 deletions(-)
rename include/hw/{arm => intc}/armv7m_nvic.h (100%)
[Qemu-devel] [PATCH 00/15] v7M: cleanups and bugfixes prior to v8M
Posted by Peter Maydell 6 years, 8 months ago
(This is 2.11 material, obviously, but it's a coherent and
large enough set of patches that I figured I might as well
push it out for review now.)

This patchset is a collection of cleanups, bugfixes, etc to
the existing v7M code which are either necessary preliminary
to implementing v8M or just things I noticed along the way.
The non-trivial stuff is:
 * migration for M profile is shifted to not use read_cpsr()
   and write_cpsr() which assume A profile semantics
   (back compatibility with old migration state is maintained)
 * we implement the "user accesses should BusFault" behaviour
   for the memory mapped registers in the SCS, though this
   won't actually kick in until we turn MEMTX_ERROR into a
   BusFault (I have patches for that)

thanks
-- PMM

Peter Maydell (15):
  target/arm: Use MMUAccessType enum rather than int
  target/arm: Don't trap WFI/WFE for M profile
  target/arm: Consolidate PMSA handling in get_phys_addr()
  target/arm: Tighten up Thumb decode where new v8M insns will be
  hw/intc/armv7m_nvic.c: Remove out of date comment
  target/arm: Remove incorrect comment about MPU_CTRL
  target/arm: Fix outdated comment about exception exit
  target/arm: Define and use XPSR bit masks
  target/arm: Don't store M profile PRIMASK and FAULTMASK in daif
  target/arm: Don't use cpsr_write/cpsr_read to transfer M profile XPSR
  target/arm: Make arm_cpu_dump_state() handle the M-profile XPSR
  target/arm: Don't calculate lr in arm_v7m_cpu_do_interrupt() until
    needed
  target/arm: Create and use new function arm_v7m_is_handler_mode()
  armv7m_nvic.h: Move from include/hw/arm to include/hw/intc
  nvic: Implement "user accesses BusFault" SCS region behaviour

 hw/intc/armv7m_nvic.c                  |  68 +++++++++++-------
 include/hw/arm/armv7m.h                |   2 +-
 include/hw/{arm => intc}/armv7m_nvic.h |   0
 target/arm/cpu.c                       |   5 --
 target/arm/cpu.h                       |  54 ++++++++++----
 target/arm/helper.c                    | 124 ++++++++++++++++-----------------
 target/arm/internals.h                 |   3 +-
 target/arm/machine.c                   |  54 +++++++++++++-
 target/arm/op_helper.c                 |   5 ++
 target/arm/translate.c                 | 106 +++++++++++++++++++++-------
 10 files changed, 286 insertions(+), 135 deletions(-)
 rename include/hw/{arm => intc}/armv7m_nvic.h (100%)

-- 
2.7.4