[Qemu-devel] [PATCH v8 00/13] More fully implement ARM PMUv3

Aaron Lindsay posted 13 patches 5 years, 4 months ago
Test asan passed
Test checkpatch passed
Test docker-quick@centos7 passed
Test docker-mingw@fedora passed
Test docker-clang@ubuntu passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20181120212553.8480-1-aaron@os.amperecomputing.com
There is a newer version of this series
docs/devel/migration.rst    |   9 +-
include/migration/vmstate.h |   1 +
migration/vmstate.c         |  13 +-
target/arm/cpu.c            |  28 +-
target/arm/cpu.h            |  71 +++-
target/arm/cpu64.c          |   4 -
target/arm/helper.c         | 801 ++++++++++++++++++++++++++++++++----
target/arm/machine.c        |  24 ++
8 files changed, 846 insertions(+), 105 deletions(-)
[Qemu-devel] [PATCH v8 00/13] More fully implement ARM PMUv3
Posted by Aaron Lindsay 5 years, 4 months ago
The ARM PMU implementation currently contains a basic cycle counter, but
it is often useful to gather counts of other events, filter them based
on execution mode, and/or be notified on counter overflow. These patches
flesh out the implementations of various PMU registers including
PM[X]EVCNTR and PM[X]EVTYPER, add a struct definition to represent
arbitrary counter types, implement mode filtering, send interrupts on
counter overflow, and add instruction, cycle, and software increment
events.

Since v7 [1] I have made the following changes:
* Updated pmu_op_start/finish to not be called upon migration under KVM
* Added a patch updating PMCEID* to be 64-bit registers for AArch64 and
  added PMCEID2/3 for AArch32
* Changed get_pmceid() return two full 64-bit register values (since the
  underlying registers are now 64-bit post-ARMv8.1)
* Updated the comment for pmu_op_start/finish to match the
  implementation
* Added a check so the pmu_timer is not freed if it was not originally
  allocated

[1] - https://lists.gnu.org/archive/html/qemu-devel/2018-11/msg00837.html

Aaron Lindsay (13):
  migration: Add post_save function to VMStateDescription
  target/arm: Reorganize PMCCNTR accesses
  target/arm: Swap PMU values before/after migrations
  target/arm: Filter cycle counter based on PMCCFILTR_EL0
  target/arm: Allow AArch32 access for PMCCFILTR
  target/arm: Implement PMOVSSET
  target-arm: Make PMCEID[01]_EL0 64 bit registers, add PMCEID[23]
  target/arm: Add array for supported PMU events, generate
    PMCEID[01]_EL0
  target/arm: Finish implementation of PM[X]EVCNTR and PM[X]EVTYPER
  target/arm: PMU: Add instruction and cycle events
  target/arm: PMU: Set PMCR.N to 4
  target/arm: Implement PMSWINC
  target/arm: Send interrupts on PMU counter overflow

 docs/devel/migration.rst    |   9 +-
 include/migration/vmstate.h |   1 +
 migration/vmstate.c         |  13 +-
 target/arm/cpu.c            |  28 +-
 target/arm/cpu.h            |  71 +++-
 target/arm/cpu64.c          |   4 -
 target/arm/helper.c         | 801 ++++++++++++++++++++++++++++++++----
 target/arm/machine.c        |  24 ++
 8 files changed, 846 insertions(+), 105 deletions(-)

-- 
2.19.1