Re: [Xen-devel] [PATCH v2 02/13] domctl: add tee_op domctl

[Julien Grall <julien.grall@xxxxxxx>]

Hi,

On 03/09/18 17:54, Volodymyr Babchuk wrote:
> Currently this call used only to enable TEE support for a domain.
>
> It is planed to extend it when there will be multiple TEE
> mediators, so toolstack can determine which TEE is available
> on a platform.
>
> Signed-off-by: Volodymyr Babchuk <volodymyr_babchuk@xxxxxxxx>
> ---
>   xen/arch/arm/domctl.c       | 10 ++++++++++
>   xen/include/public/domctl.h |  8 ++++++++
>   2 files changed, 18 insertions(+)
>
> diff --git a/xen/arch/arm/domctl.c b/xen/arch/arm/domctl.c
> index 4587c75..f128d34 100644
> --- a/xen/arch/arm/domctl.c
> +++ b/xen/arch/arm/domctl.c
> @@ -13,6 +13,7 @@
>   #include <xen/sched.h>
>   #include <xen/types.h>
>   #include <xsm/xsm.h>
> +#include <asm-arm/tee/tee.h>
>   #include <public/domctl.h>
>   
>   void arch_get_domain_info(const struct domain *d,
> @@ -171,6 +172,15 @@ long arch_do_domctl(struct xen_domctl *domctl, struct 
> domain *d,
>   
>           return rc;
>       }
> +    case XEN_DOMCTL_tee_op:
> +        switch ( domctl->u.tee_op.cmd )
> +        {
> +        case XEN_DOMCTL_TEE_OP_ENABLE:
> +            return tee_enable(d);
> +
> +        default:
> +            return -EINVAL;
> +        }
I don't think a domctl is necessary for this. You can add a flag in 
xen_arch_domainconfig.
Cheers,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel