>>> 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". Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel