Message116926
Moving the _PyEval_FiniThreads() call to Py_Initialize() solves the issue:
diff -r 9e49082da463 Python/pythonrun.c
--- a/Python/pythonrun.c Mon Sep 20 12:46:56 2010 +0200
+++ b/Python/pythonrun.c Mon Sep 20 13:41:47 2010 +0200
@@ -217,8 +217,15 @@ Py_InitializeEx(int install_sigs)
Py_FatalError("Py_Initialize: can't make first thread");
(void) PyThreadState_Swap(tstate);
- /* auto-thread-state API, if available */
#ifdef WITH_THREAD
+ /* We can't call _PyEval_FiniThreads() in Py_Finalize because
+ destroying the GIL might fail when it is being referenced from
+ another running thread (see issue #9901).
+ Instead we destroy the previously created GIL here, which ensures
+ that we can call Py_Initialize / Py_Finalize multiple times. */
+ _PyEval_FiniThreads();
+
+ /* Auto-thread-state API */
_PyGILState_Init(interp, tstate);
#endif /* WITH_THREAD */
@@ -514,10 +521,6 @@ Py_Finalize(void)
PyGrammar_RemoveAccelerators(&_PyParser_Grammar);
-#ifdef WITH_THREAD
- _PyEval_FiniThreads();
-#endif
-
#ifdef Py_TRACE_REFS
/* Display addresses (& refcnts) of all objects still alive.
* An address can be used to find the repr of the object, printed |
|
Date |
User |
Action |
Args |
2010-09-20 11:42:08 | pitrou | set | recipients:
+ pitrou, amaury.forgeotdarc |
2010-09-20 11:42:08 | pitrou | set | messageid: <1284982928.81.0.161970660626.issue9901@psf.upfronthosting.co.za> |
2010-09-20 11:42:07 | pitrou | link | issue9901 messages |
2010-09-20 11:42:06 | pitrou | create | |
|