Re: [Xen-devel] [PATCH 1/2] x86/VPMU: Support only versions 2 and 3 of architectural performance monitoring

2015-12-01 Thread Dietmar Hahn
Am Montag 30 November 2015, 13:38:40 schrieb Boris Ostrovsky: > We need to have at least version 2 since it's the first version to > support various control and status registers (such as > MSR_CORE_PERF_GLOBAL_CTRL) that VPMU relies on always having. > > With explicit testing for PMU version we

Re: [Xen-devel] [PATCH 1/2] x86/VPMU: Support only versions 2 and 3 of architectural performance monitoring

2015-12-01 Thread Jan Beulich
>>> On 30.11.15 at 19:38, wrote: > --- > xen/arch/x86/cpu/vpmu_intel.c | 55 > +++ > 1 file changed, 8 insertions(+), 47 deletions(-) Considering the history of the file, I think the VMX maintainers should be Cc-ed here. I

Re: [Xen-devel] [PATCH 1/2] x86/VPMU: Support only versions 2 and 3 of architectural performance monitoring

2015-12-01 Thread Boris Ostrovsky
On 12/01/2015 04:01 AM, Dietmar Hahn wrote: Am Montag 30 November 2015, 13:38:40 schrieb Boris Ostrovsky: We need to have at least version 2 since it's the first version to support various control and status registers (such as MSR_CORE_PERF_GLOBAL_CTRL) that VPMU relies on always having. With

[Xen-devel] [PATCH 1/2] x86/VPMU: Support only versions 2 and 3 of architectural performance monitoring

2015-11-30 Thread Boris Ostrovsky
We need to have at least version 2 since it's the first version to support various control and status registers (such as MSR_CORE_PERF_GLOBAL_CTRL) that VPMU relies on always having. With explicit testing for PMU version we can now remove CPUID model check. Signed-off-by: Boris Ostrovsky