Changeset 23042 in vbox for trunk/src

Timestamp:

Sep 15, 2009 8:19:17 PM (15 years ago)

Author:

vboxsync

Message:

VM.cpp: Note about ResetLS/ResettingLS.

File:

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

trunk/src/VBox/VMM/VM.cpp

@@ -1468 +1468 @@
             rc = rc2;
     }
+/** @todo VMR3Reset during live save (ResetLS, ResettingLS) needs to be
+ *        redone. We should suspend the VM after resetting the state, not
+ *        cancelling the save operation. In the live migration scenario we
+ *        would already have transfered most of the state and the little that
+ *        remains after a reset isn't going to be very big and it's not worth
+ *        making special paths for this. In the live snapshot case, there
+ *        would be a gain in that we wouldn't require a potentially large saved
+ *        state file. But that could be handled on VMR3Save return and size
+ *        shouldn't matter much as already mentioned..
+ *
+ *        Will address this tomorrow. */
     return rc;
 }