ArmPlatformPkg: remove EFI_MEMORY_UC attribute from normal memory
authorArd Biesheuvel <ard.biesheuvel@linaro.org>
Thu, 8 Sep 2016 08:05:45 +0000 (09:05 +0100)
committerArd Biesheuvel <ard.biesheuvel@linaro.org>
Thu, 8 Sep 2016 09:35:33 +0000 (10:35 +0100)
On ARM systems, mapping normal memory as device memory may have unintended
side effects, given that unaligned accesses or loads and stores with special
semantics (e.g., load/store exclusive) may fault or may not work as expected.
Similarly, DC ZVA instructions are only supported on normal memory, not
device memory.

So remove the EFI_MEMORY_UC attribute that we set by default on system RAM.
If any region requires this attribute, it is up to the driver to set this
attribute, and to ensure that no offending operations are performed on it.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
ArmPlatformPkg/ArmJunoPkg/Library/ArmJunoLib/ArmJunoMem.c
ArmPlatformPkg/ArmVExpressPkg/Library/ArmVExpressLibRTSM/RTSMMem.c
ArmPlatformPkg/MemoryInitPei/MemoryInitPeiLib.c

index 41731c1..aa8d7d9 100644 (file)
@@ -56,7 +56,6 @@ ArmPlatformGetVirtualMemoryMap (
   ResourceAttributes =\r
       EFI_RESOURCE_ATTRIBUTE_PRESENT |\r
       EFI_RESOURCE_ATTRIBUTE_INITIALIZED |\r
-      EFI_RESOURCE_ATTRIBUTE_UNCACHEABLE |\r
       EFI_RESOURCE_ATTRIBUTE_WRITE_COMBINEABLE |\r
       EFI_RESOURCE_ATTRIBUTE_WRITE_THROUGH_CACHEABLE |\r
       EFI_RESOURCE_ATTRIBUTE_WRITE_BACK_CACHEABLE |\r
index c6df2e1..115df24 100644 (file)
@@ -61,7 +61,6 @@ ArmPlatformGetVirtualMemoryMap (
     ResourceAttributes =\r
         EFI_RESOURCE_ATTRIBUTE_PRESENT |\r
         EFI_RESOURCE_ATTRIBUTE_INITIALIZED |\r
-        EFI_RESOURCE_ATTRIBUTE_UNCACHEABLE |\r
         EFI_RESOURCE_ATTRIBUTE_WRITE_COMBINEABLE |\r
         EFI_RESOURCE_ATTRIBUTE_WRITE_THROUGH_CACHEABLE |\r
         EFI_RESOURCE_ATTRIBUTE_WRITE_BACK_CACHEABLE |\r
index 75e6631..2feb11f 100644 (file)
@@ -89,7 +89,6 @@ MemoryPeim (
   ResourceAttributes = (\r
       EFI_RESOURCE_ATTRIBUTE_PRESENT |\r
       EFI_RESOURCE_ATTRIBUTE_INITIALIZED |\r
-      EFI_RESOURCE_ATTRIBUTE_UNCACHEABLE |\r
       EFI_RESOURCE_ATTRIBUTE_WRITE_COMBINEABLE |\r
       EFI_RESOURCE_ATTRIBUTE_WRITE_THROUGH_CACHEABLE |\r
       EFI_RESOURCE_ATTRIBUTE_WRITE_BACK_CACHEABLE |\r