Changeset 10827 in vbox

Timestamp:

Jul 23, 2008 11:50:55 AM (17 years ago)

Author:

vboxsync

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

33612

Message:

It's not very clear what this Contention field really means. Seems to be a counter for the number of times contention occurred. (​http://winprogger.com/?p=6)
So the assertion is indeed wrong and it triggers very often here and for others.

File:

• trunk/src/VBox/Runtime/r0drv/nt/semfastmutex-r0drv-nt.cpp (modified) (1 diff)

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

@@ -97 +97 @@
     ASMAtomicIncU32(&pFastInt->u32Magic);
     Assert(pFastInt->Mutex.Count == 1);
-    Assert(pFastInt->Mutex.Contention == 0);
+    /* It's not very clear what this Contention field really means. Seems to be a counter for the number of times contention occurred. (see e.g. http://winprogger.com/?p=6) 
+     * The following assertion is therefor wrong:
+     * Assert(pFastInt->Mutex.Contention == 0);
+     */
     RTMemFree(pFastInt);
     return VINF_SUCCESS;