All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 0/4] KVM: arm64: Improve PMU support on heterogeneous systems
@ 2021-12-06 17:02 ` Alexandru Elisei
  0 siblings, 0 replies; 52+ messages in thread
From: Alexandru Elisei @ 2021-12-06 17:02 UTC (permalink / raw)
  To: maz, james.morse, suzuki.poulose, will, mark.rutland,
	linux-arm-kernel, kvmarm
  Cc: tglx, mingo

(CC'ing Peter Maydell in case this might be of interest to qemu)

The series can be found on a branch at [1], and the kvmtool support at [2].
The kvmtool patches are also on the mailing list [3] and haven't changed
since v1.

Detailed explanation of the issue and symptoms that the patches attempt to
correct can be found in the cover letter for v1 [4].

A brief summary of the problem is that on heterogeneous systems KVM will
always use the same PMU for creating the VCPU events for *all* VCPUs
regardless of the physical CPU on which the VCPU is running, leading to
events suddenly stopping and resuming in the guest as the VCPU thread gets
migrated across different CPUs.

This series proposes to fix this behaviour by allowing the user to specify
which physical PMU is used when creating the VCPU events needed for guest
PMU emulation. When the PMU is set, KVM will refuse to the VCPU on a
physical which is not part of the supported CPUs for the specified PMU.

The default behaviour stays the same - without userspace setting the PMU,
events will stop counting if the VCPU is scheduled on the wrong CPU.

Changes since v1:

- Rebased on top of v5.16-rc4

- Implemented review comments: protect iterating through the list of PMUs
  with a mutex, documentation changes, initialize vcpu-arch.supported_cpus
  to cpu_possible_mask, changed vcpu->arch.cpu_not_supported to a VCPU
  flag, set exit reason to KVM_EXIT_FAIL_ENTRY and populate fail_entry when
  the VCPU is run on a CPU not in the PMU's supported cpumask. Many thanks
  for the review!

[1] https://gitlab.arm.com/linux-arm/linux-ae/-/tree/pmu-big-little-fix-v2
[2] https://gitlab.arm.com/linux-arm/kvmtool-ae/-/tree/pmu-big-little-fix-v1
[3] https://www.spinics.net/lists/arm-kernel/msg933584.html
[4] https://www.spinics.net/lists/arm-kernel/msg933579.html

Alexandru Elisei (4):
  perf: Fix wrong name in comment for struct perf_cpu_context
  KVM: arm64: Keep a list of probed PMUs
  KVM: arm64: Add KVM_ARM_VCPU_PMU_V3_SET_PMU attribute
  KVM: arm64: Refuse to run VCPU if the PMU doesn't match the physical
    CPU

 Documentation/virt/kvm/devices/vcpu.rst | 29 +++++++++++
 arch/arm64/include/asm/kvm_host.h       | 12 +++++
 arch/arm64/include/uapi/asm/kvm.h       |  4 ++
 arch/arm64/kvm/arm.c                    | 19 ++++++++
 arch/arm64/kvm/pmu-emul.c               | 64 +++++++++++++++++++++++--
 include/kvm/arm_pmu.h                   |  6 +++
 include/linux/perf_event.h              |  2 +-
 tools/arch/arm64/include/uapi/asm/kvm.h |  1 +
 8 files changed, 132 insertions(+), 5 deletions(-)

-- 
2.34.1

_______________________________________________
kvmarm mailing list
kvmarm@lists.cs.columbia.edu
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm

^ permalink raw reply	[flat|nested] 52+ messages in thread

end of thread, other threads:[~2021-12-15  6:57 UTC | newest]

Thread overview: 52+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-06 17:02 [PATCH v2 0/4] KVM: arm64: Improve PMU support on heterogeneous systems Alexandru Elisei
2021-12-06 17:02 ` Alexandru Elisei
2021-12-06 17:02 ` [PATCH v2 1/4] perf: Fix wrong name in comment for struct perf_cpu_context Alexandru Elisei
2021-12-06 17:02   ` Alexandru Elisei
2021-12-06 17:02 ` [PATCH v2 2/4] KVM: arm64: Keep a list of probed PMUs Alexandru Elisei
2021-12-06 17:02   ` Alexandru Elisei
2021-12-06 17:02 ` [PATCH v2 3/4] KVM: arm64: Add KVM_ARM_VCPU_PMU_V3_SET_PMU attribute Alexandru Elisei
2021-12-06 17:02   ` Alexandru Elisei
2021-12-08  3:13   ` Reiji Watanabe
2021-12-08  3:13     ` Reiji Watanabe
2021-12-08 12:23     ` Alexandru Elisei
2021-12-08 12:23       ` Alexandru Elisei
2021-12-08 12:43       ` Alexandru Elisei
2021-12-08 12:43         ` Alexandru Elisei
2021-12-08 14:25       ` Marc Zyngier
2021-12-08 14:25         ` Marc Zyngier
2021-12-08 15:20         ` Alexandru Elisei
2021-12-08 15:20           ` Alexandru Elisei
2021-12-08 15:44           ` Marc Zyngier
2021-12-08 15:44             ` Marc Zyngier
2021-12-08 16:11             ` Alexandru Elisei
2021-12-08 16:11               ` Alexandru Elisei
2021-12-08 16:21               ` Marc Zyngier
2021-12-08 16:21                 ` Marc Zyngier
2021-12-06 17:02 ` [PATCH v2 4/4] KVM: arm64: Refuse to run VCPU if the PMU doesn't match the physical CPU Alexandru Elisei
2021-12-06 17:02   ` Alexandru Elisei
2021-12-07 14:17   ` Alexandru Elisei
2021-12-07 14:17     ` Alexandru Elisei
2021-12-08  7:54     ` Reiji Watanabe
2021-12-08  7:54       ` Reiji Watanabe
2021-12-08 10:38       ` Alexandru Elisei
2021-12-08 10:38         ` Alexandru Elisei
2021-12-13  7:40         ` Reiji Watanabe
2021-12-13  7:40           ` Reiji Watanabe
2021-12-08  9:56     ` Marc Zyngier
2021-12-08  9:56       ` Marc Zyngier
2021-12-08 11:18       ` Alexandru Elisei
2021-12-08 11:18         ` Alexandru Elisei
2021-12-08  2:36 ` [PATCH v2 0/4] KVM: arm64: Improve PMU support on heterogeneous systems Reiji Watanabe
2021-12-08  2:36   ` Reiji Watanabe
2021-12-08  8:05   ` Marc Zyngier
2021-12-08  8:05     ` Marc Zyngier
2021-12-13  6:36     ` Reiji Watanabe
2021-12-13  6:36       ` Reiji Watanabe
2021-12-13 11:14       ` Alexandru Elisei
2021-12-13 11:14         ` Alexandru Elisei
2021-12-14  6:24         ` Reiji Watanabe
2021-12-14  6:24           ` Reiji Watanabe
2021-12-14 11:56           ` Marc Zyngier
2021-12-14 11:56             ` Marc Zyngier
2021-12-15  6:47             ` Reiji Watanabe
2021-12-15  6:47               ` Reiji Watanabe

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.