[Xen-changelog] [xen-3.0-testing] [XEN] VCPU operation register_runstate should work even when

[Xen patchbot-3.0-testing <patchbot-3.0-testing@xxxxxxxxxxxxxxxxxxx>]

# HG changeset patch
# User kaf24@xxxxxxxxxxxxxxxxxxxx
# Node ID d4312b52c3f88ff4dc4d872f9f0f1c7b556be0e2
# Parent  de0e196e1a045974cb242dff2dbdf75070f400ce
[XEN] VCPU operation register_runstate should work even when
called from non-local VCPU. Necessary as Linux always calls
this operation from VCPU0 during secondary-VCPU bringup.
Signed-off-by: Keir Fraser <keir@xxxxxxxxxxxxx>
xen-unstable changeset:   10314:b8e8b6aad8c8c38d8277c5188e548c64049299f0
xen-unstable date:        Fri Jun  9 17:05:09 2006 +0100
---
 xen/arch/x86/domain.c |    9 ++++-----
 1 files changed, 4 insertions(+), 5 deletions(-)

diff -r de0e196e1a04 -r d4312b52c3f8 xen/arch/x86/domain.c
--- a/xen/arch/x86/domain.c     Fri Jun 09 16:20:36 2006 +0100
+++ b/xen/arch/x86/domain.c     Fri Jun 09 17:06:09 2006 +0100
@@ -467,10 +467,6 @@ arch_do_vcpu_op(
     {
         struct vcpu_register_runstate_memory_area area;
 
-        rc = -EINVAL;
-        if ( v != current )
-            break;
-
         rc = -EFAULT;
         if ( copy_from_guest(&area, arg, 1) )
             break;
@@ -480,7 +476,10 @@ arch_do_vcpu_op(
 
         rc = 0;
         v->runstate_guest = area.addr.v;
-        __copy_to_user(v->runstate_guest, &v->runstate, sizeof(v->runstate));
+
+        if ( v == current )
+            __copy_to_user(v->runstate_guest, &v->runstate,
+                           sizeof(v->runstate));
 
         break;
     }

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