From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ed1-f45.google.com (mail-ed1-f45.google.com [209.85.208.45]) by mx.groups.io with SMTP id smtpd.web11.5972.1620283481792451577 for ; Wed, 05 May 2021 23:44:42 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@linaro.org header.s=google header.b=Gju4DrnX; spf=pass (domain: linaro.org, ip: 209.85.208.45, mailfrom: etienne.carriere@linaro.org) Received: by mail-ed1-f45.google.com with SMTP id bf4so4825671edb.11 for ; Wed, 05 May 2021 23:44:41 -0700 (PDT) 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=Dd6Q7ISs+tUQMtMNVRJq11MSTOnub3TC0bE9CoX5NkY=; b=Gju4DrnXwQyMTkOY/XgqKkEtKeZRx7E2vBDZNiJu2sxFCDnQMoyLBqEb9jbgEjg/3U x6eqOoFeW4z95VXK65Wo9gHbTVuG4tqTjGarKZ8UzhIqSKXoWtRi79v6X5K0PWyJRNon laNh7WSQvjFMY36nlzU8xNeQn+mNOPoO9+srjS2Lzxdd4gNqiIAM9VNyZJxxqbkj+ppm 7i/CEMFjoHiMqQs8ILGEa98+ZzoJKUzqhCA6QkVBoSuhIEuE5KIhFkBsBCXjnFzij/gx Xy61lf/vezc2ocnnQI4OOwDbMAxrh5IGnocD27MJUwzEGMRbkcybvtdNQE4p1LCaidK1 1w0Q== 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=Dd6Q7ISs+tUQMtMNVRJq11MSTOnub3TC0bE9CoX5NkY=; b=lpG/RiAopFN/xBbriT55pUoSNalvVMNOhbT91eY2Zd7rhDT9kQOKdjiggHq5khmN4P 7E74Y9dNvNl2baxJYTgBR7hVbmp3/VtPMkPBGLzKQzkCVjc3566oRZn1O/FFGkYXZUBX AHu/PAGfrBiTIYpSOEKgh47gKi8YP6L3y6wZ7lWmEnlvH/ktuAZd88dAV8BlT8sxJrTG UBz9Tjfn1G/Tkn3NoOOTT0WNFt8e8utOWnFRSNZCrGaDM+qwA+3MY0s5dVqtFyudO6zM U6ay2kgrSVEbwYrMcXnTuqVotBHbWDHMifSEZrHz936nxzme5GJkYLF8uJ4Wy4BbxZbD eyog== X-Gm-Message-State: AOAM531mZYl+yBmlmyACiyQtmmpuSwbi+yblwpI8mbNfgMZUdfZ3f3Fx opMsRmD3Y72Ze92zy1f3tRfeumE1AYrK4HIzVtCmNw== X-Google-Smtp-Source: ABdhPJze4pCQqUgrq7/MOZhHJoK1rqL09oZCfcherFHYH0OZn0JG3qerDYlU+gwxSdOndkFTv12r6HTmFkZ0OpJBZp8= X-Received: by 2002:a05:6402:154d:: with SMTP id p13mr3271017edx.371.1620283480271; Wed, 05 May 2021 23:44:40 -0700 (PDT) MIME-Version: 1.0 References: <20210504152048.8739-1-etienne.carriere@linaro.org> <000e01d74227$7b1af140$7150d3c0$@byosoft.com.cn> In-Reply-To: <000e01d74227$7b1af140$7150d3c0$@byosoft.com.cn> From: "Etienne Carriere" Date: Thu, 6 May 2021 08:44:29 +0200 Message-ID: Subject: Re: [edk2-devel] [PATCH 0/5] Arm 32bit support in StandaloveMm To: gaoliming Cc: devel@edk2.groups.io, Achin Gupta , Ard Biesheuvel , Jiewen Yao , Leif Lindholm , Sami Mujawar , Sughosh Ganu Content-Type: multipart/alternative; boundary="00000000000047005805c1a3a5cd" --00000000000047005805c1a3a5cd Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Hello Liming, Sure, I will do, thanks. Regards, Etienne On Thu, 6 May 2021 at 05:26, gaoliming wrote: > Etienne: > Can you submit one BZ (https://bugzilla.tianocore.org/) for this new > feature? > > Thanks > Liming > > -----=E9=82=AE=E4=BB=B6=E5=8E=9F=E4=BB=B6----- > > =E5=8F=91=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io =E4=BB=A3=E8=A1=A8 Etienne > > Carriere > > =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2021=E5=B9=B45=E6=9C=884=E6=97= =A5 23:21 > > =E6=94=B6=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io > > =E6=8A=84=E9=80=81: Achin Gupta ; Ard Biesheuvel > > ; Jiewen Yao ; Leif > > Lindholm ; Sami Mujawar ; > > Sughosh Ganu ; Etienne Carriere > > > > =E4=B8=BB=E9=A2=98: [edk2-devel] [PATCH 0/5] Arm 32bit support in Stan= daloveMm > > > > This series targets building StandaloneMM package for 32bit ARM > > architectures. The main parts of the changes move AArch64/* > > files to paths suitable for both 32bit and 64bit Arm machines. > > > > Because these changes move file paths, some other edk2 related > > source trees must be updated. Up to my knowledge, only edk2-platforms > > and edk2-codereview must be synchronised. [1] shows the changes > > needed in edk2-platforms to synchronise with these changes. > > > > I will post a patch series for edk2-platforms, including [1] and > > some other changes to allow building StandaloneMmPkg for 32bit Arm > > platforms. > > > > This change is a resend of [2], rebased to latest master. > > > > [1] > > https://github.com/etienne-lms/edk2-platforms/commit/becbd185167f6a115 > > dbd3895bc6071ab04e44d9b > > [2] https://edk2.groups.io/g/devel/message/72834 (with 72835, 72836, > > 72837 and 72838) > > > > Etienne Carriere (5): > > ArmPkg/IndustryStandard: 32b/64b agnostic FF-A and Mm SVC IDs > > ArmPkg: prepare 32bit ARM build of StandaloneMmPkg > > GenGv: Arm: support images entered in Thumb mode > > StandaloneMmPkg: fix pointer/int casts against 32bit architectures > > StandaloneMmPkg: build for 32bit arm machines > > > > ArmPkg/ArmPkg.dec | 2 +- > > ArmPkg/ArmPkg.dsc | 2 +- > > .../MmCommunicationDxe/MmCommunication.c | 2 +- > > ArmPkg/Include/IndustryStandard/ArmFfaSvc.h | 12 ++++++ > > ArmPkg/Include/IndustryStandard/ArmMmSvc.h | 15 ++++++++ > > .../{AArch64 =3D> }/ArmMmuStandaloneMmLib.c | 15 ++++---- > > .../ArmMmuStandaloneMmLib.inf | 6 +-- > > BaseTools/Source/C/GenFv/GenFvInternalLib.c | 38 > > ++++++++++++++----- > > StandaloneMmPkg/Core/StandaloneMmCore.inf | 2 +- > > .../{AArch64 =3D> }/EventHandle.c | 12 +++++- > > .../{AArch64 =3D> }/StandaloneMmCpu.c | 10 ++--- > > .../{AArch64 =3D> }/StandaloneMmCpu.h | 0 > > .../{AArch64 =3D> }/StandaloneMmCpu.inf | 0 > > .../StandaloneMmCoreEntryPoint.h | 0 > > .../{AArch64 =3D> Arm}/CreateHobList.c | 16 ++++---- > > .../{AArch64 =3D> Arm}/SetPermissions.c | 2 +- > > .../StandaloneMmCoreEntryPoint.c | 18 ++++----- > > .../StandaloneMmCoreEntryPoint.inf | 14 +++---- > > .../{AArch64 =3D> Arm}/StandaloneMmCoreHobLib.c | 0 > > .../StandaloneMmCoreHobLibInternal.c | 0 > > .../StandaloneMmCoreHobLib.inf | 8 ++-- > > ...rnal.c =3D> ArmStandaloneMmMemLibInternal.c} | 9 ++++- > > .../StandaloneMmMemLib/StandaloneMmMemLib.inf | 6 +-- > > .../VariableMmDependency.inf | 2 +- > > StandaloneMmPkg/StandaloneMmPkg.dsc | 8 ++-- > > 25 files changed, 131 insertions(+), 68 deletions(-) > > rename ArmPkg/Library/StandaloneMmMmuLib/{AArch64 > > =3D> }/ArmMmuStandaloneMmLib.c (92%) > > rename StandaloneMmPkg/Drivers/StandaloneMmCpu/{AArch64 > > =3D> }/EventHandle.c (92%) > > rename StandaloneMmPkg/Drivers/StandaloneMmCpu/{AArch64 > > =3D> }/StandaloneMmCpu.c (94%) > > rename StandaloneMmPkg/Drivers/StandaloneMmCpu/{AArch64 > > =3D> }/StandaloneMmCpu.h (100%) > > rename StandaloneMmPkg/Drivers/StandaloneMmCpu/{AArch64 > > =3D> }/StandaloneMmCpu.inf (100%) > > rename StandaloneMmPkg/Include/Library/{AArch64 =3D> > > Arm}/StandaloneMmCoreEntryPoint.h (100%) > > rename > > StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/{AArch64 =3D> > > Arm}/CreateHobList.c (91%) > > rename > > StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/{AArch64 =3D> > > Arm}/SetPermissions.c (96%) > > rename > > StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/{AArch64 =3D> > > Arm}/StandaloneMmCoreEntryPoint.c (94%) > > rename StandaloneMmPkg/Library/StandaloneMmCoreHobLib/{AArch64 =3D> > > Arm}/StandaloneMmCoreHobLib.c (100%) > > rename StandaloneMmPkg/Library/StandaloneMmCoreHobLib/{AArch64 =3D> > > Arm}/StandaloneMmCoreHobLibInternal.c (100%) > > rename > > StandaloneMmPkg/Library/StandaloneMmMemLib/{AArch64/StandaloneMm > > MemLibInternal.c =3D> ArmStandaloneMmMemLibInternal.c} (86%) > > > > -- > > 2.17.1 > > > > > > > >=20 > > > > > > --00000000000047005805c1a3a5cd Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Hello=C2=A0Liming,

Sure, I= =C2=A0will do, thanks.

Regards,
Etienne=

= On Thu, 6 May 2021 at 05:26, gaoliming <gaoliming@byosoft.com.cn> wrote:
Etienne:
=C2=A0 Can you submit one BZ (https://bugzilla.tianocore.org/) f= or this new
feature?

Thanks
Liming
> -----=E9=82=AE=E4=BB=B6=E5=8E=9F=E4=BB=B6-----
> =E5=8F=91=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io <devel@edk2.groups.io> =E4=BB=A3=E8=A1= =A8 Etienne
> Carriere
> =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2021=E5=B9=B45=E6=9C=884=E6=97= =A5 23:21
> =E6=94=B6=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io
> =E6=8A=84=E9=80=81: Achin Gupta <achin.gupta@arm.com>; Ard Biesheuvel
> <= ardb+tianocore@kernel.org>; Jiewen Yao <jiewen.yao@intel.com>; Leif
> Lindholm <l= eif@nuviainc.com>; Sami Mujawar <sami.mujawar@arm.com>;
> Sughosh Ganu <sughosh.ganu@linaro.org>; Etienne Carriere
> <= etienne.carriere@linaro.org>
> =E4=B8=BB=E9=A2=98: [edk2-devel] [PATCH 0/5] Arm 32bit support in Sta= ndaloveMm
>
> This series targets building StandaloneMM package for 32bit ARM
> architectures. The main parts of the changes move AArch64/*
> files to paths suitable for both 32bit and 64bit Arm machines.
>
> Because these changes move file paths, some other edk2 related
> source trees must be updated. Up to my knowledge, only edk2-platforms=
> and edk2-codereview must be synchronised. [1] shows the changes
> needed in edk2-platforms to synchronise with these changes.
>
> I will post a patch series for edk2-platforms, including [1] and
> some other changes to allow building StandaloneMmPkg for 32bit Arm > platforms.
>
> This change is a resend of [2], rebased to latest master.
>
> [1]
> https://github.com/etien= ne-lms/edk2-platforms/commit/becbd185167f6a115
> dbd3895bc6071ab04e44d9b
> [2] https://edk2.groups.io/g/devel/message/72834 (with 72835, 72836,
> 72837 and 72838)
>
> Etienne Carriere (5):
>=C2=A0 =C2=A0ArmPkg/IndustryStandard: 32b/64b agnostic FF-A and Mm SVC= IDs
>=C2=A0 =C2=A0ArmPkg: prepare 32bit ARM build of StandaloneMmPkg
>=C2=A0 =C2=A0GenGv: Arm: support images entered in Thumb mode
>=C2=A0 =C2=A0StandaloneMmPkg: fix pointer/int casts against 32bit arch= itectures
>=C2=A0 =C2=A0StandaloneMmPkg: build for 32bit arm machines
>
>=C2=A0 ArmPkg/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 2 +- >=C2=A0 ArmPkg/ArmPkg.dsc=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 2 +- >=C2=A0 .../MmCommunicationDxe/MmCommunication.c=C2=A0 =C2=A0 =C2=A0 |= =C2=A0 2 +-
>=C2=A0 ArmPkg/Include/IndustryStandard/ArmFfaSvc.h=C2=A0 =C2=A0| 12 ++= ++++
>=C2=A0 ArmPkg/Include/IndustryStandard/ArmMmSvc.h=C2=A0 =C2=A0 | 15 ++= ++++++
>=C2=A0 .../{AArch64 =3D> }/ArmMmuStandaloneMmLib.c=C2=A0 =C2=A0 =C2= = =A0| 15 ++++----
>=C2=A0 .../ArmMmuStandaloneMmLib.inf=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0= =C2=A0 =C2=A0 =C2=A0 =C2=A0|=C2=A0 6 +--
>=C2=A0 BaseTools/Source/C/GenFv/GenFvInternalLib.c=C2=A0 =C2=A0| 38 > ++++++++++++++-----
>=C2=A0 StandaloneMmPkg/Core/StandaloneMmCore.inf=C2=A0 =C2=A0 =C2=A0|= =C2=A0 2 +-
>=C2=A0 .../{AArch64 =3D> }/EventHandle.c=C2=A0 =C2=A0 =C2=A0 =C2=A0= =C2=A0 =C2=A0 =C2=A0 =C2=A0| 12 +++++-
>=C2=A0 .../{AArch64 =3D> }/StandaloneMmCpu.c=C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0| 10 ++---
>=C2=A0 .../{AArch64 =3D> }/StandaloneMmCpu.h=C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0|=C2=A0 0
>=C2=A0 .../{AArch64 =3D> }/StandaloneMmCpu.inf=C2=A0 =C2=A0 =C2=A0 = = =C2=A0 =C2=A0|=C2=A0 0
>=C2=A0 .../StandaloneMmCoreEntryPoint.h=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= = =A0 =C2=A0 =C2=A0 |=C2=A0 0
>=C2=A0 .../{AArch64 =3D> Arm}/CreateHobList.c=C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 | 16 ++++----
>=C2=A0 .../{AArch64 =3D> Arm}/SetPermissions.c=C2=A0 =C2=A0 =C2=A0 = = =C2=A0 =C2=A0|=C2=A0 2 +-
>=C2=A0 .../StandaloneMmCoreEntryPoint.c=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= = =A0 =C2=A0 =C2=A0 | 18 ++++-----
>=C2=A0 .../StandaloneMmCoreEntryPoint.inf=C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 | 14 +++----
>=C2=A0 .../{AArch64 =3D> Arm}/StandaloneMmCoreHobLib.c |=C2=A0 0 >=C2=A0 .../StandaloneMmCoreHobLibInternal.c=C2=A0 =C2=A0 =C2=A0 =C2=A0= =C2=A0 |=C2=A0 0
>=C2=A0 .../StandaloneMmCoreHobLib.inf=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 |=C2=A0 8 ++--
>=C2=A0 ...rnal.c =3D> ArmStandaloneMmMemLibInternal.c} |=C2=A0 9 ++= ++-
>=C2=A0 .../StandaloneMmMemLib/StandaloneMmMemLib.inf |=C2=A0 6 +--
>=C2=A0 .../VariableMmDependency.inf=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = = =C2=A0 =C2=A0 =C2=A0 =C2=A0 |=C2=A0 2 +-
>=C2=A0 StandaloneMmPkg/StandaloneMmPkg.dsc=C2=A0 =C2=A0 =C2=A0 =C2=A0 = = =C2=A0 =C2=A0|=C2=A0 8 ++--
>=C2=A0 25 files changed, 131 insertions(+), 68 deletions(-)
>=C2=A0 rename ArmPkg/Library/StandaloneMmMmuLib/{AArch64
> =3D> }/ArmMmuStandaloneMmLib.c (92%)
>=C2=A0 rename StandaloneMmPkg/Drivers/StandaloneMmCpu/{AArch64
> =3D> }/EventHandle.c (92%)
>=C2=A0 rename StandaloneMmPkg/Drivers/StandaloneMmCpu/{AArch64
> =3D> }/StandaloneMmCpu.c (94%)
>=C2=A0 rename StandaloneMmPkg/Drivers/StandaloneMmCpu/{AArch64
> =3D> }/StandaloneMmCpu.h (100%)
>=C2=A0 rename StandaloneMmPkg/Drivers/StandaloneMmCpu/{AArch64
> =3D> }/StandaloneMmCpu.inf (100%)
>=C2=A0 rename StandaloneMmPkg/Include/Library/{AArch64 =3D>
> Arm}/StandaloneMmCoreEntryPoint.h (100%)
>=C2=A0 rename
> StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/{AArch64 =3D> > Arm}/CreateHobList.c (91%)
>=C2=A0 rename
> StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/{AArch64 =3D> > Arm}/SetPermissions.c (96%)
>=C2=A0 rename
> StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/{AArch64 =3D> > Arm}/StandaloneMmCoreEntryPoint.c (94%)
>=C2=A0 rename StandaloneMmPkg/Library/StandaloneMmCoreHobLib/{AArch64 = = =3D>
> Arm}/StandaloneMmCoreHobLib.c (100%)
>=C2=A0 rename StandaloneMmPkg/Library/StandaloneMmCoreHobLib/{AArch64 = = =3D>
> Arm}/StandaloneMmCoreHobLibInternal.c (100%)
>=C2=A0 rename
> StandaloneMmPkg/Library/StandaloneMmMemLib/{AArch64/StandaloneMm
> MemLibInternal.c =3D> ArmStandaloneMmMemLibInternal.c} (86%)
>
> --
> 2.17.1
>
>
>
>
>



--00000000000047005805c1a3a5cd--