The DEBUG print that outputs the base and size of the page table
allocation always prints 0x0 for the size, given that BufferSize will be
updated by PageTableMap () and contain the unused allocation on return.
So move the DEBUG print right after the allocation.
Signed-off-by: Ard Biesheuvel <a...@kernel.org>
---
UefiCpuPkg/CpuMpPei/CpuPaging.c | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/UefiCpuPkg/CpuMpPei/CpuPaging.c b/UefiCpuPkg/CpuMpPei/CpuPaging.c
index b7ddb0005b6fbcac..175e47ccd737a0c1 100644
--- a/UefiCpuPkg/CpuMpPei/CpuPaging.c
+++ b/UefiCpuPkg/CpuMpPei/CpuPaging.c
@@ -396,6 +396,13 @@ EnablePaePageTable (
return EFI_OUT_OF_RESOURCES;
}
+ DEBUG ((
+ DEBUG_INFO,
+ "EnablePaePageTable: Created PageTable = 0x%x, BufferSize = %x\n",
+ PageTable,
+ BufferSize
+ ));
+
Status = PageTableMap (&PageTable, PagingPae, Buffer, &BufferSize, 0,
SIZE_4GB, &MapAttribute, &MapMask, NULL);
ASSERT_EFI_ERROR (Status);
if (EFI_ERROR (Status) || (PageTable == 0)) {
@@ -417,13 +424,6 @@ EnablePaePageTable (
//
AsmWriteCr0 (AsmReadCr0 () | BIT31);
- DEBUG ((
- DEBUG_INFO,
- "EnablePaePageTable: Created PageTable = 0x%x, BufferSize = %x\n",
- PageTable,
- BufferSize
- ));
-
return Status;
}
--
2.39.2
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#105942): https://edk2.groups.io/g/devel/message/105942
Mute This Topic: https://groups.io/mt/99411874/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
