Changeset 81564 in vbox for trunk/src/VBox/VMM

Timestamp:

Oct 29, 2019 10:38:26 AM (5 years ago)

Author:

vboxsync

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

134304

Message:

VMM/PDMDevHlp: Adding helpers for getting the mapping addresses of I/O port and MMIO regions, mainly for info handlers and logging (thus only ring-3). bugref:9218

Location:

trunk/src/VBox/VMM/VMMR3

Files:

• IOMR3IoPort.cpp (modified) (1 diff)
• IOMR3Mmio.cpp (modified) (1 diff)
• PDMDevHlp.cpp (modified) (6 diffs)

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

@@ -548 +548 @@
 
 /**
+ * Gets the mapping address of I/O ports @a hIoPorts.
+ *
+ * @returns Mapping address if mapped, UINT32_MAX if not mapped or invalid
+ *          input.
+ * @param   pVM         The cross context VM structure.
+ * @param   pDevIns     The device which allegedly owns @a hRegion.
+ * @param   hIoPorts    The handle to I/O port region.
+ */
+VMMR3_INT_DECL(uint32_t) IOMR3IoPortGetMappingAddress(PVM pVM, PPDMDEVINS pDevIns, IOMIOPORTHANDLE hIoPorts)
+{
+    AssertPtrReturn(pDevIns, UINT32_MAX);
+    AssertReturn(hIoPorts < RT_MIN(pVM->iom.s.cMmioRegs, pVM->iom.s.cMmioAlloc), UINT32_MAX);
+    IOMIOPORTENTRYR3 volatile * const pRegEntry = &pVM->iom.s.paIoPortRegs[hIoPorts];
+    AssertReturn(pRegEntry->pDevIns == pDevIns, UINT32_MAX);
+    for (uint32_t iTry = 0; ; iTry++)
+    {
+        bool        fMapped = pRegEntry->fMapped;
+        RTIOPORT    uPort   = pRegEntry->uPort;
+        if (   (   ASMAtomicReadBool(&pRegEntry->fMapped) == fMapped
+                && uPort == pRegEntry->uPort)
+            || iTry > 1024)
+            return fMapped ? uPort : UINT32_MAX;
+        ASMNopPause();
+    }
+}
+
+
+/**
  * Display a single I/O port ring-3 range.
  *

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

@@ -488 +488 @@
 
 /**
+ * Gets the mapping address of MMIO region @a hRegion.
+ *
+ * @returns Mapping address if mapped, NIL_RTGCPHYS if not mapped or invalid
+ *          input.
+ * @param   pVM         The cross context VM structure.
+ * @param   pDevIns     The device which allegedly owns @a hRegion.
+ * @param   hRegion     The handle to validate.
+ */
+VMMR3_INT_DECL(RTGCPHYS) IOMR3MmioGetMappingAddress(PVM pVM, PPDMDEVINS pDevIns, IOMMMIOHANDLE hRegion)
+{
+    AssertPtrReturn(pDevIns, NIL_RTGCPHYS);
+    AssertReturn(hRegion < RT_MIN(pVM->iom.s.cMmioRegs, pVM->iom.s.cMmioAlloc), NIL_RTGCPHYS);
+    PIOMMMIOENTRYR3 const pRegEntry = &pVM->iom.s.paMmioRegs[hRegion];
+    AssertReturn(pRegEntry->pDevIns == pDevIns, NIL_RTGCPHYS);
+    return pRegEntry->GCPhysMapping;
+}
+
+
+/**
  * Display a single MMIO range.
  *

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

@@ -145 +145 @@
     return rc;
 }
+
+
+/** @interface_method_impl{PDMDEVHLPR3,pfnIoPortGetMappingAddress} */
+static DECLCALLBACK(uint32_t) pdmR3DevHlp_IoPortGetMappingAddress(PPDMDEVINS pDevIns, IOMIOPORTHANDLE hIoPorts)
+{
+    PDMDEV_ASSERT_DEVINS(pDevIns);
+    LogFlow(("pdmR3DevHlp_IoPortGetMappingAddress: caller='%s'/%d: hIoPorts=%#x\n", pDevIns->pReg->szName, pDevIns->iInstance, hIoPorts));
+
+    uint32_t uAddress = IOMR3IoPortGetMappingAddress(pDevIns->Internal.s.pVMR3, pDevIns, hIoPorts);
+
+    LogFlow(("pdmR3DevHlp_IoPortGetMappingAddress: caller='%s'/%d: returns %#RX32\n", pDevIns->pReg->szName, pDevIns->iInstance, uAddress));
+    return uAddress;
+}
+
 
 /** @interface_method_impl{PDMDEVHLPR3,pfnIOPortRegister} */
@@ -395 +409 @@
     LogFlow(("pdmR3DevHlp_MmioReduce: caller='%s'/%d: returns %Rrc\n", pDevIns->pReg->szName, pDevIns->iInstance, rc));
     return rc;
+}
+
+
+/** @interface_method_impl{PDMDEVHLPR3,pfnMmioGetMappingAddress} */
+static DECLCALLBACK(RTGCPHYS) pdmR3DevHlp_MmioGetMappingAddress(PPDMDEVINS pDevIns, IOMMMIOHANDLE hRegion)
+{
+    PDMDEV_ASSERT_DEVINS(pDevIns);
+    LogFlow(("pdmR3DevHlp_MmioGetMappingAddress: caller='%s'/%d: hRegion=%#x\n", pDevIns->pReg->szName, pDevIns->iInstance, hRegion));
+
+    RTGCPHYS GCPhys = IOMR3MmioGetMappingAddress(pDevIns->Internal.s.pVMR3, pDevIns, hRegion);
+
+    LogFlow(("pdmR3DevHlp_MmioGetMappingAddress: caller='%s'/%d: returns %RGp\n", pDevIns->pReg->szName, pDevIns->iInstance, GCPhys));
+    return GCPhys;
 }
 
@@ -4454 +4481 @@
     pdmR3DevHlp_IoPortMap,
     pdmR3DevHlp_IoPortUnmap,
+    pdmR3DevHlp_IoPortGetMappingAddress,
     pdmR3DevHlp_IOPortRegister,
     pdmR3DevHlp_IOPortRegisterRC,
@@ -4462 +4490 @@
     pdmR3DevHlp_MmioUnmap,
     pdmR3DevHlp_MmioReduce,
+    pdmR3DevHlp_MmioGetMappingAddress,
     pdmR3DevHlp_MMIORegister,
     pdmR3DevHlp_MMIORegisterRC,
@@ -4922 +4951 @@
     pdmR3DevHlp_IoPortMap,
     pdmR3DevHlp_IoPortUnmap,
+    pdmR3DevHlp_IoPortGetMappingAddress,
     pdmR3DevHlp_IOPortRegister,
     pdmR3DevHlp_IOPortRegisterRC,
@@ -4930 +4960 @@
     pdmR3DevHlp_MmioUnmap,
     pdmR3DevHlp_MmioReduce,
+    pdmR3DevHlp_MmioGetMappingAddress,
     pdmR3DevHlp_MMIORegister,
     pdmR3DevHlp_MMIORegisterRC,