mbox series

[kvmtool,0/9] arm64: Improve PMU support on heterogeneous systems

Message ID 20211115165705.195736-1-alexandru.elisei@arm.com (mailing list archive)
Headers show
Series arm64: Improve PMU support on heterogeneous systems | expand

Message

Alexandru Elisei Nov. 15, 2021, 4:56 p.m. UTC
The series can be found at [1], and the Linux patches that this series is
based on at [2].

The series adds support for the KVM_ARM_VCPU_PMU_V3_SET_PMU PMU attribute,
which allows userspace to set a PMU for a VCPU. This PMU is used by KVM
when creating perf events to emulate the guest PMU.

Without settings this attribute, the PMU used when creating events is the
first one that successfully probed when booting, but this is unreliable as
the probe order can change (if the order of the PMUs is changed in the DTB
or if asynchronous driver probing is enabled at the host's command line),
and furthermore it requires the user to have intimate knowledge of how the
PMU was chosen in order to pin the VM on the correct physical CPUs.

With KVM_ARM_VCPU_PMU_V3_SET_PMU, the user is still expected to pin the VM
on a particular set of CPUs, but now it can be any CPUs as long as they
share the same PMU. The set does not depend anymore on the driver probe
order and all that is necessary for the user to know is which CPUs are the
little core and which are the big cores, in a big.little configuration,
which I believe is more reasonable.

Patches #1-#5 move the PMU code to aarch64, where it belongs, because the
PMU has never been supported on KVM for arm. This also paves the way for
pulling in the KVM_ARM_VCPU_PMU_V3_SET_PMU attribute, which was not defined
for KVM for arm (when KVM supported arm). This can be merged right now,
independently of the other patches.

Patches #6-#7 add the function cpulist_parse(), copied almost as-is from
Linux v5.15.

Patch #8 pulls all the necessary user API definitions, and finally patch #9
adds support for KVM_ARM_VCPU_PMU_V3_SET_PMU.

[1] https://gitlab.arm.com/linux-arm/kvmtool-ae/-/tree/pmu-big-little-fix-v1
[2] https://gitlab.arm.com/linux-arm/linux-ae/-/tree/pmu-big-little-fix-v1

Alexandru Elisei (9):
  linux/err.h: Add missing stdbool.h include
  arm: Move arch specific VCPU features to the arch specific function
  arm: Get rid of the ARM_VCPU_FEATURE_FLAGS() macro
  arm: Make the PMUv3 emulation code arm64 specific
  arm64: Rework set_pmu_attr()
  util: Add basic cpumask functions
  util: Add cpulist_parse()
  update_headers.sh: Sync headers with Linux v5.16-rc1 + SET_PMU
    attribute
  arm64: Add support for KVM_ARM_VCPU_PMU_V3_SET_PMU

 Makefile                                      |   5 +-
 arm/aarch32/include/asm/kernel.h              |   8 +
 arm/aarch32/include/kvm/kvm-cpu-arch.h        |   4 -
 arm/aarch64/arm-cpu.c                         |   3 +-
 arm/aarch64/include/asm/kernel.h              |   8 +
 arm/aarch64/include/asm/kvm.h                 |  57 ++-
 .../arm-common => aarch64/include/asm}/pmu.h  |   0
 arm/aarch64/include/kvm/kvm-cpu-arch.h        |   6 -
 arm/aarch64/kvm-cpu.c                         |  12 +
 arm/aarch64/pmu.c                             | 201 ++++++++
 arm/kvm-cpu.c                                 |  14 +-
 arm/pmu.c                                     |  76 ---
 include/linux/bitmap.h                        |  45 ++
 include/linux/bitops.h                        |   2 +
 include/linux/cpumask.h                       |  38 ++
 include/linux/err.h                           |   2 +
 include/linux/kernel.h                        |   3 +
 include/linux/kvm.h                           | 441 +++++++++++++++++-
 mips/include/asm/kernel.h                     |   8 +
 powerpc/include/asm/kernel.h                  |   8 +
 powerpc/include/asm/kvm.h                     |  10 +
 util/bitmap.c                                 | 221 +++++++++
 x86/include/asm/kernel.h                      |   8 +
 x86/include/asm/kvm.h                         |  64 ++-
 24 files changed, 1132 insertions(+), 112 deletions(-)
 create mode 100644 arm/aarch32/include/asm/kernel.h
 create mode 100644 arm/aarch64/include/asm/kernel.h
 rename arm/{include/arm-common => aarch64/include/asm}/pmu.h (100%)
 create mode 100644 arm/aarch64/pmu.c
 delete mode 100644 arm/pmu.c
 create mode 100644 include/linux/bitmap.h
 create mode 100644 include/linux/cpumask.h
 create mode 100644 mips/include/asm/kernel.h
 create mode 100644 powerpc/include/asm/kernel.h
 create mode 100644 util/bitmap.c
 create mode 100644 x86/include/asm/kernel.h