Re: [Xen-devel] [PATCH] x86emul: drop dead code from SYSENTER handling

2016-12-07 Thread Andrew Cooper
On 07/12/16 14:02, Jan Beulich wrote: > There's no point reading CS - all of the fields get set from scratch > right afterwards. Also correct a wrong comment. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper , although > > --- a/xen/arch/x86/x86_emulate/x86_emulate.c > +++ b/xen/arch/x8

[Xen-devel] [PATCH] x86emul: drop dead code from SYSENTER handling

2016-12-07 Thread Jan Beulich
There's no point reading CS - all of the fields get set from scratch right afterwards. Also correct a wrong comment. Signed-off-by: Jan Beulich --- a/xen/arch/x86/x86_emulate/x86_emulate.c +++ b/xen/arch/x86/x86_emulate/x86_emulate.c @@ -4841,12 +4841,10 @@ x86_emulate( _regs.eflags &