[Python-checkins] cpython: Issue #11870: threading: Properly reinitialize threads internal locks and
charles-francois.natali
python-checkins at python.org
Sun Dec 18 18:48:33 CET 2011
http://hg.python.org/cpython/rev/cec0d77d01c4
changeset: 74038:cec0d77d01c4
parent: 74035:764234983120
user: Charles-François Natali <neologix at free.fr>
date: Sun Dec 18 18:45:16 2011 +0100
summary:
Issue #11870: threading: Properly reinitialize threads internal locks and
condition variables to avoid deadlocks in child processes.
files:
Lib/threading.py | 11 ++++-------
Misc/NEWS | 3 +++
2 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/Lib/threading.py b/Lib/threading.py
--- a/Lib/threading.py
+++ b/Lib/threading.py
@@ -1047,21 +1047,18 @@
current = current_thread()
with _active_limbo_lock:
for thread in _active.values():
+ # Any lock/condition variable may be currently locked or in an
+ # invalid state, so we reinitialize them.
+ thread._reset_internal_locks()
if thread is current:
# There is only one active thread. We reset the ident to
# its new value since it can have changed.
ident = get_ident()
thread._ident = ident
- # Any condition variables hanging off of the active thread may
- # be in an invalid state, so we reinitialize them.
- thread._reset_internal_locks()
new_active[ident] = thread
else:
# All the others are already stopped.
- # We don't call _Thread__stop() because it tries to acquire
- # thread._Thread__block which could also have been held while
- # we forked.
- thread._stopped = True
+ thread._stop()
_limbo.clear()
_active.clear()
diff --git a/Misc/NEWS b/Misc/NEWS
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -419,6 +419,9 @@
Library
-------
+- Issue #11870: threading: Properly reinitialize threads internal locks and
+ condition variables to avoid deadlocks in child processes.
+
- Issue #8035: urllib: Fix a bug where the client could remain stuck after a
redirection or an error.
--
Repository URL: http://hg.python.org/cpython
More information about the Python-checkins
mailing list