Re: [PATCH 0/2] Xen FF-A mediator

[Jens Wiklander <jens.wiklander@xxxxxxxxxx>]

Hi Stefano,

On Tue, Jun 7, 2022 at 11:55 PM Stefano Stabellini
<sstabellini@xxxxxxxxxx> wrote:
>
> On Tue, 7 Jun 2022, Jens Wiklander wrote:
> > Hi,
> >
> > This patch sets add a FF-A [1] mediator modeled after the TEE mediator
> > already present in Xen. The FF-A mediator implements the subset of the FF-A
> > 1.1 specification needed to communicate with OP-TEE using FF-A as transport
> > mechanism instead of SMC/HVC as with the TEE mediator. It allows a similar
> > design in OP-TEE as with the TEE mediator where OP-TEE presents one virtual
> > partition of itself to each guest in Xen.
> >
> > The FF-A mediator is generic in the sense it has nothing OP-TEE specific
> > except that only the subset needed for OP-TEE is implemented so far. The
> > hooks needed to inform OP-TEE that a guest is created or destroyed is part
> > of the FF-A specification.
> >
> > It should be possible to extend the FF-A mediator to implement a larger
> > portion of the FF-A 1.1 specification without breaking with the way OP-TEE
> > is communicated with here. So it should be possible to support any TEE or
> > Secure Partition using FF-A as transport with this mediator.
> >
> > [1] https://developer.arm.com/documentation/den0077/latest
> >
> > Thanks,
> > Jens
>
> Hi Jens,
>
> Many thanks for the patches! I tried to apply them to the master branch
> but unfortunately they don't apply any longer. Could you please rebase
> them on master (or even better rebase them on staging) and resend?

No problem, I'll rebase and send out a v2.

Thanks,
Jens

>
> Thank you!
>
>
>
> > Jens Wiklander (2):
> >   xen/arm: smccc: add support for SMCCCv1.2 extended input/output
> >     registers
> >   xen/arm: add FF-A mediator
> >
> >  xen/arch/arm/Kconfig         |   11 +
> >  xen/arch/arm/Makefile        |    1 +
> >  xen/arch/arm/arm64/smc.S     |   43 +
> >  xen/arch/arm/domain.c        |   10 +
> >  xen/arch/arm/ffa.c           | 1624 ++++++++++++++++++++++++++++++++++
> >  xen/arch/arm/vsmc.c          |   19 +-
> >  xen/include/asm-arm/domain.h |    4 +
> >  xen/include/asm-arm/ffa.h    |   71 ++
> >  xen/include/asm-arm/smccc.h  |   42 +
> >  9 files changed, 1821 insertions(+), 4 deletions(-)
> >  create mode 100644 xen/arch/arm/ffa.c
> >  create mode 100644 xen/include/asm-arm/ffa.h
>