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 giampaolo.rodola, gvanrossum, neologix, pitrou, vstinner
Date 2014-01-28.23:55:56
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1390953356.29.0.70792679287.issue12187@psf.upfronthosting.co.za>
In-reply-to
Content
The new asyncio module doesn't have this performance issue: it allows to wait asynchronously for the process exit without busy loop.

Right now, there is no high-level API for that, but it is very likely that Python 3.4 final will provide a simple proc.wait() method for that. See #20400 and related Tulip issue:
http://code.google.com/p/tulip/issues/detail?id=115

On Unix, the default implementation sets an handler for SIGCHLD signal which calls waitpid(pid, WNOHANG) on all processes to detect process exit. But it has also a faster implementation which calls waitpid(-1, WNOHANG) only once.

asyncio uses signal.set_wakeup_fd() to wake up its event loop when it gets a signal.

Charles-François wrote:
> Honestly, I think the extra complexity and non-portability isn't worth it.

I agree. And any change may break the backward compatibility, because signal handling is tricky and many detail are platform specific.

asyncio is well designed and solves this issue in a portable way. On Windows, RegisterWaitWithQueue() is used with an overlapped object and a proactor event loop to wait for the process exit.

I leave the issue open until all the new subprocess code is merged into Tulip and Python asyncio.
History
Date User Action Args
2014-01-28 23:55:56vstinnersetrecipients: + vstinner, gvanrossum, pitrou, giampaolo.rodola, neologix
2014-01-28 23:55:56vstinnersetmessageid: <1390953356.29.0.70792679287.issue12187@psf.upfronthosting.co.za>
2014-01-28 23:55:56vstinnerlinkissue12187 messages
2014-01-28 23:55:56vstinnercreate