From: Antoine Pitrou <solipsis@pitrou.net>
Date: Sat, 15 Jan 2011 11:37:11 +0000 (+0000)
Subject: Fix the GIL with subinterpreters. Hopefully this will allow mod_wsgi to work with... 
X-Git-Tag: v3.2rc1~15
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=6a002c02dad9e33847f6f123be499c998e40cccf;p=python

Fix the GIL with subinterpreters. Hopefully this will allow mod_wsgi to work with 3.2.
(we need some tests for this)
---

diff --git a/Python/ceval_gil.h b/Python/ceval_gil.h
index 40e45f75f2..bf7a350791 100644
--- a/Python/ceval_gil.h
+++ b/Python/ceval_gil.h
@@ -334,12 +334,15 @@ static void recreate_gil(void)
 
 static void drop_gil(PyThreadState *tstate)
 {
-    /* NOTE: tstate is allowed to be NULL. */
     if (!_Py_atomic_load_relaxed(&gil_locked))
         Py_FatalError("drop_gil: GIL is not locked");
-    if (tstate != NULL &&
-        tstate != _Py_atomic_load_relaxed(&gil_last_holder))
-        Py_FatalError("drop_gil: wrong thread state");
+    /* tstate is allowed to be NULL (early interpreter init) */
+    if (tstate != NULL) {
+        /* Sub-interpreter support: threads might have been switched
+           under our feet using PyThreadState_Swap(). Fix the GIL last
+           holder variable so that our heuristics work. */
+        _Py_atomic_store_relaxed(&gil_last_holder, tstate);
+    }
 
     MUTEX_LOCK(gil_mutex);
     _Py_ANNOTATE_RWLOCK_RELEASED(&gil_locked, /*is_write=*/1);