Changeset 96960 in vbox for trunk/src/VBox

Timestamp:

Oct 1, 2022 8:00:19 AM (2 years ago)

Author:

vboxsync

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

153879

Message:

VMM/IEM: Nested VMX: bugref:10092 Fix for nested VMs without nested EPT until PGM_WITH_NESTED_APIC_ACCESS_PAGE is sorted out.

File:

• trunk/src/VBox/VMM/VMMAll/IEMAllCImplVmxInstr.cpp (modified) (1 diff)

trunk/src/VBox/VMM/VMMAll/IEMAllCImplVmxInstr.cpp ¶

@@ -9972 +9972 @@
 
             /*
+             * WARNING: HACK AHEAD!
+             * The below is a the old behavior which composes an incomplete APIC-access VM-exit.
+             * This will result in the inner hypervisor emulating the access since it lacks info
+             * for a linear read/write accesses. Unfortunately, this is required till the
+             * virtual VMX APIC-access page mapping using PGM_WITH_NESTED_APIC_ACCESS_PAGE is
+             * fixed properly.
+             */
+            if (HmExitAux.Vmx.uReason == VMX_EXIT_EPT_MISCONFIG)
+            {
+                VMXVEXITINFO ExitInfo;
+                RT_ZERO(ExitInfo);
+                ExitInfo.uReason = VMX_EXIT_APIC_ACCESS;
+                ExitInfo.u64Qual = RT_BF_MAKE(VMX_BF_EXIT_QUAL_APIC_ACCESS_OFFSET, offAccess)
+                                 | RT_BF_MAKE(VMX_BF_EXIT_QUAL_APIC_ACCESS_TYPE,   VMXAPICACCESS_PHYSICAL_INSTR);
+                ExitInfo.cbInstr = 0;
+
+                VMXVEXITEVENTINFO ExitEventInfo;
+                RT_ZERO(ExitEventInfo);
+                ExitEventInfo.uIdtVectoringInfo    = HmExitAux.Vmx.uIdtVectoringInfo;
+                ExitEventInfo.uIdtVectoringErrCode = HmExitAux.Vmx.uIdtVectoringErrCode;
+
+                LogFlowFunc(("Raising APIC-access VM-exit as phys access from #PF handler at offset %#x\n", offAccess));
+                VBOXSTRICTRC rcStrict = iemVmxVmexitApicAccessWithInfo(pVCpu, &ExitInfo, &ExitEventInfo);
+                return iemExecStatusCodeFiddling(pVCpu, rcStrict);
+            }
+
+            /*
              * Verify the VM-exit reason must be an EPT violation.
              * Other accesses should go through the other handler (iemVmxApicAccessPageHandler).