>>> 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

Reply via email to