Message 123244 - Python tracker

➜

This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author	vstinner
Recipients	amaury.forgeotdarc, gregory.p.smith, isandler, pitrou, stutzbach, vstinner
Date	2010-12-03.13:30:38
SpamBayes Score	2.8289367e-07
Marked as misclassified	No
Message-id	<[email protected]>
In-reply-to

Content
Ok, so +1 to apply immediatly your patch which "fixes" the deadlock. If someone is motived to make Buffered* classes reentrant, (s)he can remove this exception. io and signal documentation should also be improved to indicate that using buffered I/O in a signal handler may raise a RuntimeError on reentrant call (and give an example to explain the problem?). About the patch: can't you move "&& (self->owner = PyThread_get_thread_ident(), 1) )" in _enter_buffered_busy()?

Ok, so +1 to apply immediatly your patch which "fixes" the deadlock. If someone is motived to make Buffered* classes reentrant, (s)he can remove this exception.

io and signal documentation should also be improved to indicate that using buffered I/O in a signal handler may raise a RuntimeError on reentrant call (and give an example to explain the problem?).

About the patch: can't you move "&& (self->owner = PyThread_get_thread_ident(), 1) )" in _enter_buffered_busy()?

History
Date	User	Action	Args
2010-12-03 13:30:41	vstinner	set	recipients: + vstinner, gregory.p.smith, isandler, amaury.forgeotdarc, pitrou, stutzbach
2010-12-03 13:30:41	vstinner	set	messageid: <[email protected]>
2010-12-03 13:30:38	vstinner	link	issue10478 messages
2010-12-03 13:30:38	vstinner	create