[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] Regression introduced by a3db233 x86emul: use DstEax also for {, I}{MUL, DIV



>>> On 29.09.16 at 11:11, <konrad.wilk@xxxxxxxxxx> wrote:
> The commit a3db233 x86emul: use DstEax also for {,I}{MUL,DIV}
> introduces an regression when doing SR-IOV passthrough.

I'll see if I can repro, but I can't see anything wrong with the
commit (yet). The more that ...

> Specifically I get:
> ..
> (d1) Invoking ROMBIOS ...
> (XEN) stdvga.c:174:d1v0 entering stdvga mode
> (d1) VGABios $Id: vgabios.c,v 1.67 2008/01/27 09:44:12 vruppert Exp $
> (XEN) realmode.c:119:d1v0 Failed to emulate insn.
> (XEN) Real-mode emulation failed: d1v0 Real @ c000:00000154 -> 0f 75 05 e8 b0 
> 5f eb 7c 80 fc 1a 75 05 e8 d3 6a

... we don't emulate opcode 0f 75 at all, and looking at the following
bytes it feels like we're in the middle of an instruction, yet that patch
clearly doesn't alter what size instructions end up being.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.