Gregory P. Smith added the comment:

It sounds like doing this is fine (as Glyph suggests in the email thread) but 
it isn't magically going to solve all EINTR problems, just reduce the number of 
calls that could encounter them.

http://man7.org/linux/man-pages/man7/signal.7.html see the "Interruption of 
system calls and library functions" section.

Note that with this SA_RESTART flag set via siginterrupt you _may_ be making a 
trade off between being able to process python signal handlers during long 
reads or writes vs having to wait until the entire thing has finished.

given that, I'm ambivalent about this patch being worthy.

(re: Explicitly testing this, it is hard. for the broader Python test suite as 
a whole I've been pondering a regrtest mode that'll continually pester all of 
the test processes with signals to try and expose EINTR issues.  Low on my TODO 
ideas list, I haven't written code to do it.)

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue19850>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to