Merge "Prevent CPU spinning when there are no children"

This commit is contained in:
Jenkins 2016-07-21 04:39:13 +00:00 committed by Gerrit Code Review
commit 45080a5177

View File

@ -951,6 +951,7 @@ def run_wsgi(conf_path, app_section, *args, **kwargs):
loop_timeout = strategy.loop_timeout() loop_timeout = strategy.loop_timeout()
with Timeout(loop_timeout, exception=False): with Timeout(loop_timeout, exception=False):
try:
try: try:
pid, status = green_os.wait() pid, status = green_os.wait()
if os.WIFEXITED(status) or os.WIFSIGNALED(status): if os.WIFEXITED(status) or os.WIFSIGNALED(status):
@ -958,6 +959,14 @@ def run_wsgi(conf_path, app_section, *args, **kwargs):
except OSError as err: except OSError as err:
if err.errno not in (errno.EINTR, errno.ECHILD): if err.errno not in (errno.EINTR, errno.ECHILD):
raise raise
if err.errno == errno.ECHILD:
# If there are no children at all (ECHILD), then
# there's nothing to actually wait on. We sleep
# for a little bit to avoid a tight CPU spin
# and still are able to catch any KeyboardInterrupt
# events that happen. The value of 0.01 matches the
# value in eventlet's waitpid().
sleep(0.01)
except KeyboardInterrupt: except KeyboardInterrupt:
logger.notice('User quit') logger.notice('User quit')
running[0] = False running[0] = False