On 30.07.2019 11:39, Paul Durrant wrote:
> --- a/xen/common/domain.c
> +++ b/xen/common/domain.c
> @@ -313,11 +313,19 @@ static int sanitise_domain_config(struct
> xen_domctl_createdomain *config)
> return -EINVAL;
> }
>
> - if ( !(config->flags & XEN_DOMCTL_CDF_hvm_guest) &&
> - config->flags & XEN_DOMCTL_CDF_hap )
> + if ( !(config->flags & XEN_DOMCTL_CDF_hvm_guest) )
> {
> - dprintk(XENLOG_INFO, "HAP enabled for non-HVM guest\n");
> - return -EINVAL;
> + if ( config->flags & XEN_DOMCTL_CDF_hap )
> + {
> + dprintk(XENLOG_INFO, "HAP enabled for non-HVM guest\n");
> + return -EINVAL;
> + }
> +
> + /*
> + * It is only meaningful for XEN_DOMCTL_CDF_oos_off to be clear
> + * for HVM guests.
> + */
> + config->flags |= XEN_DOMCTL_CDF_oos_off;
> }
I'm not convinced this needs to be in common code, but I also
don't mind, so
Reviewed-by: Jan Beulich <jbeul...@suse.com>
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
