On 06/05/15 05:52, Jan Beulich wrote:
On 22.05.15 at 17:50, <dsl...@verizon.com> wrote:
@@ -5805,6 +5808,12 @@ static int hvmop_set_param(
              break;
          if ( a.value > 1 )
              rc = -EINVAL;
+        /* Prevent nestedhvm with vmport */
+        if ( d->arch.hvm_domain.is_vmware_port_enabled )
+        {
+            rc = -EOPNOTSUPP;
+            break;
+        }
Surrounding code avoiding the use of "break" makes the result look
rather inconsistent. Please move this up immediately after the XSM
check, or drop the "break".

Will do.

   -Don Slutz

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Reply via email to