Changeset 84431 in vbox for trunk/src/VBox/VMM/VMMR3

Timestamp:

May 21, 2020 8:42:19 AM (5 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

138135

Message:

AMD IOMMU: bugref:9654 IOMMU interrupt remapping callback skeleton.

File:

• trunk/src/VBox/VMM/VMMR3/PDMDevHlp.cpp (modified) (4 diffs)

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

@@ -3348 +3348 @@
                     ("%s/%d: u32Version=%#x expected %#x\n", pDevIns->pReg->szName, pDevIns->iInstance, pIommuReg->u32Version, PDM_IOMMUREGR3_VERSION),
                     VERR_INVALID_PARAMETER);
-    /** @todo IOMMU: Validate other parameters, see also tstDevicePdmDevHlp.cpp. */
+    AssertPtrReturn(pIommuReg->pfnMemRead, VERR_INVALID_POINTER);
+    AssertPtrReturn(pIommuReg->pfnMemWrite, VERR_INVALID_POINTER);
+    AssertPtrReturn(pIommuReg->pfnMsiRemap, VERR_INVALID_POINTER);
     AssertMsgReturn(pIommuReg->u32TheEnd == PDM_IOMMUREGR3_VERSION,
                     ("%s/%d: u32TheEnd=%#x expected %#x\n", pDevIns->pReg->szName, pDevIns->iInstance, pIommuReg->u32TheEnd, PDM_IOMMUREGR3_VERSION),
@@ -3362 +3364 @@
      */
     unsigned idxIommu = 0;
+#if 0
     for (idxIommu = 0; idxIommu < RT_ELEMENTS(pVM->pdm.s.aIommus); idxIommu++)
         if (!pVM->pdm.s.aIommus[idxIommu].pDevInsR3)
@@ -3368 +3371 @@
                           ("Too many IOMMUs. Max=%u\n", RT_ELEMENTS(pVM->pdm.s.aIommus)),
                           VERR_OUT_OF_RESOURCES);
+#else
+    /* Currently we support only a single IOMMU. */
+    AssertMsgReturn(!pVM->pdm.s.aIommus[0].pDevInsR3,
+                    ("%s/%u: Only one IOMMU device is supported!\n", pDevIns->pReg->szName, pDevIns->iInstance),
+                    VERR_ALREADY_EXISTS);
+#endif
     PPDMIOMMU pIommu = &pVM->pdm.s.aIommus[idxIommu];
 
@@ -3377 +3386 @@
     pIommu->pfnMemRead  = pIommuReg->pfnMemRead;
     pIommu->pfnMemWrite = pIommuReg->pfnMemWrite;
+    pIommu->pfnMsiRemap = pIommuReg->pfnMsiRemap;
     Log(("PDM: Registered IOMMU device '%s'/%d pDevIns=%p\n", pDevIns->pReg->szName, pDevIns->iInstance, pDevIns));