Message296895
If I understood corretly, the problem is that the Python C signal handler is not reentrant because it calls Py_AddPendingCall() which uses a lock and a list.
Before, the signal handler queued a new call to checksignals_witharg() (which just calls PyErr_CheckSignals()) for each received signal. Now, we only queue a single call to checksignals_witharg().
To prevent reentrency issues, can't we hardcoded a call to PyErr_CheckSignals() in ceval.c when SIGNAL_PENDING_CALLS() is called?
Py_AddPendingCall() feature is rarely used, it's mostly used for processing signals, no? Calling PyErr_CheckSignals() when no signal was received is cheap, so it shouldn't hurt. |
|
Date |
User |
Action |
Args |
2017-06-26 13:44:40 | vstinner | set | recipients:
+ vstinner, ronaldoussoren, pitrou, ned.deily, njs, neologix, mattbillenstein |
2017-06-26 13:44:40 | vstinner | set | messageid: <1498484680.64.0.430323199675.issue30703@psf.upfronthosting.co.za> |
2017-06-26 13:44:40 | vstinner | link | issue30703 messages |
2017-06-26 13:44:40 | vstinner | create | |
|