Hi Jens, > On 3 Dec 2025, at 14:55, Jens Wiklander <[email protected]> wrote: > > Hi Bertrand, > > On Thu, Nov 27, 2025 at 4:53 PM Bertrand Marquis > <[email protected]> wrote: >> >> Share the SMCCC plumbing used by SEND_DIRECT and RUN via a common >> ffa_finish_direct_req_run() helper so canonical success and error cases >> are handled in one place. >> >> The dispatcher now routes FFA_RUN through ffa_handle_run(), and direct >> requests bail out early if a guest targets itself or a non-secure >> endpoint. This simplifies the direct path and prepares the mediator for >> the wider v1.2 register ABI. > > From the above, it sounds like FFA_RUN was already supported, but I > don't see how.
Yes It seems i forgot to rework the commit message when i rework the split in my serie. I will update the commit message in next version. > >> >> Signed-off-by: Bertrand Marquis <[email protected]> >> --- >> xen/arch/arm/tee/ffa.c | 3 + >> xen/arch/arm/tee/ffa_msg.c | 111 ++++++++++++++++++++++++--------- >> xen/arch/arm/tee/ffa_private.h | 1 + >> 3 files changed, 84 insertions(+), 31 deletions(-) >> >> diff --git a/xen/arch/arm/tee/ffa.c b/xen/arch/arm/tee/ffa.c >> index 7392bb6c3db9..92cb6ad7ec97 100644 >> --- a/xen/arch/arm/tee/ffa.c >> +++ b/xen/arch/arm/tee/ffa.c > > FFA_RUN should be removed from the list of unsupported FF-A interfaces > close to the top of this file. In fact i do that in the last patch of the serie with adding support for FFA_RUN in ffa_features but this does not really make sense as this is not bounded to 1.2. I will move those from patch 10 into this one instead in next version. > > The rest looks good. > Thanks Cheers Bertrand
