Hi Jens > On 26 Apr 2024, at 10:47, Jens Wiklander <jens.wiklan...@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 >
