From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-il1-f180.google.com (mail-il1-f180.google.com [209.85.166.180]) by mx.groups.io with SMTP id smtpd.web09.5886.1608117205580082070 for ; Wed, 16 Dec 2020 03:13:25 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@linaro.org header.s=google header.b=CYhRBoug; spf=pass (domain: linaro.org, ip: 209.85.166.180, mailfrom: sughosh.ganu@linaro.org) Received: by mail-il1-f180.google.com with SMTP id 75so10145186ilv.13 for ; Wed, 16 Dec 2020 03:13:25 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=0A3AKwQ31LdHAppl57jGLFLcW2kjfD9rz6F7TLtIwe8=; b=CYhRBougeGAOP/PCsZde32wCnO44aYzkqX+qqFb4D8aJbNNh+B1MEStNhmHupbfLrg IBAfK7MDv9adGtkge39DoAFQ9DbTK84Uih5CbBqjYiPzxge/mf4n9o/v/CjFkEpeY2FP rtQQB9E1eEQIuLAXXB2BCkSYvyfd8zBoYCIH/YvH2fO5asFwTYO4ifkZw2hNuCtIbp3T NlWbrIiluV9cN9FeLXeM/9i3Yjm+djzO4TMw1UizhH7lRhsme+UAbeEO1EGXTIzWcIaW 4vB/6JpMTwMvW1bKKRE8XXh3WEEXIcdjbxHVmdDH8Nol7U+dNEHhWX6/HN0/G2WJ2Dt0 c1Rw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=0A3AKwQ31LdHAppl57jGLFLcW2kjfD9rz6F7TLtIwe8=; b=mbraa8pxOKcMaSd6V+EpsFyt7hRLtlsIITDsd7xzFiCJogZTCwVV/fgRgn1hy/NAw1 Lbeu5SSjWIcWTNprgZIy5iuq8YEraI11nVXXMQ/OK06h0o/hDI0ls5Mx67NmMHQGWjVx vGY6JJlD3r6/FoEqVHkVKKB55XOu6S16Hb6rEJiWt967cjOzm6ENMgx+f9d5vt3/VINh rlLr90I3YGZfY4NZNaEpBqa5Ut7nVsWf5ModebE9Ch8wKYel9oCnzxt8CWL73gDGxfln jB3c6VxrUJ0n0idZdPQn2tMMQ4n/rTI2kKOMqjzgT83HBqr+Ya+BQ1NAhWqWGVCthveU 7fdQ== X-Gm-Message-State: AOAM533YsDZHN0rUEUtYr6b+E5/wgImSxNXJsq8kf9R8sAB5UdRYB1sp wu/9U+HjwS4LGJk/rFK+EHXe4q/hKVx1cQUK9a0e4PbEzmE3PkaR X-Google-Smtp-Source: ABdhPJyI0FIs8P0yUtw/D6thGP72nsW1mch8hPCwnkuEZaTarSNPYkMEOp1gNsuwpUhKaiXoPzlNlH/c6yfCjeEJfC8= X-Received: by 2002:a92:489b:: with SMTP id j27mr2855815ilg.168.1608117204639; Wed, 16 Dec 2020 03:13:24 -0800 (PST) MIME-Version: 1.0 References: <16509C53970A5E2F.4679@groups.io> In-Reply-To: <16509C53970A5E2F.4679@groups.io> From: "Sughosh Ganu" Date: Wed, 16 Dec 2020 16:43:13 +0530 Message-ID: Subject: Re: [edk2-devel] [PATCH v2 00/13] Add support for using FF-A calls To: devel@edk2.groups.io, Sughosh Ganu Cc: Sami Mujawar , Ard Biesheuvel , Leif Lindholm , Sahil Malhotra Content-Type: multipart/alternative; boundary="000000000000bd79f505b692f634" --000000000000bd79f505b692f634 Content-Type: text/plain; charset="UTF-8" On Mon, 14 Dec 2020 at 20:12, Sughosh Ganu via groups.io wrote: > > The following patch series adds support for using the Firmware > Framework(FF-A) as a transport mechanism for requesting services from > the Secure Partition Manager(SPM). This is done through a Pcd which > can be used to enable the FF-A mechanism or to use the earlier used > SVC calls. > > > Changes since V1: > Handled review comments from Sami Mujawar > Please review these patches along with V3 of the patch series for adding support for StandaloneMm as an Optee TA[1]. -sughosh [1] - https://edk2.groups.io/g/devel/message/68951 > > Achin Gupta (7): > ArmPkg/IndustryStandard: Add barebones FF-A header > ArmPkg/ArmSvcLib: Return x4-x7 in output parameters > StandaloneMmPkg: Use FF-A header file in Standalone MM Core entry > point > StandaloneMmPkg: Add option to use FF-A calls for communication with > SPM > StandaloneMmPkg: Use FF-A header file in Standalone MM Arm MMU library > StandaloneMmMmuLib: Add option to use FF-A calls to get memory > region's permissions > StandaloneMmMmuLib: Add option to use FF-A calls to set memory > region's permissions > > Ilias Apalodimas (2): > MdeModulePkg/VariableStandaloneMm: Set PcdFlashNvStorageVariableBase > to Pcd > StandaloneMmPkg: Allow sending FFA Direct Request message to > StandaloneMm > > Sughosh Ganu (4): > ArmPkg: Introduce support for PcdFfaEnable > StandaloneMmPkg: Add macros for SPM version > StandaloneMmPkg: Add the SPM version for FF-A > StandaloneMmPkg: Add option to use FF-A calls for getting SPM version > > ArmPkg/ArmPkg.dec | 6 + > .../ArmMmuStandaloneMmLib.inf | 3 + > .../RuntimeDxe/VariableStandaloneMm.inf | 6 +- > .../StandaloneMmCoreEntryPoint.inf | 3 + > ArmPkg/Include/IndustryStandard/ArmFfaSvc.h | 44 ++++++ > ArmPkg/Include/IndustryStandard/ArmMmSvc.h | 3 + > ArmPkg/Include/Library/ArmSvcLib.h | 10 +- > .../AArch64/ArmMmuStandaloneMmLib.c | 145 +++++++++++++----- > .../StandaloneMmCpu/AArch64/EventHandle.c | 4 +- > .../AArch64/StandaloneMmCoreEntryPoint.c | 130 ++++++++++++---- > ArmPkg/Library/ArmSvcLib/AArch64/ArmSvc.S | 4 +- > 11 files changed, 291 insertions(+), 67 deletions(-) > create mode 100644 ArmPkg/Include/IndustryStandard/ArmFfaSvc.h > > -- > 2.17.1 > > > > > > > > --000000000000bd79f505b692f634 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable

On Mon, 14 Dec 2020 at 20:12, Sughosh Gan= u via groups.io <sughosh.ganu=3Dlinaro.org@groups.io> wrote:

The following patch series adds support for using the Firmware
Framework(FF-A) as a transport mechanism for requesting services from
the Secure Partition Manager(SPM). This is done through a Pcd which
can be used to enable the FF-A mechanism or to use the earlier used
SVC calls.


Changes since V1:
Handled review comments from Sami Mujawar

<= div>Please review these patches along with V3 of the patch series for addin= g support for StandaloneMm as an Optee TA[1].

-sug= hosh

[1] -=C2=A0https://edk2.groups.io/g/devel/message/68951



Achin Gupta (7):
=C2=A0 ArmPkg/IndustryStandard: Add barebones FF-A header
=C2=A0 ArmPkg/ArmSvcLib: Return x4-x7 in output parameters
=C2=A0 StandaloneMmPkg: Use FF-A header file in Standalone MM Core entry =C2=A0 =C2=A0 point
=C2=A0 StandaloneMmPkg: Add option to use FF-A calls for communication wit= h
=C2=A0 =C2=A0 SPM
=C2=A0 StandaloneMmPkg: Use FF-A header file in Standalone MM Arm MMU libr= ary
=C2=A0 StandaloneMmMmuLib: Add option to use FF-A calls to get memory
=C2=A0 =C2=A0 region's permissions
=C2=A0 StandaloneMmMmuLib: Add option to use FF-A calls to set memory
=C2=A0 =C2=A0 region's permissions

Ilias Apalodimas (2):
=C2=A0 MdeModulePkg/VariableStandaloneMm: Set PcdFlashNvStorageVariableBas= e
=C2=A0 =C2=A0 to Pcd
=C2=A0 StandaloneMmPkg: Allow sending FFA Direct Request message to
=C2=A0 =C2=A0 StandaloneMm

Sughosh Ganu (4):
=C2=A0 ArmPkg: Introduce support for PcdFfaEnable
=C2=A0 StandaloneMmPkg: Add macros for SPM version
=C2=A0 StandaloneMmPkg: Add the SPM version for FF-A
=C2=A0 StandaloneMmPkg: Add option to use FF-A calls for getting SPM versi= on

=C2=A0ArmPkg/ArmPkg.dec=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0|=C2=A0 =C2=A06 + =C2=A0.../ArmMmuStandaloneMmLib.inf=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0|=C2=A0 =C2=A03 +
=C2=A0.../RuntimeDxe/VariableStandaloneMm.inf=C2=A0 =C2=A0 =C2=A0 =C2=A0|= =C2=A0 =C2=A06 +-
=C2=A0.../StandaloneMmCoreEntryPoint.inf=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0= =C2=A0 |=C2=A0 =C2=A03 +
=C2=A0ArmPkg/Include/IndustryStandard/ArmFfaSvc.h=C2=A0 =C2=A0|=C2=A0 44 += +++++
=C2=A0ArmPkg/Include/IndustryStandard/ArmMmSvc.h=C2=A0 =C2=A0 |=C2=A0 =C2= =A03 +
=C2=A0ArmPkg/Include/Library/ArmSvcLib.h=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0= =C2=A0 |=C2=A0 10 +-
=C2=A0.../AArch64/ArmMmuStandaloneMmLib.c=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0| 145 +++++++++++++-----
=C2=A0.../StandaloneMmCpu/AArch64/EventHandle.c=C2=A0 =C2=A0 =C2=A0|=C2=A0= =C2=A04 +-
=C2=A0.../AArch64/StandaloneMmCoreEntryPoint.c=C2=A0 =C2=A0 =C2=A0 | 130 += +++++++++++----
=C2=A0ArmPkg/Library/ArmSvcLib/AArch64/ArmSvc.S=C2=A0 =C2=A0 =C2=A0|=C2=A0= =C2=A04 +-
=C2=A011 files changed, 291 insertions(+), 67 deletions(-)
=C2=A0create mode 100644 ArmPkg/Include/IndustryStandard/ArmFfaSvc.h

--
2.17.1







--000000000000bd79f505b692f634--