Changeset 30576 in vbox

Timestamp:

Jul 2, 2010 12:53:46 PM (15 years ago)

Author:

vboxsync

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

63299

Message:

semeventmulti-r0drv-solaris.c: that the version changed is sufficient, testing that it has to be higher is wrong because the value may wrap around.

File:

• trunk/src/VBox/Runtime/r0drv/solaris/semeventmulti-r0drv-solaris.c (modified) (3 diffs)

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

@@ -5 +5 @@
 
 /*
- * Copyright (C) 2006-2007 Oracle Corporation
+ * Copyright (C) 2006-2010 Oracle Corporation
  *
  * This file is part of VirtualBox Open Source Edition (OSE), as
@@ -61 +61 @@
     /** The number of threads in the process of waking up, doubles up as a ref counter. */
     uint32_t volatile   cWaking;
-    /** Object version indicating a valid signal event occurred. */
+    /** Object version that is incremented every time the object is signaled. */
     uint32_t volatile   u32Version;
     /** The Solaris mutex protecting this structure and pairing up the with the cv. */
@@ -280 +280 @@
                 {
                     /* Retured due to call to cv_signal() or cv_broadcast() */
-                    if (u32Version > u32VersionBeforeWait)
+                    if (u32Version != u32VersionBeforeWait)
                     {
                         rc = VINF_SUCCESS;