It would be nice if there was a way to globally provide these updated link 
flags for all AArch64 RUNTIME_DXE targets - do you know of a way to do do this?

Otherwise we should ensure that the other reference platforms get updated to 
assure a better chance of success for other implementations.  A comment to the 
effect that this is required for UEFI runtime memory protection option would be 
handy as well.

Reviewed-by: Eugene Cohen <eug...@hp.com>

Thanks again.

Eugene

-----Original Message-----
From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] 
Sent: Monday, June 29, 2015 12:37 PM
To: edk2-devel@lists.sourceforge.net; ler...@redhat.com; 
olivier.mar...@arm.com; af...@apple.com; Cohen, Eugene; jiewen....@intel.com
Cc: yingke.d....@intel.com; matt.flem...@intel.com; Ard Biesheuvel
Subject: [PATCH 2/2] ArmVirtPkg: build runtime drivers with 64 KB section 
alignment

This adds the 64 KB alignment overlay linker script to the linker
command line of DXE_RUNTIME_DRIVER modules. This makes these modules
compatible with the new Properties Table feature by aligning the .text
and .data sections to 64 KB.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ard Biesheuvel <ard.biesheu...@linaro.org>
---
 ArmVirtPkg/ArmVirt.dsc.inc | 3 +++
 1 files changed, 3 insertions(+)

diff --git a/ArmVirtPkg/ArmVirt.dsc.inc b/ArmVirtPkg/ArmVirt.dsc.inc
index c6e684fd8512..f3f29d70917e 100644
--- a/ArmVirtPkg/ArmVirt.dsc.inc
+++ b/ArmVirtPkg/ArmVirt.dsc.inc
@@ -15,6 +15,9 @@
 [Defines]
   DEFINE DEBUG_PRINT_ERROR_LEVEL = 0x8000004F
 
+[BuildOptions.AARCH64.EDKII.DXE_RUNTIME_DRIVER]
+  *_*_AARCH64_DLINK_FLAGS = 
--script=$(EDK_TOOLS_PATH)/Scripts/gcc-aarch64-64K-align-ld-script
+
 [LibraryClasses.common]
 !if $(TARGET) == RELEASE
   DebugLib|MdePkg/Library/BaseDebugLibNull/BaseDebugLibNull.inf
diff --git a/ArmVirtPkg/ArmVirtQemu.dsc b/ArmVirtPkg/ArmVirtQemu.dsc
index 374cf7a9ee02..cded653a2c96 100644
--- a/ArmVirtPkg/ArmVirtQemu.dsc
+++ b/ArmVirtPkg/ArmVirtQemu.dsc
@@ -78,7 +78,6 @@ [BuildOptions]
   GCC:*_*_ARM_PLATFORM_FLAGS == -mcpu=cortex-a15 
-I$(WORKSPACE)/ArmVirtPkg/Include
   *_*_AARCH64_PLATFORM_FLAGS == -I$(WORKSPACE)/ArmVirtPkg/Include
 
-
 
################################################################################
 #
 # Pcd Section - list of all EDK II PCD Entries defined by this Platform
-- 
1.9.1


------------------------------------------------------------------------------
Don't Limit Your Business. Reach for the Cloud.
GigeNET's Cloud Solutions provide you with the tools and support that
you need to offload your IT needs and focus on growing your business.
Configured For All Businesses. Start Your Cloud Today.
https://www.gigenetcloud.com/
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to