Re: [Xen-devel] [PATCH] x86: properly macroize the two XRSTOR flavors

2015-11-27 Thread Jan Beulich
>>> On 27.11.15 at 14:49, wrote: > On 27/11/15 11:54, Jan Beulich wrote: >> switch ( __builtin_expect(ptr->fpu_sse.x[FPU_WORD_SIZE_OFFSET], 8) ) >> { >> +#define XRSTOR(pfx) \ >> +alternative_io("1: .byte " pfx "0x0f,0xae,0x2f\n" \ >> + " .section .fixup,\

Re: [Xen-devel] [PATCH] x86: properly macroize the two XRSTOR flavors

2015-11-27 Thread Andrew Cooper
On 27/11/15 11:54, Jan Beulich wrote: > All they differ by is the REX64 prefix. Create a single macro covering > both, at once allowing to get rid of the disconnect between the current > partial macro and its two use sites. > > No change in generated code. > > Signed-off-by: Jan Beulich > > --- a/

[Xen-devel] [PATCH] x86: properly macroize the two XRSTOR flavors

2015-11-27 Thread Jan Beulich
All they differ by is the REX64 prefix. Create a single macro covering both, at once allowing to get rid of the disconnect between the current partial macro and its two use sites. No change in generated code. Signed-off-by: Jan Beulich --- a/xen/arch/x86/xstate.c +++ b/xen/arch/x86/xstate.c @@