diff options
| author | Laszlo Ersek <lersek@redhat.com> | 2017-08-30 13:31:30 +0200 |
|---|---|---|
| committer | Laszlo Ersek <lersek@redhat.com> | 2017-09-01 14:20:33 +0200 |
| commit | 3728ea5a95e4321a6fdcec13188b712eba55f4ad (patch) | |
| tree | a71c81fda4c7c0913450cecafb40421b2040f434 /OvmfPkg/Library/BaseMemEncryptSevLib | |
| parent | 631bd7e084c042b799bc1497be3bfc01ed1d5f83 (diff) | |
| download | edk2-3728ea5a95e4321a6fdcec13188b712eba55f4ad.tar.gz edk2-3728ea5a95e4321a6fdcec13188b712eba55f4ad.tar.bz2 edk2-3728ea5a95e4321a6fdcec13188b712eba55f4ad.zip | |
OvmfPkg/BaseMemEncryptSevLib: promote DEBUG_WARN levels to DEBUG_ERROR
In SetMemoryEncDec(), we have four locations where we (a) log a message on
the DEBUG_WARN level that says "ERROR", (b) return the status code
RETURN_NO_MAPPING right after.
These messages clearly describe actual errors (bad PML4, PDPE, PDE, PTE).
Promote their debug levels to DEBUG_ERROR, and remove the word "ERROR"
from the messages.
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>
Diffstat (limited to 'OvmfPkg/Library/BaseMemEncryptSevLib')
| -rw-r--r-- | OvmfPkg/Library/BaseMemEncryptSevLib/X64/VirtualMemory.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/OvmfPkg/Library/BaseMemEncryptSevLib/X64/VirtualMemory.c b/OvmfPkg/Library/BaseMemEncryptSevLib/X64/VirtualMemory.c index 58c793dae6..c37d799bc1 100644 --- a/OvmfPkg/Library/BaseMemEncryptSevLib/X64/VirtualMemory.c +++ b/OvmfPkg/Library/BaseMemEncryptSevLib/X64/VirtualMemory.c @@ -287,8 +287,8 @@ SetMemoryEncDec ( PageMapLevel4Entry += PML4_OFFSET(PhysicalAddress);
if (!PageMapLevel4Entry->Bits.Present) {
DEBUG ((
- DEBUG_WARN,
- "%a:%a: ERROR bad PML4 for Physical=0x%Lx\n",
+ DEBUG_ERROR,
+ "%a:%a: bad PML4 for Physical=0x%Lx\n",
gEfiCallerBaseName,
__FUNCTION__,
PhysicalAddress
@@ -300,8 +300,8 @@ SetMemoryEncDec ( PageDirectory1GEntry += PDP_OFFSET(PhysicalAddress);
if (!PageDirectory1GEntry->Bits.Present) {
DEBUG ((
- DEBUG_WARN,
- "%a:%a: ERROR bad PDPE for Physical=0x%Lx\n",
+ DEBUG_ERROR,
+ "%a:%a: bad PDPE for Physical=0x%Lx\n",
gEfiCallerBaseName,
__FUNCTION__,
PhysicalAddress
@@ -351,8 +351,8 @@ SetMemoryEncDec ( PageDirectory2MEntry += PDE_OFFSET(PhysicalAddress);
if (!PageDirectory2MEntry->Bits.Present) {
DEBUG ((
- DEBUG_WARN,
- "%a:%a: ERROR bad PDE for Physical=0x%Lx\n",
+ DEBUG_ERROR,
+ "%a:%a: bad PDE for Physical=0x%Lx\n",
gEfiCallerBaseName,
__FUNCTION__,
PhysicalAddress
@@ -391,8 +391,8 @@ SetMemoryEncDec ( PageTableEntry += PTE_OFFSET(PhysicalAddress);
if (!PageTableEntry->Bits.Present) {
DEBUG ((
- DEBUG_WARN,
- "%a:%a: ERROR bad PTE for Physical=0x%Lx\n",
+ DEBUG_ERROR,
+ "%a:%a: bad PTE for Physical=0x%Lx\n",
gEfiCallerBaseName,
__FUNCTION__,
PhysicalAddress
|