Changeset 88078 in vbox

Timestamp:

Mar 10, 2021 7:25:11 PM (4 years ago)

Author:

vboxsync

Message:

AMD IOMMU: Doxygen fixes. bugref:9654

Location:

trunk/src/VBox

Files:

• Devices/Bus/DevIommuAmd.cpp (modified) (1 diff)
• VMM/VMMAll/PDMAllIommu.cpp (modified) (15 diffs)
• VMM/VMMR0/PDMR0DevHlp.cpp (modified) (2 diffs)
• VMM/VMMR3/PDMDevMiscHlp.cpp (modified) (2 diffs)

trunk/src/VBox/Devices/Bus/DevIommuAmd.cpp

@@ -1239 +1239 @@
  *
  * @param   pThis           The shared IOMMU device state.
- * @param   pThis           The ring-3 IOMMU device state.
+ * @param   pThisR3         The ring-3 IOMMU device state.
  * @param   idDomain        The domain ID.
  * @param   uIova           The I/O virtual address.

trunk/src/VBox/VMM/VMMAll/PDMAllIommu.cpp

@@ -68 +68 @@
 
 /** @copydoc PDMIOMMUREGR3::pfnMsiRemap */
-int pdmIommuMsiRemap(PPDMDEVINS pDevIns, uint16_t uDeviceId, PCMSIMSG pMsiIn, PMSIMSG pMsiOut)
-{
-    PPDMIOMMU  pIommu       = PDMDEVINS_TO_IOMMU(pDevIns);
-    PPDMDEVINS pDevInsIommu = pIommu->CTX_SUFF(pDevIns);
-    if (   pDevInsIommu
-        && pDevInsIommu != pDevIns)
-    {
-        int rc = pIommu->pfnMsiRemap(pDevInsIommu, uDeviceId, pMsiIn, pMsiOut);
+int pdmIommuMsiRemap(PPDMDEVINS pDevIns, uint16_t idDevice, PCMSIMSG pMsiIn, PMSIMSG pMsiOut)
+{
+    PPDMIOMMU  pIommu       = PDMDEVINS_TO_IOMMU(pDevIns);
+    PPDMDEVINS pDevInsIommu = pIommu->CTX_SUFF(pDevIns);
+    if (   pDevInsIommu
+        && pDevInsIommu != pDevIns)
+    {
+        int rc = pIommu->pfnMsiRemap(pDevInsIommu, idDevice, pMsiIn, pMsiOut);
         if (RT_FAILURE(rc))
         {
-            LogFunc(("MSI remap failed. uDeviceId=%#x pMsiIn=(%#RX64, %#RU32) rc=%Rrc\n", uDeviceId, pMsiIn->Addr.u64,
+            LogFunc(("MSI remap failed. idDevice=%#x pMsiIn=(%#RX64, %#RU32) rc=%Rrc\n", idDevice, pMsiIn->Addr.u64,
                      pMsiIn->Data.u32, rc));
         }
@@ -110 +110 @@
         && pDevInsIommu != pDevIns)
     {
-        uint16_t const uDeviceId = pdmIommuGetPciDeviceId(pDevIns, pPciDev);
+        uint16_t const idDevice = pdmIommuGetPciDeviceId(pDevIns, pPciDev);
         int rc = VINF_SUCCESS;
         while (cbRead > 0)
@@ -116 +116 @@
             RTGCPHYS GCPhysOut;
             size_t   cbContig;
-            rc = pIommu->pfnMemAccess(pDevInsIommu, uDeviceId, GCPhys, cbRead, PDMIOMMU_MEM_F_READ, &GCPhysOut, &cbContig);
+            rc = pIommu->pfnMemAccess(pDevInsIommu, idDevice, GCPhys, cbRead, PDMIOMMU_MEM_F_READ, &GCPhysOut, &cbContig);
             if (RT_SUCCESS(rc))
             {
@@ -133 +133 @@
             else
             {
-                LogFunc(("IOMMU memory read failed. uDeviceId=%#x GCPhys=%#RGp cb=%zu rc=%Rrc\n", uDeviceId, GCPhys, cbRead, rc));
+                LogFunc(("IOMMU memory read failed. idDevice=%#x GCPhys=%#RGp cb=%zu rc=%Rrc\n", idDevice, GCPhys, cbRead, rc));
 
                 /*
@@ -174 +174 @@
         && pDevInsIommu != pDevIns)
     {
-        uint16_t const uDeviceId = pdmIommuGetPciDeviceId(pDevIns, pPciDev);
+        uint16_t const idDevice = pdmIommuGetPciDeviceId(pDevIns, pPciDev);
         int rc = VINF_SUCCESS;
         while (cbWrite > 0)
@@ -180 +180 @@
             RTGCPHYS GCPhysOut;
             size_t   cbContig;
-            rc = pIommu->pfnMemAccess(pDevInsIommu, uDeviceId, GCPhys, cbWrite, PDMIOMMU_MEM_F_WRITE, &GCPhysOut, &cbContig);
+            rc = pIommu->pfnMemAccess(pDevInsIommu, idDevice, GCPhys, cbWrite, PDMIOMMU_MEM_F_WRITE, &GCPhysOut, &cbContig);
             if (RT_SUCCESS(rc))
             {
@@ -197 +197 @@
             else
             {
-                LogFunc(("IOMMU memory write failed. uDeviceId=%#x GCPhys=%#RGp cb=%zu rc=%Rrc\n", uDeviceId, GCPhys, cbWrite,
+                LogFunc(("IOMMU memory write failed. idDevice=%#x GCPhys=%#RGp cb=%zu rc=%Rrc\n", idDevice, GCPhys, cbWrite,
                          rc));
                 break;
@@ -235 +235 @@
         && pDevInsIommu != pDevIns)
     {
-        uint16_t const uDeviceId = pdmIommuGetPciDeviceId(pDevIns, pPciDev);
+        uint16_t const idDevice = pdmIommuGetPciDeviceId(pDevIns, pPciDev);
         size_t   cbContig  = 0;
         RTGCPHYS GCPhysOut = NIL_RTGCPHYS;
-        int rc = pIommu->pfnMemAccess(pDevInsIommu, uDeviceId, GCPhys & X86_PAGE_BASE_MASK, X86_PAGE_SIZE, PDMIOMMU_MEM_F_READ,
+        int rc = pIommu->pfnMemAccess(pDevInsIommu, idDevice, GCPhys & X86_PAGE_BASE_MASK, X86_PAGE_SIZE, PDMIOMMU_MEM_F_READ,
                                       &GCPhysOut, &cbContig);
         if (RT_SUCCESS(rc))
@@ -247 +247 @@
         }
 
-        LogFunc(("IOMMU memory read for pointer access failed. uDeviceId=%#x GCPhys=%#RGp rc=%Rrc\n", uDeviceId, GCPhys, rc));
+        LogFunc(("IOMMU memory read for pointer access failed. idDevice=%#x GCPhys=%#RGp rc=%Rrc\n", idDevice, GCPhys, rc));
         return rc;
     }
@@ -280 +280 @@
         && pDevInsIommu != pDevIns)
     {
-        uint16_t const uDeviceId = pdmIommuGetPciDeviceId(pDevIns, pPciDev);
+        uint16_t const idDevice = pdmIommuGetPciDeviceId(pDevIns, pPciDev);
         size_t   cbContig  = 0;
         RTGCPHYS GCPhysOut = NIL_RTGCPHYS;
-        int rc = pIommu->pfnMemAccess(pDevInsIommu, uDeviceId, GCPhys & X86_PAGE_BASE_MASK, X86_PAGE_SIZE, PDMIOMMU_MEM_F_WRITE,
+        int rc = pIommu->pfnMemAccess(pDevInsIommu, idDevice, GCPhys & X86_PAGE_BASE_MASK, X86_PAGE_SIZE, PDMIOMMU_MEM_F_WRITE,
                                       &GCPhysOut, &cbContig);
         if (RT_SUCCESS(rc))
@@ -292 +292 @@
         }
 
-        LogFunc(("IOMMU memory write for pointer access failed. uDeviceId=%#x GCPhys=%#RGp rc=%Rrc\n", uDeviceId, GCPhys, rc));
+        LogFunc(("IOMMU memory write for pointer access failed. idDevice=%#x GCPhys=%#RGp rc=%Rrc\n", idDevice, GCPhys, rc));
         return rc;
     }
@@ -341 +341 @@
 
         /* Ask the IOMMU for corresponding translated physical addresses. */
-        uint16_t const uDeviceId = pdmIommuGetPciDeviceId(pDevIns, pPciDev);
+        uint16_t const idDevice = pdmIommuGetPciDeviceId(pDevIns, pPciDev);
         AssertCompile(sizeof(RTGCPHYS) == sizeof(uint64_t));
-        int rc = pIommu->pfnMemBulkAccess(pDevInsIommu, uDeviceId, cPages, (uint64_t const *)paGCPhysPages, PDMIOMMU_MEM_F_READ,
+        int rc = pIommu->pfnMemBulkAccess(pDevInsIommu, idDevice, cPages, (uint64_t const *)paGCPhysPages, PDMIOMMU_MEM_F_READ,
                                           paGCPhysOut);
         if (RT_SUCCESS(rc))
@@ -353 +353 @@
         }
         else
-            LogFunc(("Bulk translation for read access failed. uDeviceId=%#x cPages=%zu rc=%Rrc\n", uDeviceId, cPages, rc));
+            LogFunc(("Bulk translation for read access failed. idDevice=%#x cPages=%zu rc=%Rrc\n", idDevice, cPages, rc));
 
         RTMemFree(paGCPhysOut);
@@ -404 +404 @@
 
         /* Ask the IOMMU for corresponding translated physical addresses. */
-        uint16_t const uDeviceId = pdmIommuGetPciDeviceId(pDevIns, pPciDev);
+        uint16_t const idDevice = pdmIommuGetPciDeviceId(pDevIns, pPciDev);
         AssertCompile(sizeof(RTGCPHYS) == sizeof(uint64_t));
-        int rc = pIommu->pfnMemBulkAccess(pDevInsIommu, uDeviceId, cPages, (uint64_t const *)paGCPhysPages, PDMIOMMU_MEM_F_WRITE,
+        int rc = pIommu->pfnMemBulkAccess(pDevInsIommu, idDevice, cPages, (uint64_t const *)paGCPhysPages, PDMIOMMU_MEM_F_WRITE,
                                           paGCPhysOut);
         if (RT_SUCCESS(rc))
@@ -416 +416 @@
         }
         else
-            LogFunc(("IOMMU bulk translation failed. uDeviceId=%#x cPages=%zu rc=%Rrc\n", uDeviceId, cPages, rc));
+            LogFunc(("IOMMU bulk translation failed. idDevice=%#x cPages=%zu rc=%Rrc\n", idDevice, cPages, rc));
 
         RTMemFree(paGCPhysOut);

trunk/src/VBox/VMM/VMMR0/PDMR0DevHlp.cpp

@@ -1524 +1524 @@
 
 /** @interface_method_impl{PDMIOAPICHLP,pfnIommuMsiRemap} */
-static DECLCALLBACK(int) pdmR0IoApicHlp_IommuMsiRemap(PPDMDEVINS pDevIns, uint16_t uDeviceId, PCMSIMSG pMsiIn, PMSIMSG pMsiOut)
+static DECLCALLBACK(int) pdmR0IoApicHlp_IommuMsiRemap(PPDMDEVINS pDevIns, uint16_t idDevice, PCMSIMSG pMsiIn, PMSIMSG pMsiOut)
 {
     PDMDEV_ASSERT_DEVINS(pDevIns);
@@ -1531 +1531 @@
 
 #ifdef VBOX_WITH_IOMMU_AMD
-    int rc = pdmIommuMsiRemap(pDevIns, uDeviceId, pMsiIn, pMsiOut);
+    int rc = pdmIommuMsiRemap(pDevIns, idDevice, pMsiIn, pMsiOut);
     if (RT_SUCCESS(rc) || rc != VERR_IOMMU_NOT_PRESENT)
         return rc;
 #else
-    RT_NOREF(pDevIns, uDeviceId);
+    RT_NOREF(pDevIns, idDevice);
 #endif
 

trunk/src/VBox/VMM/VMMR3/PDMDevMiscHlp.cpp

@@ -142 +142 @@
 
 /** @interface_method_impl{PDMIOAPICHLP,pfnIommuMsiRemap} */
-static DECLCALLBACK(int) pdmR3IoApicHlp_IommuMsiRemap(PPDMDEVINS pDevIns, uint16_t uDeviceId, PCMSIMSG pMsiIn, PMSIMSG pMsiOut)
+static DECLCALLBACK(int) pdmR3IoApicHlp_IommuMsiRemap(PPDMDEVINS pDevIns, uint16_t idDevice, PCMSIMSG pMsiIn, PMSIMSG pMsiOut)
 {
     PDMDEV_ASSERT_DEVINS(pDevIns);
@@ -149 +149 @@
 
 #ifdef VBOX_WITH_IOMMU_AMD
-    int rc = pdmIommuMsiRemap(pDevIns, uDeviceId, pMsiIn, pMsiOut);
+    int rc = pdmIommuMsiRemap(pDevIns, idDevice, pMsiIn, pMsiOut);
     if (RT_SUCCESS(rc) || rc != VERR_IOMMU_NOT_PRESENT)
         return rc;
 #else
-    RT_NOREF(pDevIns, uDeviceId);
+    RT_NOREF(pDevIns, idDevice);
 #endif