Changeset 36979 in vbox for trunk/src

Timestamp:

May 6, 2011 11:55:42 AM (14 years ago)

Author:

vboxsync

Message:

semfastmutex-r0drv-linux.c: Check mutex ownership in strict builds, not just builds with IPRT_DEBUG_SEMS defined.

File:

• trunk/src/VBox/Runtime/r0drv/linux/semfastmutex-r0drv-linux.c (modified) (6 diffs)

trunk/src/VBox/Runtime/r0drv/linux/semfastmutex-r0drv-linux.c

@@ -5 +5 @@
 
 /*
- * Copyright (C) 2006-2007 Oracle Corporation
+ * Copyright (C) 2006-2011 Oracle Corporation
  *
  * This file is part of VirtualBox Open Source Edition (OSE), as
@@ -36 +36 @@
 #include <iprt/asm.h>
 #include <iprt/err.h>
-#ifdef IPRT_DEBUG_SEMS
+#if defined(RT_STRICT) || defined(IPRT_DEBUG_SEMS)
 # include <iprt/thread.h>
 #endif
@@ -55 +55 @@
     /** the linux semaphore. */
     struct semaphore    Semaphore;
-#ifdef IPRT_DEBUG_SEMS
+#if defined(RT_STRICT) || defined(IPRT_DEBUG_SEMS)
     /** For check. */
     RTNATIVETHREAD volatile Owner;
@@ -77 +77 @@
     pThis->u32Magic = RTSEMFASTMUTEX_MAGIC;
     sema_init(&pThis->Semaphore, 1);
-#ifdef IPRT_DEBUG_SEMS
+#if defined(RT_STRICT) || defined(IPRT_DEBUG_SEMS)
     pThis->Owner = NIL_RTNATIVETHREAD;
 #endif
@@ -116 +116 @@
     IPRT_DEBUG_SEMS_STATE(pThis, 'd');
     down(&pThis->Semaphore);
-#ifdef IPRT_DEBUG_SEMS
+#if defined(RT_STRICT) || defined(IPRT_DEBUG_SEMS)
     IPRT_DEBUG_SEMS_STATE(pThis, 'o');
     AssertRelease(pThis->Owner == NIL_RTNATIVETHREAD);
@@ -135 +135 @@
     AssertMsgReturn(pThis->u32Magic == RTSEMFASTMUTEX_MAGIC, ("u32Magic=%RX32 pThis=%p\n", pThis->u32Magic, pThis), VERR_INVALID_HANDLE);
 
-#ifdef IPRT_DEBUG_SEMS
+#if defined(RT_STRICT) || defined(IPRT_DEBUG_SEMS)
     AssertRelease(pThis->Owner == RTThreadNativeSelf());
     ASMAtomicUoWriteSize(&pThis->Owner, NIL_RTNATIVETHREAD);