Changeset 20678 in vbox for trunk/src/VBox/VMM

Timestamp:

Jun 18, 2009 9:22:30 AM (16 years ago)

Author:

vboxsync

Message:

TM: Count calls to TMCpuTickSet and take the VM handle as an argument.

Location:

trunk/src/VBox/VMM

Files:

• TM.cpp (modified) (1 diff)
• TMInternal.h (modified) (1 diff)
• VMMAll/TMAllCpu.cpp (modified) (1 diff)

trunk/src/VBox/VMM/TM.cpp

@@ -602 +602 @@
     STAM_REG(pVM, &pVM->tm.s.StatTSCSyncNotTicking,                   STAMTYPE_COUNTER, "/TM/TSC/Intercept/SyncNotTicking",    STAMUNIT_OCCURENCES, "VirtualSync isn't ticking.");
     STAM_REG(pVM, &pVM->tm.s.StatTSCWarp,                             STAMTYPE_COUNTER, "/TM/TSC/Intercept/Warp",              STAMUNIT_OCCURENCES, "Warpdrive is active.");
+    STAM_REG(pVM, &pVM->tm.s.StatTSCSet,                              STAMTYPE_COUNTER, "/TM/TSC/Sets",                        STAMUNIT_OCCURENCES, "Calls to TMCpuTickSet.");
 
     STAM_REG(pVM, &pVM->tm.s.StatVirtualSyncCatchup,              STAMTYPE_PROFILE_ADV, "/TM/VirtualSync/CatchUp",    STAMUNIT_TICKS_PER_OCCURENCE, "Counting and measuring the times spent catching up.");

trunk/src/VBox/VMM/TMInternal.h

@@ -508 +508 @@
     STAMCOUNTER                 StatTimerCallbackSetFF;
 
+    /** Calls to TMCpuTickSet. */
+    STAMCOUNTER                 StatTSCSet;
+
     /** @name Reasons for refusing TSC offsetting in TMCpuTickCanUseRealTSC.
      * @{ */

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

@@ -245 +245 @@
  *
  * @returns VBox status code.
- * @param   pVCpu       The VMCPU to operate on.
+ * @param   pVM         The VM handle.
+ * @param   pVCpu       The virtual CPU to operate on.
  * @param   u64Tick     The new timestamp value.
- */
-VMMDECL(int) TMCpuTickSet(PVMCPU pVCpu, uint64_t u64Tick)
-{
+ *
+ * @thread  EMT which TSC is to be set.
+ */
+VMMDECL(int) TMCpuTickSet(PVM pVM, PVMCPU pVCpu, uint64_t u64Tick)
+{
+    VMCPU_ASSERT_EMT(pVCpu);
+    STAM_COUNTER_INC(&pVM->tm.s.StatTSCSet);
+
     /*
      * This is easier to do when the TSC is paused since resume will
      * do all the calcuations for us.
      */
-    PVM     pVM         = pVCpu->CTX_SUFF(pVM);
     bool    fTSCTicking = pVCpu->tm.s.fTSCTicking;
     if (fTSCTicking)
         tmCpuTickPause(pVM, pVCpu);
+
     pVCpu->tm.s.u64TSC = u64Tick;
+    /** @todo Try help synchronizing it better among the virtual CPUs? */
 
     if (fTSCTicking)
         tmCpuTickResume(pVM, pVCpu);
-    /** @todo Try help synchronizing it better among the virtual CPUs? */
     return VINF_SUCCESS;
 }