Changeset 97157 in vbox

Timestamp:

Oct 14, 2022 12:13:50 PM (2 years ago)

Author:

vboxsync

Message:

VMM/IEM: Nested VMX: bugref:10092 todo regarding eager de-registeration of the VMX APIC-access page.

File:

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

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

@@ -10000 +10000 @@
     }
 
+    /** @todo This isn't ideal but works for now as nested-hypervisors generally play
+     *        nice because the spec states that this page should be modified only when
+     *        no CPU refers to it VMX non-root mode. Nonetheless, we could use an atomic
+     *        reference counter to ensure the aforementioned condition before
+     *        de-registering the page.  */
     LogFunc(("Accessed outside VMX non-root mode, deregistering page handler for %#RGp\n", GCPhysPage));
     int const rc = PGMHandlerPhysicalDeregister(pVM, GCPhysPage);