On 29/09/16 14:08, Jan Beulich wrote: > Commit a3db233ede ("x86emul: use DstEax also for {,I}{MUL,DIV}") went > a little too far: DstEax and SrcEax weren't really meant to be used > together with ModRM - they assume modrm_reg remains zero by the time > the destination / source register pointer gets calculated. Don't fully > undo that commit though, but instead just correct the register pointer, > and don't use dst.val as input for mul and imul (div and idiv did avoid > that already). > > Reported-by: Konrad Rzeszutek Wilk <konrad.w...@oracle.com> > Signed-off-by: Jan Beulich <jbeul...@suse.com>
Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com> _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel