KVM: x86/pmu: Introduce the ctrl_mask value for fixed counter
[ Upstream commit 2c985527dd8d283e786ad7a67e532ef7f6f00fac ] The mask value of fixed counter control register should be dynamic adjusted with the number of fixed counters. This patch introduces a variable that includes the reserved bits of fixed counter control registers. This is a generic code refactoring. Co-developed-by: Luwei Kang <luwei.kang@intel.com> Signed-off-by: Luwei Kang <luwei.kang@intel.com> Signed-off-by: Like Xu <like.xu@linux.intel.com> Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org> Message-Id: <20220411101946.20262-6-likexu@tencent.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:

committed by
Greg Kroah-Hartman

parent
2ba1feb143
commit
46ec3d8e90
@@ -432,6 +432,7 @@ struct kvm_pmu {
|
|||||||
unsigned nr_arch_fixed_counters;
|
unsigned nr_arch_fixed_counters;
|
||||||
unsigned available_event_types;
|
unsigned available_event_types;
|
||||||
u64 fixed_ctr_ctrl;
|
u64 fixed_ctr_ctrl;
|
||||||
|
u64 fixed_ctr_ctrl_mask;
|
||||||
u64 global_ctrl;
|
u64 global_ctrl;
|
||||||
u64 global_status;
|
u64 global_status;
|
||||||
u64 global_ovf_ctrl;
|
u64 global_ovf_ctrl;
|
||||||
|
@@ -259,7 +259,7 @@ static int intel_pmu_set_msr(struct kvm_vcpu *vcpu, struct msr_data *msr_info)
|
|||||||
case MSR_CORE_PERF_FIXED_CTR_CTRL:
|
case MSR_CORE_PERF_FIXED_CTR_CTRL:
|
||||||
if (pmu->fixed_ctr_ctrl == data)
|
if (pmu->fixed_ctr_ctrl == data)
|
||||||
return 0;
|
return 0;
|
||||||
if (!(data & 0xfffffffffffff444ull)) {
|
if (!(data & pmu->fixed_ctr_ctrl_mask)) {
|
||||||
reprogram_fixed_counters(pmu, data);
|
reprogram_fixed_counters(pmu, data);
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
@@ -326,6 +326,7 @@ static void intel_pmu_refresh(struct kvm_vcpu *vcpu)
|
|||||||
struct kvm_cpuid_entry2 *entry;
|
struct kvm_cpuid_entry2 *entry;
|
||||||
union cpuid10_eax eax;
|
union cpuid10_eax eax;
|
||||||
union cpuid10_edx edx;
|
union cpuid10_edx edx;
|
||||||
|
int i;
|
||||||
|
|
||||||
pmu->nr_arch_gp_counters = 0;
|
pmu->nr_arch_gp_counters = 0;
|
||||||
pmu->nr_arch_fixed_counters = 0;
|
pmu->nr_arch_fixed_counters = 0;
|
||||||
@@ -334,6 +335,7 @@ static void intel_pmu_refresh(struct kvm_vcpu *vcpu)
|
|||||||
pmu->version = 0;
|
pmu->version = 0;
|
||||||
pmu->reserved_bits = 0xffffffff00200000ull;
|
pmu->reserved_bits = 0xffffffff00200000ull;
|
||||||
pmu->raw_event_mask = X86_RAW_EVENT_MASK;
|
pmu->raw_event_mask = X86_RAW_EVENT_MASK;
|
||||||
|
pmu->fixed_ctr_ctrl_mask = ~0ull;
|
||||||
|
|
||||||
entry = kvm_find_cpuid_entry(vcpu, 0xa, 0);
|
entry = kvm_find_cpuid_entry(vcpu, 0xa, 0);
|
||||||
if (!entry)
|
if (!entry)
|
||||||
@@ -367,6 +369,8 @@ static void intel_pmu_refresh(struct kvm_vcpu *vcpu)
|
|||||||
((u64)1 << edx.split.bit_width_fixed) - 1;
|
((u64)1 << edx.split.bit_width_fixed) - 1;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
for (i = 0; i < pmu->nr_arch_fixed_counters; i++)
|
||||||
|
pmu->fixed_ctr_ctrl_mask &= ~(0xbull << (i * 4));
|
||||||
pmu->global_ctrl = ((1ull << pmu->nr_arch_gp_counters) - 1) |
|
pmu->global_ctrl = ((1ull << pmu->nr_arch_gp_counters) - 1) |
|
||||||
(((1ull << pmu->nr_arch_fixed_counters) - 1) << INTEL_PMC_IDX_FIXED);
|
(((1ull << pmu->nr_arch_fixed_counters) - 1) << INTEL_PMC_IDX_FIXED);
|
||||||
pmu->global_ctrl_mask = ~pmu->global_ctrl;
|
pmu->global_ctrl_mask = ~pmu->global_ctrl;
|
||||||
|
Reference in New Issue
Block a user