mirror of
https://github.com/xen-project/xen.git
synced 2026-10-28 02:01:24 +01:00
x86/vpmu: Expose up to 8 Intel event selectors in PV Dom0
Most (if not all) Intel CPUs starting from Sandy Bridge have up to 8
event selectors per core, which could be halved per hyperthread.
However, current PV emulation logic doesn't support up to 8 event selector,
leading to errors when trying to access them, hence, preventing Linux from
driving the vPMU correctly.
Make sure up to MSR_P6_EVNTSEL(7) is usable, which is the same upper bound as
used in VMX code.
The check if the event selector actually exist for the hardware is done in
core2_vpmu_do_{rdmsr,wrmsr}, hence we're not allowing to access non-existent
MSRs.
Fixes: 27c5541986 ("x86/VPMU: add support for PMU register handling on PV guests")
Signed-off-by: Teddy Astie <teddy.astie@vates.tech>
Reviewed-by: Jan Beulich <jbeulich@suse.com>
This commit is contained in:
1 parent
efacf86685
commit
4e9d9e7cc9
1 file changed
+2
-2
@@ -992,7 +992,7 @@ static int cf_check read_msr(
|
||||
return X86EMUL_OKAY;
|
||||
|
||||
case MSR_P6_PERFCTR(0) ... MSR_P6_PERFCTR(7):
|
||||
case MSR_P6_EVNTSEL(0) ... MSR_P6_EVNTSEL(3):
|
||||
case MSR_P6_EVNTSEL(0) ... MSR_P6_EVNTSEL(7):
|
||||
case MSR_CORE_PERF_FIXED_CTR0 ... MSR_CORE_PERF_FIXED_CTR2:
|
||||
case MSR_CORE_PERF_FIXED_CTR_CTRL ... MSR_CORE_PERF_GLOBAL_OVF_CTRL:
|
||||
if ( boot_cpu_data.vendor == X86_VENDOR_INTEL )
|
||||
@@ -1175,7 +1175,7 @@ static int cf_check write_msr(
|
||||
break;
|
||||
|
||||
case MSR_P6_PERFCTR(0) ... MSR_P6_PERFCTR(7):
|
||||
case MSR_P6_EVNTSEL(0) ... MSR_P6_EVNTSEL(3):
|
||||
case MSR_P6_EVNTSEL(0) ... MSR_P6_EVNTSEL(7):
|
||||
case MSR_CORE_PERF_FIXED_CTR0 ... MSR_CORE_PERF_FIXED_CTR2:
|
||||
case MSR_CORE_PERF_FIXED_CTR_CTRL ... MSR_CORE_PERF_GLOBAL_OVF_CTRL:
|
||||
if ( boot_cpu_data.vendor == X86_VENDOR_INTEL )
|
||||
|
||||