Hi Jens, > On 12 Feb 2026, at 15:37, Jens Wiklander <[email protected]> wrote: > > Hi Bertrand, > > On Wed, Feb 11, 2026 at 6:16 PM Bertrand Marquis > <[email protected]> wrote: >> >> FF-A 1.2 extends the endpoint memory access descriptor (EMAD) from >> 16 to 32 bytes, adding implementation-defined (IMPDEF) fields and >> reserved space. The MEM_SHARE path currently assumes the 1.1 EMAD >> size and rejects the 1.2 layout. >> >> Add FF-A 1.2 EMAD support to MEM_SHARE: >> - define ffa_mem_access_1_2 and store IMPDEF payload in ffa_shm_mem >> - emit 1.2 EMADs to the SPMC for FF-A 1.2 guests, forwarding IMPDEF >> - refactor header parsing into read_mem_transaction() for 1.0/1.1+ >> - detect EMAD format by mem_access_size to allow 1.1 on 1.2 guests >> >> Export ffa_fw_version to build memory descriptors according to the >> firware version to relay share memory requests and remove unused offset > > firmware
Right, > >> macros. >> >> Functional impact: MEM_SHARE supports FF-A 1.2 EMADs. >> >> Signed-off-by: Bertrand Marquis <[email protected]> >> --- >> Changes since v1: >> - export ffa_fw_version and use it to construct descriptors according to >> firmware version >> - remove unused offset macros >> --- >> xen/arch/arm/tee/ffa.c | 2 +- >> xen/arch/arm/tee/ffa_private.h | 1 + >> xen/arch/arm/tee/ffa_shm.c | 121 +++++++++++++++++++++++---------- >> 3 files changed, 87 insertions(+), 37 deletions(-) > > Looks good: > Reviewed-by: Jens Wiklander <[email protected]> Thanks. Maybe that one can be fixed on commit. Cheers Bertrand
