Re: [Xen-devel] [PATCH 2/2] x86/HVM: don't calculate XSTATE area sizes in software

2016-06-01 Thread Wei Liu
On Wed, Jun 01, 2016 at 09:50:10AM -0600, Jan Beulich wrote: > >>> On 01.06.16 at 17:35, wrote: > > On 01/06/16 16:06, Jan Beulich wrote: > >> @@ -3440,42 +3440,24 @@ void hvm_cpuid(unsigned int input, unsig > >> *eax = *ebx = *ecx = *edx = 0; > >> break; > >> }

Re: [Xen-devel] [PATCH 2/2] x86/HVM: don't calculate XSTATE area sizes in software

2016-06-01 Thread Jan Beulich
>>> On 01.06.16 at 17:35, wrote: > On 01/06/16 16:06, Jan Beulich wrote: >> @@ -3440,42 +3440,24 @@ void hvm_cpuid(unsigned int input, unsig >> *eax = *ebx = *ecx = *edx = 0; >> break; >> } >> -/* EBX value of main leaf 0 depends on enabled xsave features

Re: [Xen-devel] [PATCH 2/2] x86/HVM: don't calculate XSTATE area sizes in software

2016-06-01 Thread Wei Liu
On Wed, Jun 01, 2016 at 04:35:44PM +0100, Andrew Cooper wrote: > On 01/06/16 16:06, Jan Beulich wrote: > > @@ -3440,42 +3440,24 @@ void hvm_cpuid(unsigned int input, unsig > > *eax = *ebx = *ecx = *edx = 0; > > break; > > } > > -/* EBX value of main leaf 0

Re: [Xen-devel] [PATCH 2/2] x86/HVM: don't calculate XSTATE area sizes in software

2016-06-01 Thread Andrew Cooper
On 01/06/16 16:06, Jan Beulich wrote: > @@ -3440,42 +3440,24 @@ void hvm_cpuid(unsigned int input, unsig > *eax = *ebx = *ecx = *edx = 0; > break; > } > -/* EBX value of main leaf 0 depends on enabled xsave features */ > -if ( count == 0 && v->arch

[Xen-devel] [PATCH 2/2] x86/HVM: don't calculate XSTATE area sizes in software

2016-06-01 Thread Jan Beulich
Use hardware output instead, brining HVM behavior in line with PV one in this regard. Signed-off-by: Jan Beulich --- a/xen/arch/x86/hvm/hvm.c +++ b/xen/arch/x86/hvm/hvm.c @@ -3362,7 +3362,7 @@ void hvm_cpuid(unsigned int input, unsig switch ( input ) { -unsigned int sub_leaf,