[Xen-changelog] [xen-unstable] Revert 26503:69398345c10e

[Xen patchbot-unstable <patchbot@xxxxxxx>]

# HG changeset patch
# User Jan Beulich <jbeulich@xxxxxxxx>
# Date 1360052167 -3600
# Node ID 4f53ddbee940d58e4559d84e0b9fd1dec5e909fc
# Parent  ff77e84ddfdcd5ab93078d9ee6af7b6c60577408
Revert 26503:69398345c10e

This conflicts with changes done in 26486:7648ef657fe7 and
26489:83a3fa9c8434 (i.e. the code added by them needs adjustment in
order for the change here to be correct).
---


diff -r ff77e84ddfdc -r 4f53ddbee940 xen/arch/x86/hvm/hvm.c
--- a/xen/arch/x86/hvm/hvm.c    Mon Feb 04 12:10:26 2013 +0100
+++ b/xen/arch/x86/hvm/hvm.c    Tue Feb 05 09:16:07 2013 +0100
@@ -3900,25 +3900,20 @@ long do_hvm_op(unsigned long op, XEN_GUE
                     rc = -EPERM;
                     break;
                 }
-                if ( !a.value )
-                    break;
                 if ( a.value > 1 )
                     rc = -EINVAL;
+                if ( !is_hvm_domain(d) )
+                    rc = -EINVAL;
                 /* Remove the check below once we have
                  * shadow-on-shadow.
                  */
-                if ( cpu_has_svm && !paging_mode_hap(d) )
+                if ( cpu_has_svm && !paging_mode_hap(d) && a.value )
                     rc = -EINVAL;
                 /* Set up NHVM state for any vcpus that are already up */
                 if ( !d->arch.hvm_domain.params[HVM_PARAM_NESTEDHVM] )
-                {
                     for_each_vcpu(d, v)
                         if ( rc == 0 )
                             rc = nestedhvm_vcpu_initialise(v);
-                    if ( rc )
-                        for_each_vcpu(d, v)
-                            nestedhvm_vcpu_destroy(v);
-                }
                 break;
             case HVM_PARAM_BUFIOREQ_EVTCHN:
                 rc = -EINVAL;
diff -r ff77e84ddfdc -r 4f53ddbee940 xen/arch/x86/hvm/nestedhvm.c
--- a/xen/arch/x86/hvm/nestedhvm.c      Mon Feb 04 12:10:26 2013 +0100
+++ b/xen/arch/x86/hvm/nestedhvm.c      Tue Feb 05 09:16:07 2013 +0100
@@ -87,7 +87,7 @@ nestedhvm_vcpu_initialise(struct vcpu *v
 void
 nestedhvm_vcpu_destroy(struct vcpu *v)
 {
-    if ( hvm_funcs.nhvm_vcpu_destroy )
+    if ( nestedhvm_enabled(v->domain) && hvm_funcs.nhvm_vcpu_destroy )
         hvm_funcs.nhvm_vcpu_destroy(v);
 }
 

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