On 02/22/2017 04:55 AM, Jan Beulich wrote: >>>> On 17.02.17 at 18:40, <boris.ostrov...@oracle.com> wrote: >> --- a/xen/arch/x86/cpu/vpmu_intel.c >> +++ b/xen/arch/x86/cpu/vpmu_intel.c >> @@ -884,6 +884,10 @@ int vmx_vpmu_initialise(struct vcpu *v) >> if ( vpmu_mode == XENPMU_MODE_OFF ) >> return 0; >> >> + if ( MASK_EXTR(v->domain->arch.cpuid->basic.raw[0xa].a, >> + PMU_VERSION_MASK) == 0 ) >> + return -EINVAL; > How about other unsupported (too large) values?
Yes, we can check here for version >=5 as well. (I don't think we should make this additional test in update_domain_cpuid_info()) -boris _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
