Changeset 60143 in vbox for trunk/src/VBox

Timestamp:

Mar 22, 2016 10:49:27 PM (9 years ago)

Author:

vboxsync

Message:

RTTimerGetSystemGranularity/nt: Removed assertion on NtExSetTimerResolution return value as it was subject to a race with other threads in the system. Ran into it several times while playing music on W10/skylake.

File:

• trunk/src/VBox/Runtime/r0drv/nt/RTTimerGetSystemGranularity-r0drv-nt.cpp (modified) (2 diffs)

trunk/src/VBox/Runtime/r0drv/nt/RTTimerGetSystemGranularity-r0drv-nt.cpp

@@ -5 +5 @@
 
 /*
- * Copyright (C) 2006-2015 Oracle Corporation
+ * Copyright (C) 2006-2016 Oracle Corporation
  *
  * This file is part of VirtualBox Open Source Edition (OSE), as
@@ -55 +55 @@
      * count of these calls, we have to do two calls that cancel each other out.
      */
-    ULONG ulResolution1 = g_pfnrtNtExSetTimerResolution(ulTimeInc, TRUE);
-    ULONG ulResolution2 = g_pfnrtNtExSetTimerResolution(0 /*ignored*/, FALSE);
-    AssertMsg(ulResolution1 == ulResolution2, ("%ld, %ld\n", ulResolution1, ulResolution2)); /* not supposed to change it! */
-    return ulResolution2 * 100; /* NT -> ns */
+    g_pfnrtNtExSetTimerResolution(ulTimeInc, TRUE);
+    ULONG ulResolution = g_pfnrtNtExSetTimerResolution(0 /*ignored*/, FALSE);
+    return ulResolution * 100; /* NT -> ns */
 }