[Xen-changelog] Currently, we mark an IO request to STATE_IOREQ_READY state before

[Xen patchbot -unstable <patchbot-unstable@xxxxxxxxxxxxxxxxxxx>]

# HG changeset patch
# User kaf24@xxxxxxxxxxxxxxxxxxxx
# Node ID 48f542f8d906f988b6d7ffc0c5d44f5a9dfc8a31
# Parent  40b4860f554a91d537aeb021a038ffd4b335a221
Currently, we mark an IO request to STATE_IOREQ_READY state before
calling IO handlers in xen HV, i.e. vmx_portio_intercept or
vmx_mmio_intercept.  Actually, STATE_IOREQ_READY means it's ready for
DM handling, only if IO handlers in xen HV won't handle it, it should
be passed to DM. This patch fixes this issue.

Signed-off-by: Yunhong Jiang <yunhong.jiang@xxxxxxxxx>
Signed-off-by: Xin Li <xin.b.li@xxxxxxxxx>
Signed-off-by: Nakajima Jun <nakajima.jun@xxxxxxxxx>

diff -r 40b4860f554a -r 48f542f8d906 xen/arch/x86/vmx.c
--- a/xen/arch/x86/vmx.c        Wed Oct 12 20:10:14 2005
+++ b/xen/arch/x86/vmx.c        Thu Oct 13 07:06:27 2005
@@ -646,13 +646,13 @@
     } else
         p->u.data = value;
 
-    p->state = STATE_IOREQ_READY;
-
     if (vmx_portio_intercept(p)) {
         /* no blocking & no evtchn notification */
         clear_bit(ARCH_VMX_IO_WAIT, &v->arch.arch_vmx.flags);
         return;
     }
+
+    p->state = STATE_IOREQ_READY;
 
     evtchn_send(iopacket_port(v->domain));
     vmx_wait_io();
diff -r 40b4860f554a -r 48f542f8d906 xen/arch/x86/vmx_platform.c
--- a/xen/arch/x86/vmx_platform.c       Wed Oct 12 20:10:14 2005
+++ b/xen/arch/x86/vmx_platform.c       Thu Oct 13 07:06:27 2005
@@ -643,13 +643,13 @@
     } else
         p->u.data = value;
 
-    p->state = STATE_IOREQ_READY;
-
     if (vmx_mmio_intercept(p)){
         p->state = STATE_IORESP_READY;
         vmx_io_assist(v);
         return;
     }
+
+    p->state = STATE_IOREQ_READY;
 
     evtchn_send(iopacket_port(v->domain));
     vmx_wait_io();

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog