]> git.ipfire.org Git - thirdparty/Python/cpython.git/commit
Adds a sanity check to avoid a *very rare* infinite loop due to a corrupt tls
authorGregory P. Smith <greg@mad-scientist.com>
Mon, 4 Aug 2008 07:33:37 +0000 (07:33 +0000)
committerGregory P. Smith <greg@mad-scientist.com>
Mon, 4 Aug 2008 07:33:37 +0000 (07:33 +0000)
commitd868be8805b7f5c4f0356eb01f6d8feab43cc9b3
tree7a85f9173c432c9146ff2629eac9557edec6693b
parente7829a5b1b75fdee4f9b7afebeceb98be2d3b0d7
Adds a sanity check to avoid a *very rare* infinite loop due to a corrupt tls
key list data structure in the thread startup path.

This change is a companion to r60148 which already successfully dealt with a
similar issue on thread shutdown.

In particular this loop has been observed happening from this call path:
 #0  in find_key ()
 #1  in PyThread_set_key_value ()
 #2  in _PyGILState_NoteThreadState ()
 #3  in PyThreadState_New ()
 #4  in t_bootstrap ()
 #5  in pthread_start_thread ()

I don't know how this happens but it does, *very* rarely.  On more than
one hardware platform.  I have not been able to reproduce it manually.
(A flaky mutex implementation on the system in question is one hypothesis).

As with r60148, the spinning we managed to observe in the wild was due to a
single list element pointing back upon itself.
Python/pystate.c
Python/thread.c