[Python-bugs-list] [ python-Bugs-513033 ] unsafe call to PyThreadState_Swap
noreply@sourceforge.net
noreply@sourceforge.net
Mon, 04 Feb 2002 15:41:41 -0800
Bugs item #513033, was opened at 2002-02-04 15:16
You can respond by visiting:
http://sourceforge.net/tracker/?func=detail&atid=105470&aid=513033&group_id=5470
Category: Python Library
Group: Python 2.1.2
Status: Open
Resolution: None
Priority: 5
Submitted By: Jake McGuire (jamcguir)
>Assigned to: Guido van Rossum (gvanrossum)
Summary: unsafe call to PyThreadState_Swap
Initial Comment:
It appears that there is a blatantly unsafe call to
PyThreadState_Swap in the functions on_hook and
on_completer in Modules/Readline.c
The diff adding these calls is viewable at
http://cvs.sourceforge.net/cgi-
bin/viewcvs.cgi/python/python/dist/src/Modules/readline
.c.diff?r1=2.5&r2=2.6&only_with_tag=MAIN
The call to PyThreadState_Swap is added directly below
a comment pointing out that readline() is called with
the interpreter lock released. Viewing the code shows
that the interpreter lock is indeed released before
calling readline (in myreadline.c).
Multithreaded programs that define callback functions
suffer from intermittent crashes, often Py_FatalError-
ing claiming "tstate mix-up" from ceval.c
Removing the calls to PyThreadState_Swap makes these
problems go away.
Can someone explain how the call to PyThreadState_Swap
is indeed the right thing to be doing?
----------------------------------------------------------------------
>Comment By: Tim Peters (tim_one)
Date: 2002-02-04 15:41
Message:
Logged In: YES
user_id=31435
Guido's checkin comment said:
"""
Darn. When thread support is disabled, the BEGIN/END
macros don't save and restore the tstate, but explicitly
calling PyEval_SaveThread() does reset it! While I think
about how to fix this for real, here's a fix that avoids
getting a fatal error.
"""
Therefore I assigned the bug to Guido <wink>. It would
help if you could describe a specific simple scenario that
provokes the problems you're seeing.
----------------------------------------------------------------------
You can respond by visiting:
http://sourceforge.net/tracker/?func=detail&atid=105470&aid=513033&group_id=5470