Mohammed Gamal wrote:
As per Avi's recommendation, this patch uses DstReg to autodecode destination
registers when emulating xchg instructions


Applied, thanks. I had to fix some coding style errors, please read Documentation/CodingStyle (or just follow the style in the surrounding code).

+       case 0x91 ... 0x97: /* xchg reg,rax */
+               c->src.type = c->dst.type = OP_REG;
+               c->src.bytes = c->dst.bytes = c->op_bytes;
+               c->src.ptr = (unsigned long *) &c->regs[VCPU_REGS_RAX];
+               c->src.val = *(c->src.ptr);
+               goto xchg;              

How about SrcAcc and DstAcc so this code doesn't have to be repeated? There are many potential use cases.

--
I have a truly marvellous patch that fixes the bug which this
signature is too narrow to contain.

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to