Changeset 20793 in vbox for trunk/src/VBox/Runtime/r0drv

Timestamp:

Jun 22, 2009 5:05:03 PM (16 years ago)

Author:

vboxsync

Message:

semevent*-r0drv-solaris.c(+spinlock-r0drv-solaris.c+the-solaris-kernel.h): Protect the CV with a spinlock to make it possible to signal the sem with preemption disabled.

Location:

trunk/src/VBox/Runtime/r0drv/solaris

Files:

• semevent-r0drv-solaris.c (modified) (2 diffs)
• semeventmulti-r0drv-solaris.c (modified) (1 diff)
• spinlock-r0drv-solaris.c (modified) (1 diff)
• the-solaris-kernel.h (modified) (1 diff)

trunk/src/VBox/Runtime/r0drv/solaris/semevent-r0drv-solaris.c

@@ -33 +33 @@
 *******************************************************************************/
 #include "the-solaris-kernel.h"
-#include <time.h>
 
 #include <iprt/semaphore.h>
@@ -79 +78 @@
         pEventInt->cWaking = 0;
         pEventInt->fSignaled = 0;
-        mutex_init(&pEventInt->Mtx, "IPRT Event Semaphore", MUTEX_DRIVER, NULL);
+        mutex_init(&pEventInt->Mtx, "IPRT Event Semaphore", MUTEX_DRIVER, (void *)ipltospl(DISP_LEVEL));
         cv_init(&pEventInt->Cnd, "IPRT CV", CV_DRIVER, NULL);
         *pEventSem = pEventInt;

trunk/src/VBox/Runtime/r0drv/solaris/semeventmulti-r0drv-solaris.c

@@ -79 +79 @@
         pThis->cWaking = 0;
         pThis->fSignaled = 0;
-        mutex_init(&pThis->Mtx, "IPRT Multiple Release Event Semaphore", MUTEX_DRIVER, NULL);
+        mutex_init(&pThis->Mtx, "IPRT Multiple Release Event Semaphore", MUTEX_DRIVER, (void *)ipltospl(DISP_LEVEL));
         cv_init(&pThis->Cnd, "IPRT CV", CV_DRIVER, NULL);
         *pEventMultiSem = pThis;

trunk/src/VBox/Runtime/r0drv/solaris/spinlock-r0drv-solaris.c

@@ -33 +33 @@
 *******************************************************************************/
 #include "the-solaris-kernel.h"
-#include <sys/spl.h>
 
 #include <iprt/spinlock.h>

trunk/src/VBox/Runtime/r0drv/solaris/the-solaris-kernel.h

@@ -56 +56 @@
 #include <sys/ddi.h>
 #include <sys/sunddi.h>
+#include <sys/spl.h>
 
 #undef u /* /usr/include/sys/user.h:249:1 is where this is defined to (curproc->p_user). very cool. */