Issue #8407: signal.sigwait() releases the GIL

Victor Stinner · Victor Stinner · commit 10c30d676432 · 2011-06-10T01:39:53.000+02:00
Initial patch by Charles-François Natali.
diff --git a/Lib/test/test_signal.py b/Lib/test/test_signal.py
@@ -607,6 +607,25 @@ def test_sigwait(self):
         signal.alarm(1)
         self.assertEqual(signal.sigwait([signal.SIGALRM]), signal.SIGALRM)
 
+    @unittest.skipUnless(hasattr(signal, 'sigwait'),
+                         'need signal.sigwait()')
+    @unittest.skipIf(threading is None, "test needs threading module")
+    def test_sigwait_thread(self):
+        signum = signal.SIGUSR1
+        old_handler = signal.signal(signum, self.handler)
+        self.addCleanup(signal.signal, signum, old_handler)
+
+        def kill_later():
+            time.sleep(1)
+            os.kill(os.getpid(), signum)
+
+        killer = threading.Thread(target=kill_later)
+        killer.start()
+        try:
+            self.assertEqual(signal.sigwait([signum]), signum)
+        finally:
+            killer.join()
+
     @unittest.skipUnless(hasattr(signal, 'pthread_sigmask'),
                          'need signal.pthread_sigmask()')
     def test_pthread_sigmask_arguments(self):
diff --git a/Modules/signalmodule.c b/Modules/signalmodule.c
@@ -662,7 +662,9 @@ signal_sigwait(PyObject *self, PyObject *args)
     if (iterable_to_sigset(signals, &set))
         return NULL;
 
+    Py_BEGIN_ALLOW_THREADS
     err = sigwait(&set, &signum);
+    Py_END_ALLOW_THREADS
     if (err) {
         errno = err;
         return PyErr_SetFromErrno(PyExc_OSError);
