Changeset 90028 in vbox for trunk/src/VBox/Main/src-client

Timestamp:

Jul 5, 2021 2:25:35 PM (4 years ago)

Author:

vboxsync

Message:

AMD IOMMU: bugref:9654 Fixed address translations resulting in non-contiguous physical regions.
Fixed IOMMU from trying to remap MSIs generated by itself (e.g, IOPF).
Fixed destroying the IOTLB cache properly on VM reset.
Fixed some missing conditions in I/O page walk (reserved bits checks).
Slightly faster IOVA skipped bits checking in I/O page walk.
Added a couple of extra statistics to track non-contiguous translations and usage of non-standard page sizes.

File:

• trunk/src/VBox/Main/src-client/ConsoleImpl2.cpp (modified) (1 diff)

trunk/src/VBox/Main/src-client/ConsoleImpl2.cpp

@@ -1603 +1603 @@
                 InsertConfigNode(pInst,    "Config", &pCfg);
                 hrc = pBusMgr->assignPCIDevice("iommu-amd", pInst);                         H();
+
+                /* The AMD IOMMU device needs to know which PCI slot it's in, see @bugref{9654#c104}. */
+                {
+                    PCIBusAddress Address;
+                    if (pBusMgr->findPCIAddress("iommu-amd", 0, Address))
+                    {
+                        uint32_t const u32IommuAddress = (Address.miDevice << 16) | Address.miFn;
+                        InsertConfigInteger(pCfg, "PCIAddress", u32IommuAddress);
+                    }
+                    else
+                        return VMR3SetError(pUVM, VERR_INVALID_PARAMETER, RT_SRC_POS,
+                                            N_("Failed to find PCI address of the assigned IOMMU device!"));
+                }
 
                 PCIBusAddress PCIAddr = PCIBusAddress((int32_t)uIoApicPciAddress);