Hi Jens
> On 26 Apr 2024, at 10:47, Jens Wiklander <jens.wiklander@linaro.org> wrote:
>
> Hi,
>
> This patch set adds support for FF-A notifications. We only support
> global notifications, per vCPU notifications remain unsupported.
>
> The first three patches are further cleanup and can be merged before the
> rest if desired.
>
> A physical SGI is used to make Xen aware of pending FF-A notifications. The
> physical SGI is selected by the SPMC in the secure world. Since it must not
> already be used by Xen the SPMC is in practice forced to donate one of the
> secure SGIs, but that's normally not a problem. The SGI handling in Xen is
> updated to support registration of handlers for SGIs that aren't statically
> assigned, that is, SGI IDs above GIC_SGI_MAX.
From my point of view:
- patches 1 to 3 are ready to be commited.
- patch 4 will need a R-b from an other maintainer.
- patch 5 has still some stuff to be checked or fixed but could be
handled as a single patch if the rest or the serie is merged.
Regards
Bertrand
>
> Thanks,
> Jens
>
> v2->v3:
> - "xen/arm: ffa: support notification" and
> "xen/arm: allow dynamically assigned SGI handlers" updated as requestsed,
> details in each patch.
>
> v1->v2:
> - "xen/arm: ffa: support notification" and
> "xen/arm: allow dynamically assigned SGI handlers" updated as requestsed,
> details in each patch.
> - Added Bertrands R-B for "xen/arm: ffa: refactor ffa_handle_call()",
> "xen/arm: ffa: use ACCESS_ONCE()", and
> "xen/arm: ffa: simplify ffa_handle_mem_share()"
>
> Jens Wiklander (5):
> xen/arm: ffa: refactor ffa_handle_call()
> xen/arm: ffa: use ACCESS_ONCE()
> xen/arm: ffa: simplify ffa_handle_mem_share()
> xen/arm: allow dynamically assigned SGI handlers
> xen/arm: ffa: support notification
>
> xen/arch/arm/gic.c | 12 +-
> xen/arch/arm/include/asm/gic.h | 2 +-
> xen/arch/arm/irq.c | 18 +-
> xen/arch/arm/tee/Makefile | 1 +
> xen/arch/arm/tee/ffa.c | 83 +++++--
> xen/arch/arm/tee/ffa_notif.c | 378 ++++++++++++++++++++++++++++++++
> xen/arch/arm/tee/ffa_partinfo.c | 9 +-
> xen/arch/arm/tee/ffa_private.h | 56 ++++-
> xen/arch/arm/tee/ffa_shm.c | 33 ++-
> xen/include/public/arch-arm.h | 14 ++
> 10 files changed, 551 insertions(+), 55 deletions(-)
> create mode 100644 xen/arch/arm/tee/ffa_notif.c
>
> --
> 2.34.1
>