>>> On 22.02.16 at 06:35, <shuai.r...@linux.intel.com> wrote:
> --- a/xen/arch/x86/hvm/hvm.c
> +++ b/xen/arch/x86/hvm/hvm.c
> @@ -4703,7 +4703,19 @@ void hvm_cpuid(unsigned int input, unsigned int *eax,
> unsigned int *ebx,
> for ( sub_leaf = 2; sub_leaf < 63; sub_leaf++ )
> if ( (v->arch.xcr0 | v->arch.hvm_vcpu.msr_xss) &
> (1ULL << sub_leaf) )
> + {
> + domain_cpuid(d, input, sub_leaf, &_eax, &_ebx,
> + &_ecx, &_edx);
> + /*
> + * The value return by _ecx[1] indicates the
> + * alignment of the state component i when the
> + * compacted format of the extended region of
> + * an xsave area is used.
> + */
> + if (_ecx & XSTATE_ALIGN64)
> + *ebx = ROUNDUP(*ebx, 64);
> *ebx += xstate_sizes[sub_leaf];
> + }
> }
Besides the various coding style issues I wonder how you get
away without any similar adjustment to pv_cpuid().
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
