In the SetMemoryEncDec() function, the way we currently report
PhysicalAddress is not uniform:
- mostly we say "for %lx",
- in one spot we say "at %lx" (even though the 2MB page being split does
not live *at* PhysicalAddress, instead it maps PhysicalAddress),
- in another spot we don't log PhysicalAddress at all (when splitting a
1GB page).
Unify this, using the format string "for Physical=0x%Lx".
Cc: Brijesh Singh <brijesh.singh@amd.com>
Cc: Jordan Justen <jordan.l.justen@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Brijesh Singh <brijesh.singh@amd.com>
Tested-by: Brijesh Singh <brijesh.singh@amd.com>
if (!PageMapLevel4Entry->Bits.Present) {\r
DEBUG ((\r
DEBUG_WARN,\r
if (!PageMapLevel4Entry->Bits.Present) {\r
DEBUG ((\r
DEBUG_WARN,\r
- "%a:%a: ERROR bad PML4 for %lx\n",\r
+ "%a:%a: ERROR bad PML4 for Physical=0x%Lx\n",\r
gEfiCallerBaseName,\r
__FUNCTION__,\r
PhysicalAddress\r
gEfiCallerBaseName,\r
__FUNCTION__,\r
PhysicalAddress\r
if (!PageDirectory1GEntry->Bits.Present) {\r
DEBUG ((\r
DEBUG_WARN,\r
if (!PageDirectory1GEntry->Bits.Present) {\r
DEBUG ((\r
DEBUG_WARN,\r
- "%a:%a: ERROR bad PDPE for %lx\n",\r
+ "%a:%a: ERROR bad PDPE for Physical=0x%Lx\n",\r
gEfiCallerBaseName,\r
__FUNCTION__,\r
PhysicalAddress\r
gEfiCallerBaseName,\r
__FUNCTION__,\r
PhysicalAddress\r
SetOrClearCBit(&PageDirectory1GEntry->Uint64, Mode);\r
DEBUG ((\r
DEBUG_VERBOSE,\r
SetOrClearCBit(&PageDirectory1GEntry->Uint64, Mode);\r
DEBUG ((\r
DEBUG_VERBOSE,\r
- "%a:%a: Updated 1GB entry for %lx\n",\r
+ "%a:%a: Updated 1GB entry for Physical=0x%Lx\n",\r
gEfiCallerBaseName,\r
__FUNCTION__,\r
PhysicalAddress\r
gEfiCallerBaseName,\r
__FUNCTION__,\r
PhysicalAddress\r
//\r
DEBUG ((\r
DEBUG_VERBOSE,\r
//\r
DEBUG ((\r
DEBUG_VERBOSE,\r
- "%a:%a: Spliting 1GB page\n",\r
+ "%a:%a: Spliting 1GB page for Physical=0x%Lx\n",\r
+ __FUNCTION__,\r
+ PhysicalAddress\r
));\r
Split1GPageTo2M(((UINT64)PageDirectory1GEntry->Bits.PageTableBaseAddress)<<30, (UINT64*) PageDirectory1GEntry, 0, 0);\r
continue;\r
));\r
Split1GPageTo2M(((UINT64)PageDirectory1GEntry->Bits.PageTableBaseAddress)<<30, (UINT64*) PageDirectory1GEntry, 0, 0);\r
continue;\r
if (!PageDirectory2MEntry->Bits.Present) {\r
DEBUG ((\r
DEBUG_WARN,\r
if (!PageDirectory2MEntry->Bits.Present) {\r
DEBUG ((\r
DEBUG_WARN,\r
- "%a:%a: ERROR bad PDE for %lx\n",\r
+ "%a:%a: ERROR bad PDE for Physical=0x%Lx\n",\r
gEfiCallerBaseName,\r
__FUNCTION__,\r
PhysicalAddress\r
gEfiCallerBaseName,\r
__FUNCTION__,\r
PhysicalAddress\r
//\r
DEBUG ((\r
DEBUG_VERBOSE,\r
//\r
DEBUG ((\r
DEBUG_VERBOSE,\r
- "%a:%a: Spliting 2MB page at %lx\n",\r
+ "%a:%a: Spliting 2MB page for Physical=0x%Lx\n",\r
gEfiCallerBaseName,\r
__FUNCTION__,\r
PhysicalAddress\r
gEfiCallerBaseName,\r
__FUNCTION__,\r
PhysicalAddress\r
if (!PageTableEntry->Bits.Present) {\r
DEBUG ((\r
DEBUG_WARN,\r
if (!PageTableEntry->Bits.Present) {\r
DEBUG ((\r
DEBUG_WARN,\r
- "%a:%a: ERROR bad PTE for %lx\n",\r
+ "%a:%a: ERROR bad PTE for Physical=0x%Lx\n",\r
gEfiCallerBaseName,\r
__FUNCTION__,\r
PhysicalAddress\r
gEfiCallerBaseName,\r
__FUNCTION__,\r
PhysicalAddress\r