On Mon, May 17, 2021 at 07:50:47AM +0200, Etienne Carriere wrote: > Build PlatformStandaloneMmRpmb for ARM architecture (32bit arm machine). > The generated image targets an execution environment similar to AArch64 > StMM secure partition in OP-TEE but in 32bit mode. > > GCC flag -fno-stack-protector > added. The stack protection code bring > GOT dependencies we prefer avoid when StMM runs in OP-TEE. > > Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org> > Cc: Ilias Apalodimas <ilias.apalodi...@linaro.org> > Cc: Leif Lindholm <l...@nuviainc.com> > Cc: Sami Mujawar <sami.muja...@arm.com> > Signed-off-by: Etienne Carriere <etienne.carri...@linaro.org> > --- > Changes since v1: > - Remove useless duplication of ArmSvcLib loading. > - Move BaseStackCheckLib to generic library classes instead of ARM only. > - include MdePkg/MdeLibs.dsc.inc > instead of loading > RegisterFilterLibNull.inf for ARM architecture. > --- > Platform/StandaloneMm/PlatformStandaloneMmPkg/PlatformStandaloneMmRpmb.dsc | > 12 +++++++++++- > 1 file changed, 11 insertions(+), 1 deletion(-) > > diff --git > a/Platform/StandaloneMm/PlatformStandaloneMmPkg/PlatformStandaloneMmRpmb.dsc > b/Platform/StandaloneMm/PlatformStandaloneMmPkg/PlatformStandaloneMmRpmb.dsc > index cb3f1ddf52..33364deb1e 100644 > --- > a/Platform/StandaloneMm/PlatformStandaloneMmPkg/PlatformStandaloneMmRpmb.dsc > +++ > b/Platform/StandaloneMm/PlatformStandaloneMmPkg/PlatformStandaloneMmRpmb.dsc > @@ -16,12 +16,14 @@ > PLATFORM_VERSION = 1.0 > DSC_SPECIFICATION = 0x0001001C > OUTPUT_DIRECTORY = Build/$(PLATFORM_NAME) > - SUPPORTED_ARCHITECTURES = AARCH64 > + SUPPORTED_ARCHITECTURES = ARM|AARCH64 > BUILD_TARGETS = DEBUG|RELEASE|NOOPT > SKUID_IDENTIFIER = DEFAULT > FLASH_DEFINITION = > Platform/StandaloneMm/PlatformStandaloneMmPkg/PlatformStandaloneMmRpmb.fdf > DEFINE DEBUG_MESSAGE = TRUE > > +!include MdePkg/MdeLibs.dsc.inc > + > > ################################################################################ > # > # Library Class section - list of all Library Classes needed by this > Platform. > @@ -39,6 +41,7 @@ > FvLib|StandaloneMmPkg/Library/FvLib/FvLib.inf > > HobLib|StandaloneMmPkg/Library/StandaloneMmCoreHobLib/StandaloneMmCoreHobLib.inf > IoLib|MdePkg/Library/BaseIoLibIntrinsic/BaseIoLibIntrinsic.inf > + NULL|MdePkg/Library/BaseStackCheckLib/BaseStackCheckLib.inf > MemLib|StandaloneMmPkg/Library/StandaloneMmMemLib/StandaloneMmMemLib.inf > > MemoryAllocationLib|StandaloneMmPkg/Library/StandaloneMmCoreMemoryAllocationLib/StandaloneMmCoreMemoryAllocationLib.inf > PcdLib|MdePkg/Library/BasePcdLibNull/BasePcdLibNull.inf > @@ -68,6 +71,9 @@ > # > NULL|ArmPkg/Library/CompilerIntrinsicsLib/CompilerIntrinsicsLib.inf > > +[LibraryClasses.ARM] > + ArmSoftFloatLib|ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.inf > + > [LibraryClasses.common.MM_STANDALONE] > HobLib|StandaloneMmPkg/Library/StandaloneMmHobLib/StandaloneMmHobLib.inf > > MmServicesTableLib|MdePkg/Library/StandaloneMmServicesTableLib/StandaloneMmServicesTableLib.inf > @@ -160,3 +166,7 @@ > [BuildOptions.AARCH64] > GCC:*_*_*_DLINK_FLAGS = -z common-page-size=0x1000 -march=armv8-a+nofp > GCC:*_*_*_CC_FLAGS = -mstrict-align > + > +[BuildOptions.ARM] > +GCC:*_*_*_DLINK_FLAGS = -z common-page-size=0x1000 -march=armv7-a > +GCC:*_*_*_CC_FLAGS = -fno-stack-protector > -- > 2.17.1 >
Acked-by: Ilias Apalodimas <ilias.apalodi...@linaro.org> -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#75205): https://edk2.groups.io/g/devel/message/75205 Mute This Topic: https://groups.io/mt/82880299/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
