Changeset 20089 in vbox for trunk/src/VBox/VMM/VMMAll

Timestamp:

May 27, 2009 2:53:47 PM (16 years ago)

Author:

vboxsync

Message:

TM: The critsect assertions.

File:

• trunk/src/VBox/VMM/VMMAll/TMAll.cpp (modified) (4 diffs)

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

@@ -42 +42 @@
 #ifdef IN_RING3
 # include <iprt/thread.h>
+#endif
+
+
+/*******************************************************************************
+*   Defined Constants And Macros                                               *
+*******************************************************************************/
+/** @def TMTIMER_ASSERT_CRITSECT
+ * Checks that the caller owns the critical section if one is associated with
+ * the timer. */
+#ifdef VBOX_STRICT
+# define TMTIMER_ASSERT_CRITSECT(pTimer) \
+    do { \
+        if ((pTimer)->pCritSect) \
+        { \
+            PPDMCRITSECT pCritSect = (PPDMCRITSECT)MMHyperR3ToCC((pTimer)->CTX_SUFF(pVM), (pTimer)->pCritSect); \
+            Assert(pCritSect && PDMCritSectIsOwner(pCritSect)); \
+        } \
+    } while (0)
+#else
+# define TMTIMER_ASSERT_CRITSECT(pTimer) do { } while (0)
 #endif
 
@@ -731 +751 @@
 {
     STAM_PROFILE_START(&pTimer->CTX_SUFF(pVM)->tm.s.CTXALLSUFF(StatTimerSet), a);
+    TMTIMER_ASSERT_CRITSECT(pTimer);
 
     /** @todo find the most frequently used paths and make them skip tmSchedule and tmTimerTryWithLink. */
@@ -942 +963 @@
 {
     STAM_PROFILE_START(&pTimer->CTX_SUFF(pVM)->tm.s.CTXALLSUFF(StatTimerStop), a);
+    TMTIMER_ASSERT_CRITSECT(pTimer);
+
     /** @todo see if this function needs optimizing. */
     int cRetries = 1000;
@@ -1326 +1349 @@
 VMMDECL(uint64_t) TMTimerGetExpire(PTMTIMER pTimer)
 {
+    TMTIMER_ASSERT_CRITSECT(pTimer);
     int cRetries = 1000;
     do