summaryrefslogtreecommitdiff
path: root/ArmPkg
diff options
context:
space:
mode:
authorOliver Smith-Denny <osde@linux.microsoft.com>2024-07-19 12:40:18 -0700
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>2024-07-23 03:32:41 +0000
commite10de1cb0345e54c0b4d83f5979a76111d10d6c7 (patch)
tree9d392f44d4dce79adadc7bbf6fffd875f83ae75b /ArmPkg
parent74833ca459577307da02c3d74a9e65fb3d3161c2 (diff)
downloadedk2-e10de1cb0345e54c0b4d83f5979a76111d10d6c7.zip
edk2-e10de1cb0345e54c0b4d83f5979a76111d10d6c7.tar.gz
edk2-e10de1cb0345e54c0b4d83f5979a76111d10d6c7.tar.bz2
ArmPkg: ArmMmuLib: Add AARCH64 Memory Attribute Update Failure Log
This adds logging in failure cases of SetMemoryAttributes. This is useful generally as if an attribute update fails, code will typically break, but is added in particular to make debugging incorrect bootloader usage of the Memory Attribute Protocol. This patch updates the AARCH64 UpdateRegionMapping. Signed-off-by: Oliver Smith-Denny <osde@linux.microsoft.com>
Diffstat (limited to 'ArmPkg')
-rw-r--r--ArmPkg/Library/ArmMmuLib/AArch64/ArmMmuLibCore.c7
1 files changed, 7 insertions, 0 deletions
diff --git a/ArmPkg/Library/ArmMmuLib/AArch64/ArmMmuLibCore.c b/ArmPkg/Library/ArmMmuLib/AArch64/ArmMmuLibCore.c
index 6a1f3f9..b83373d 100644
--- a/ArmPkg/Library/ArmMmuLib/AArch64/ArmMmuLibCore.c
+++ b/ArmPkg/Library/ArmMmuLib/AArch64/ArmMmuLibCore.c
@@ -382,6 +382,13 @@ UpdateRegionMapping (
UINTN T0SZ;
if (((RegionStart | RegionLength) & EFI_PAGE_MASK) != 0) {
+ DEBUG ((
+ DEBUG_ERROR,
+ "%a RegionStart: 0x%llx or RegionLength: 0x%llx are not page aligned!\n",
+ __func__,
+ RegionStart,
+ RegionLength
+ ));
return EFI_INVALID_PARAMETER;
}