OvmfPkg/IoMmuDxe: IoMmuUnmap(): clean up DEBUG message

The only important external information for this function, and for the
human looking at the log, is the Mapping input parameter. Log it on entry.

Stop logging the contents of the MAP_INFO structure pointed-to by Mapping.
Thanks to the previous patch, we can now associate IoMmuUnmap() messages
with IoMmuMap() messages -- and thereby with MAP_INFO contents -- purely
via Mapping.

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 59cee95c0e219051efb48c3c6c1cb687c1f87dbc..a153d250d54562e1acd4cfda22e060f1177fad11 100644 (file)
--- a/OvmfPkg/IoMmuDxe/AmdSevIoMmu.c
+++ b/OvmfPkg/IoMmuDxe/AmdSevIoMmu.c
@@ -348,6 +348,8 @@ IoMmuUnmap (
   COMMON_BUFFER_HEADER     *CommonBufferHeader;\r
   VOID                     *EncryptionTarget;\r
 \r
+  DEBUG ((DEBUG_VERBOSE, "%a: Mapping=0x%p\n", __FUNCTION__, Mapping));\r
+\r
   if (Mapping == NULL) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
@@ -399,16 +401,6 @@ IoMmuUnmap (
     break;\r
   }\r
 \r
-  DEBUG ((\r
-    DEBUG_VERBOSE,\r
-    "%a PlainText 0x%Lx Crypted 0x%Lx Pages 0x%Lx Bytes 0x%Lx\n",\r
-    __FUNCTION__,\r
-    MapInfo->PlainTextAddress,\r
-    MapInfo->CryptedAddress,\r
-    (UINT64)MapInfo->NumberOfPages,\r
-    (UINT64)MapInfo->NumberOfBytes\r
-    ));\r
-\r
   //\r
   // Restore the memory encryption mask on the area we used to hold the\r
   // plaintext.\r