Changeset 88521 in vbox for trunk/src/VBox/Devices/Bus

Timestamp:

Apr 15, 2021 10:56:57 AM (4 years ago)

Author:

vboxsync

Message:

Intel IOMMU: bugref:9967 Updates to adjust for changes in Oct 2020 and Apr 2021 Intel VT-d specs.

File:

• trunk/src/VBox/Devices/Bus/DevIommuIntel.cpp (modified) (6 diffs)

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

@@ -868 +868 @@
     uint8_t const fSlts  = 1;                    /* Second-Level translation support. */
     uint8_t const fPt    = 1;                    /* Pass-Through support. */
+    uint8_t const fSmts  = fFlts & fSlts & fPt;  /* Scalable mode translation support.*/
     uint8_t const fNest  = 0;                    /* Nested translation support. */
-    uint8_t const fSmts  = fFlts & fSlts & fPt;  /* Scalable mode translation support.*/
 
     /* CAP_REG */
@@ -886 +886 @@
                                 X86_PAGE_1G_SHIFT : X86_PAGE_2M_SHIFT) - X86_PAGE_4K_SHIFT;
         uint8_t const fNd     = 2;                              /* Number of domains (0=16, 1=64, 2=256, 3=1K, 4=4K, 5=16K, 6=64K,
-                                                                  7=Reserved). */
+                                                                   7=Reserved). */
         uint8_t const fPsi    = 1;                              /* Page selective invalidation. */
         uint8_t const uMgaw   = cGstPhysAddrBits - 1;           /* Maximum guest address width. */
@@ -892 +892 @@
         uint16_t const offFro = DMAR_MMIO_OFF_FRCD_LO_REG >> 4; /* MMIO offset of FRCD registers. */
 
-        pThis->fCap = RT_BF_MAKE(VTD_BF_CAP_REG_ND,     fNd)
-                    | RT_BF_MAKE(VTD_BF_CAP_REG_AFL,    0)      /* Advanced fault logging not supported. */
-                    | RT_BF_MAKE(VTD_BF_CAP_REG_RWBF,   0)      /* Software need not flush write-buffers. */
-                    | RT_BF_MAKE(VTD_BF_CAP_REG_PLMR,   0)      /* Protected Low-Memory Region not supported. */
-                    | RT_BF_MAKE(VTD_BF_CAP_REG_PHMR,   0)      /* Protected High-Memory Region not supported. */
-                    | RT_BF_MAKE(VTD_BF_CAP_REG_CM,     1)      /** @todo Figure out if required when we impl. caching. */
-                    | RT_BF_MAKE(VTD_BF_CAP_REG_SAGAW,  fSlts & uSagaw)
-                    | RT_BF_MAKE(VTD_BF_CAP_REG_MGAW,   uMgaw)
-                    | RT_BF_MAKE(VTD_BF_CAP_REG_ZLR,    1)      /** @todo Figure out if/how to support zero-length reads. */
-                    | RT_BF_MAKE(VTD_BF_CAP_REG_FRO,    offFro)
-                    | RT_BF_MAKE(VTD_BF_CAP_REG_SLLPS,  fSlts & fSllps)
-                    | RT_BF_MAKE(VTD_BF_CAP_REG_PSI,    fPsi)
-                    | RT_BF_MAKE(VTD_BF_CAP_REG_NFR,    DMAR_FRCD_REG_COUNT - 1)
-                    | RT_BF_MAKE(VTD_BF_CAP_REG_MAMV,   fPsi & fMamv)
-                    | RT_BF_MAKE(VTD_BF_CAP_REG_DWD,    1)
-                    | RT_BF_MAKE(VTD_BF_CAP_REG_FL1GP,  fFlts & fFl1gp)
-                    | RT_BF_MAKE(VTD_BF_CAP_REG_PI,     0)      /* Posted Interrupts not supported. */
-                    | RT_BF_MAKE(VTD_BF_CAP_REG_FL5LP,  fFlts & fFl5lp);
+        pThis->fCap = RT_BF_MAKE(VTD_BF_CAP_REG_ND,      fNd)
+                    | RT_BF_MAKE(VTD_BF_CAP_REG_AFL,     0)     /* Advanced fault logging not supported. */
+                    | RT_BF_MAKE(VTD_BF_CAP_REG_RWBF,    0)     /* Software need not flush write-buffers. */
+                    | RT_BF_MAKE(VTD_BF_CAP_REG_PLMR,    0)     /* Protected Low-Memory Region not supported. */
+                    | RT_BF_MAKE(VTD_BF_CAP_REG_PHMR,    0)     /* Protected High-Memory Region not supported. */
+                    | RT_BF_MAKE(VTD_BF_CAP_REG_CM,      1)     /** @todo Figure out if required when we impl. caching. */
+                    | RT_BF_MAKE(VTD_BF_CAP_REG_SAGAW,   fSlts & uSagaw)
+                    | RT_BF_MAKE(VTD_BF_CAP_REG_MGAW,    uMgaw)
+                    | RT_BF_MAKE(VTD_BF_CAP_REG_ZLR,     1)     /** @todo Figure out if/how to support zero-length reads. */
+                    | RT_BF_MAKE(VTD_BF_CAP_REG_FRO,     offFro)
+                    | RT_BF_MAKE(VTD_BF_CAP_REG_SLLPS,   fSlts & fSllps)
+                    | RT_BF_MAKE(VTD_BF_CAP_REG_PSI,     fPsi)
+                    | RT_BF_MAKE(VTD_BF_CAP_REG_NFR,     DMAR_FRCD_REG_COUNT - 1)
+                    | RT_BF_MAKE(VTD_BF_CAP_REG_MAMV,    fPsi & fMamv)
+                    | RT_BF_MAKE(VTD_BF_CAP_REG_DWD,     1)
+                    | RT_BF_MAKE(VTD_BF_CAP_REG_FL1GP,   fFlts & fFl1gp)
+                    | RT_BF_MAKE(VTD_BF_CAP_REG_PI,      0)     /* Posted Interrupts not supported. */
+                    | RT_BF_MAKE(VTD_BF_CAP_REG_FL5LP,   fFlts & fFl5lp)
+                    | RT_BF_MAKE(VTD_BF_CAP_REG_ESIRTPS, 0)     /* Whether we invalidate interrupt cache on SIRTP flow. */
+                    | RT_BF_MAKE(VTD_BF_CAP_REG_ESRTPS,  0);    /* Whether we invalidate translation cache on SRTP flow. */
         dmarRegWriteRaw64(pThis, VTD_MMIO_OFF_CAP_REG, pThis->fCap);
     }
@@ -921 +923 @@
         uint8_t const  fSrs   = 1;                              /* Supervisor request support. */
         uint8_t const  fEim   = 1;                              /* Extended interrupt mode.*/
+        uint8_t const  fAdms  = 1;                              /* Abort DMA mode support. */
 
         pThis->fExtCap = RT_BF_MAKE(VTD_BF_ECAP_REG_C,      0)  /* Accesses don't snoop CPU cache. */
@@ -937 +940 @@
                        | RT_BF_MAKE(VTD_BF_ECAP_REG_SRS,    fSmts & fSrs)
                        | RT_BF_MAKE(VTD_BF_ECAP_REG_NWFS,   0)  /* 0 as DT not supported. */
-                       | RT_BF_MAKE(VTD_BF_ECAP_REG_EAFS,   0)  /* 0 as PASID not supported. */
+                       | RT_BF_MAKE(VTD_BF_ECAP_REG_EAFS,   0)  /** @todo figure out if EAFS is required? */
                        | RT_BF_MAKE(VTD_BF_ECAP_REG_PSS,    0)  /* 0 as PASID not supported. */
                        | RT_BF_MAKE(VTD_BF_ECAP_REG_PASID,  0)  /* PASID support. */
@@ -948 +951 @@
                        | RT_BF_MAKE(VTD_BF_ECAP_REG_FLTS,   fFlts)
                        | RT_BF_MAKE(VTD_BF_ECAP_REG_SMPWCS, 0)  /* 0 as PASID not supported. */
-                       | RT_BF_MAKE(VTD_BF_ECAP_REG_RPS,    0); /* 0 as PASID not supported. */
+                       | RT_BF_MAKE(VTD_BF_ECAP_REG_RPS,    0)  /* We don't support RID_PASID field in SM context entry. */
+                       | RT_BF_MAKE(VTD_BF_ECAP_REG_ADMS,   fAdms)
+                       | RT_BF_MAKE(VTD_BF_ECAP_REG_RPRIVS, 0); /** @todo figure out if we should/can support this? */
         dmarRegWriteRaw64(pThis, VTD_MMIO_OFF_ECAP_REG, pThis->fExtCap);
     }