diff options
| author | Laszlo Ersek <lersek@redhat.com> | 2017-08-30 14:49:17 +0200 |
|---|---|---|
| committer | Laszlo Ersek <lersek@redhat.com> | 2017-09-01 14:20:47 +0200 |
| commit | 1afbb85f8736b63bc32f5827c3cb7b2e583114eb (patch) | |
| tree | 37e68cd8632b51bf0400c6dd7dce7c32cf34a415 | |
| parent | 80ddd336daf9ee398ace88c95ec410fd5c41f804 (diff) | |
| download | edk2-1afbb85f8736b63bc32f5827c3cb7b2e583114eb.zip edk2-1afbb85f8736b63bc32f5827c3cb7b2e583114eb.tar.gz edk2-1afbb85f8736b63bc32f5827c3cb7b2e583114eb.tar.bz2 | |
OvmfPkg/IoMmuDxe: IoMmuFreeBuffer(): clean up DEBUG message
Log all relevant IN parameters on entry. (There are only IN parameters.)
Beautify the format string.
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>
| -rw-r--r-- | OvmfPkg/IoMmuDxe/AmdSevIoMmu.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/OvmfPkg/IoMmuDxe/AmdSevIoMmu.c b/OvmfPkg/IoMmuDxe/AmdSevIoMmu.c index 0ab7043..bc57de5 100644 --- a/OvmfPkg/IoMmuDxe/AmdSevIoMmu.c +++ b/OvmfPkg/IoMmuDxe/AmdSevIoMmu.c @@ -611,6 +611,14 @@ IoMmuFreeBuffer ( UINTN CommonBufferPages;
COMMON_BUFFER_HEADER *CommonBufferHeader;
+ DEBUG ((
+ DEBUG_VERBOSE,
+ "%a: Host=0x%p Pages=0x%Lx\n",
+ __FUNCTION__,
+ HostAddress,
+ (UINT64)Pages
+ ));
+
CommonBufferPages = Pages + 1;
CommonBufferHeader = (COMMON_BUFFER_HEADER *)(
(UINTN)HostAddress - EFI_PAGE_SIZE
@@ -630,14 +638,6 @@ IoMmuFreeBuffer ( //
FreePages (CommonBufferHeader->StashBuffer, Pages);
- DEBUG ((
- DEBUG_VERBOSE,
- "%a Address 0x%Lx Pages 0x%Lx\n",
- __FUNCTION__,
- (UINT64)(UINTN)HostAddress,
- (UINT64)Pages
- ));
-
//
// Release the common buffer itself. Unmap() has re-encrypted it in-place, so
// no need to zero it.
|