>>> On 10.08.16 at 17:00, <tamas.leng...@zentific.com> wrote:
> @@ -5238,18 +5238,19 @@ static int do_altp2m_op(
> goto out;
> }
>
> - if ( (rc = xsm_hvm_altp2mhvm_op(XSM_TARGET, d)) )
> + if ( !d->arch.hvm_domain.params[HVM_PARAM_ALTP2M] )
> + {
> + rc = -EINVAL;
> + goto out;
> + }
> +
> + if ( (rc = xsm_hvm_altp2mhvm_op(XSM_OTHER, d,
> + d->arch.hvm_domain.params[HVM_PARAM_ALTP2M])) )
I'm sorry that this didn't occur to me on v1 already, but is there
really a need for passing this extra argument, when the callee
could - if it cared in the first place - read the value itself?
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
