Fix deadlock issue in signal handling of sleeping threads

ryoqun · Feb 6, 2013 · d8fcda5 · d8fcda5
1 parent 5a1b06e
commit d8fcda5
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/vm/park.cpp b/vm/park.cpp
@@ -18,10 +18,13 @@ namespace rubinius {
 
         cond_.wait(mutex_);
       }
+      mutex_.unlock();
       if(!state->check_async(call_frame)) {
+        mutex_.lock();
         result = NULL;
         break;
       }
+      mutex_.lock();
     }
 
     sleeping_ = false;
@@ -49,10 +52,13 @@ namespace rubinius {
           break;
         }
       }
+      mutex_.unlock();
       if(!state->check_async(call_frame)) {
+        mutex_.lock();
         timeout = NULL;
         break;
       }
+      mutex_.lock();
     }
 
     sleeping_ = false;