Changeset 54181 in vbox

Timestamp:

Feb 12, 2015 5:34:24 PM (10 years ago)

Author:

vboxsync

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

98237

Message:

HostDrivers/support: fix reinit of GipCpu data where TSC-delta isn't really relevant.

Location:

trunk

Files:

• include/VBox/sup.h (modified) (1 diff)
• src/VBox/HostDrivers/Support/SUPDrv.c (modified) (2 diffs)

trunk/include/VBox/sup.h

@@ -1649 +1649 @@
         uint64_t u64Tsc = UINT64_MAX;
         int rc = SUPGetTsc(&u64Tsc, NULL /* pidApic */);
-#ifndef DEBUG_michael
+#ifdef DEBUG_ramshankar
         AssertRC(rc);
 #endif

trunk/src/VBox/HostDrivers/Support/SUPDrv.c

@@ -3917 +3917 @@
 static void supdrvGipReInitCpu(PSUPGLOBALINFOPAGE pGip, PSUPGIPCPU pGipCpu, uint64_t u64NanoTS)
 {
-    pGipCpu->u64TSC    = SUPReadTsc() - pGipCpu->u32UpdateIntervalTSC;
+    /*
+     * Here we don't really care about applying the TSC delta. The re-initialization of this
+     * value is not relevant especially while (re)starting the GIP as the first few ones will
+     * be ignored anyway, see supdrvGipDoUpdateCpu().
+     */
+    pGipCpu->u64TSC    = ASMReadTSC() - pGipCpu->u32UpdateIntervalTSC;
     pGipCpu->u64NanoTS = u64NanoTS;
 }
@@ -7586 +7591 @@
 
 /**
- * Worker routine for supdrvGipUpdate and supdrvGipUpdatePerCpu that
+ * Worker routine for supdrvGipUpdate() and supdrvGipUpdatePerCpu() that
  * updates all the per cpu data except the transaction id.
  *