mirror of
https://github.com/python/cpython.git
synced 2025-07-28 21:55:21 +00:00
#1683 prevent forking from interfering in threading storage
This should prevent some test_multiprocessing failures
This commit is contained in:
parent
26305a03e3
commit
114f7e5fff
4 changed files with 37 additions and 0 deletions
|
@ -40,6 +40,9 @@ PyAPI_FUNC(int) PyThread_set_key_value(int, void *);
|
||||||
PyAPI_FUNC(void *) PyThread_get_key_value(int);
|
PyAPI_FUNC(void *) PyThread_get_key_value(int);
|
||||||
PyAPI_FUNC(void) PyThread_delete_key_value(int key);
|
PyAPI_FUNC(void) PyThread_delete_key_value(int key);
|
||||||
|
|
||||||
|
/* Cleanup after a fork */
|
||||||
|
PyAPI_FUNC(void) PyThread_ReInitTLS(void);
|
||||||
|
|
||||||
#ifdef __cplusplus
|
#ifdef __cplusplus
|
||||||
}
|
}
|
||||||
#endif
|
#endif
|
||||||
|
|
|
@ -925,5 +925,6 @@ PyOS_AfterFork(void)
|
||||||
main_thread = PyThread_get_thread_ident();
|
main_thread = PyThread_get_thread_ident();
|
||||||
main_pid = getpid();
|
main_pid = getpid();
|
||||||
_PyImport_ReInitLock();
|
_PyImport_ReInitLock();
|
||||||
|
PyThread_ReInitTLS();
|
||||||
#endif
|
#endif
|
||||||
}
|
}
|
||||||
|
|
|
@ -2,6 +2,7 @@
|
||||||
/* Check for interrupts */
|
/* Check for interrupts */
|
||||||
|
|
||||||
#include "Python.h"
|
#include "Python.h"
|
||||||
|
#include "pythread.h"
|
||||||
|
|
||||||
#ifdef QUICKWIN
|
#ifdef QUICKWIN
|
||||||
|
|
||||||
|
@ -172,5 +173,6 @@ PyOS_AfterFork(void)
|
||||||
{
|
{
|
||||||
#ifdef WITH_THREAD
|
#ifdef WITH_THREAD
|
||||||
PyEval_ReInitThreads();
|
PyEval_ReInitThreads();
|
||||||
|
PyThread_ReInitTLS();
|
||||||
#endif
|
#endif
|
||||||
}
|
}
|
||||||
|
|
|
@ -381,4 +381,35 @@ PyThread_delete_key_value(int key)
|
||||||
PyThread_release_lock(keymutex);
|
PyThread_release_lock(keymutex);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/* Forget everything not associated with the current thread id.
|
||||||
|
* This function is called from PyOS_AfterFork(). It is necessary
|
||||||
|
* because other thread ids which were in use at the time of the fork
|
||||||
|
* may be reused for new threads created in the forked process.
|
||||||
|
*/
|
||||||
|
void
|
||||||
|
PyThread_ReInitTLS(void)
|
||||||
|
{
|
||||||
|
long id = PyThread_get_thread_ident();
|
||||||
|
struct key *p, **q;
|
||||||
|
|
||||||
|
if (!keymutex)
|
||||||
|
return;
|
||||||
|
|
||||||
|
/* As with interpreter_lock in PyEval_ReInitThreads()
|
||||||
|
we just create a new lock without freeing the old one */
|
||||||
|
keymutex = PyThread_allocate_lock();
|
||||||
|
|
||||||
|
/* Delete all keys which do not match the current thread id */
|
||||||
|
q = &keyhead;
|
||||||
|
while ((p = *q) != NULL) {
|
||||||
|
if (p->id != id) {
|
||||||
|
*q = p->next;
|
||||||
|
free((void *)p);
|
||||||
|
/* NB This does *not* free p->value! */
|
||||||
|
}
|
||||||
|
else
|
||||||
|
q = &p->next;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
#endif /* Py_HAVE_NATIVE_TLS */
|
#endif /* Py_HAVE_NATIVE_TLS */
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue