Changeset 47959 in vbox

Timestamp:

Aug 21, 2013 9:46:55 AM (11 years ago)

Author:

vboxsync

Message:

VMM/HMR0: Comment update.

File:

• trunk/src/VBox/VMM/VMMR0/HMR0.cpp (modified) (2 diffs)

trunk/src/VBox/VMM/VMMR0/HMR0.cpp

@@ -1399 +1399 @@
  *
  * @param   pvCpu       Pointer to the VMCPU.
+ *
+ * @remarks No-long-jump zone!!!
  */
 VMMR0_INT_DECL(void) HMR0EnterEx(PVMCPU pVCpu)
@@ -1455 +1457 @@
     AssertRC(rc);
 
-    /* We must save the host context here (VT-x) as we might be rescheduled on
-       a different cpu after a long jump back to ring 3. */
-    /** @todo This will change with preemption hooks. */
+    /* Load the host as we may be resuming code after a longjmp and quite
+       possibly be scheduled on a different CPU. */
     rc |= g_HvmR0.pfnSaveHostState(pVM, pVCpu);
     AssertRC(rc);
 
+    /** @todo This is not needed to be done here anymore, can fix/optimize later. */
     rc |= g_HvmR0.pfnLoadGuestState(pVM, pVCpu, pCtx);
     AssertRC(rc);