[Xen-changelog] [xen-unstable] libxc: do not lock VCPU context in xc_ia64_pv_recv_vcpu_context

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

# HG changeset patch
# User Ian Campbell <ian.campbell@xxxxxxxxxx>
# Date 1287419984 -3600
# Node ID aebd6083796c7a5096fde71f00c61f185ac20aa5
# Parent  ad0219b0b3f876b9a654ac7ed00ab0e1327d35f8
libxc: do not lock VCPU context in xc_ia64_pv_recv_vcpu_context

xc_ia64_pv_recv_vcpu_context does not need to lock the ctxt buffer
since it calls xc_ia64_recv_vcpu_context which calls
xc_vcpu_setcontext which takes care of any necessary bouncing.

Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
Signed-off-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
---
 tools/libxc/ia64/xc_ia64_linux_restore.c |    7 -------
 1 files changed, 7 deletions(-)

diff -r ad0219b0b3f8 -r aebd6083796c tools/libxc/ia64/xc_ia64_linux_restore.c
--- a/tools/libxc/ia64/xc_ia64_linux_restore.c  Mon Oct 18 17:38:15 2010 +0100
+++ b/tools/libxc/ia64/xc_ia64_linux_restore.c  Mon Oct 18 17:39:44 2010 +0100
@@ -246,12 +246,6 @@ xc_ia64_pv_recv_vcpu_context(xc_interfac
     vcpu_guest_context_any_t ctxt_any;
     vcpu_guest_context_t *ctxt = &ctxt_any.c;
 
-    if (lock_pages(&ctxt_any, sizeof(ctxt_any))) {
-        /* needed for build domctl, but might as well do early */
-        ERROR("Unable to lock_pages ctxt");
-        return -1;
-    }
-
     if (xc_ia64_recv_vcpu_context(xch, io_fd, dom, vcpu, &ctxt_any))
         goto out;
 
@@ -264,7 +258,6 @@ xc_ia64_pv_recv_vcpu_context(xc_interfac
     rc = 0;
 
  out:
-    unlock_pages(&ctxt, sizeof(ctxt));
     return rc;
 }
 

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