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>

diff --git a/OvmfPkg/IoMmuDxe/AmdSevIoMmu.c b/OvmfPkg/IoMmuDxe/AmdSevIoMmu.c
index 0ab7043498bda817c54c32970805f92a8f82313d..bc57de5b572b6048b6be7a4954ca1ca149a078df 100644 (file)
--- a/OvmfPkg/IoMmuDxe/AmdSevIoMmu.c
+++ b/OvmfPkg/IoMmuDxe/AmdSevIoMmu.c
@@ -611,6 +611,14 @@ IoMmuFreeBuffer (
   UINTN                CommonBufferPages;\r
   COMMON_BUFFER_HEADER *CommonBufferHeader;\r
 \r
+  DEBUG ((\r
+    DEBUG_VERBOSE,\r
+    "%a: Host=0x%p Pages=0x%Lx\n",\r
+    __FUNCTION__,\r
+    HostAddress,\r
+    (UINT64)Pages\r
+    ));\r
+\r
   CommonBufferPages = Pages + 1;\r
   CommonBufferHeader = (COMMON_BUFFER_HEADER *)(\r
                          (UINTN)HostAddress - EFI_PAGE_SIZE\r
@@ -630,14 +638,6 @@ IoMmuFreeBuffer (
   //\r
   FreePages (CommonBufferHeader->StashBuffer, Pages);\r
 \r
-  DEBUG ((\r
-    DEBUG_VERBOSE,\r
-    "%a Address 0x%Lx Pages 0x%Lx\n",\r
-    __FUNCTION__,\r
-    (UINT64)(UINTN)HostAddress,\r
-    (UINT64)Pages\r
-    ));\r
-\r
   //\r
   // Release the common buffer itself. Unmap() has re-encrypted it in-place, so\r
   // no need to zero it.\r