mirror of https://github.com/python/cpython
Merged revisions 80484 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk ........ r80484 | antoine.pitrou | 2010-04-25 23:40:32 +0200 (dim., 25 avril 2010) | 6 lines Issue #2302: Fix a race condition in SocketServer.BaseServer.shutdown, where the method could block indefinitely if called just before the event loop started running. This also fixes the occasional freezes witnessed in test_httpservers. ........
This commit is contained in:
parent
212067b93c
commit
53d7d06ed3
|
@ -197,7 +197,7 @@ class BaseServer:
|
|||
self.server_address = server_address
|
||||
self.RequestHandlerClass = RequestHandlerClass
|
||||
self.__is_shut_down = threading.Event()
|
||||
self.__serving = False
|
||||
self.__shutdown_request = False
|
||||
|
||||
def server_activate(self):
|
||||
"""Called by constructor to activate the server.
|
||||
|
@ -214,17 +214,19 @@ class BaseServer:
|
|||
self.timeout. If you need to do periodic tasks, do them in
|
||||
another thread.
|
||||
"""
|
||||
self.__serving = True
|
||||
self.__is_shut_down.clear()
|
||||
while self.__serving:
|
||||
# XXX: Consider using another file descriptor or
|
||||
# connecting to the socket to wake this up instead of
|
||||
# polling. Polling reduces our responsiveness to a
|
||||
# shutdown request and wastes cpu at all other times.
|
||||
r, w, e = select.select([self], [], [], poll_interval)
|
||||
if r:
|
||||
self._handle_request_noblock()
|
||||
self.__is_shut_down.set()
|
||||
try:
|
||||
while not self.__shutdown_request:
|
||||
# XXX: Consider using another file descriptor or
|
||||
# connecting to the socket to wake this up instead of
|
||||
# polling. Polling reduces our responsiveness to a
|
||||
# shutdown request and wastes cpu at all other times.
|
||||
r, w, e = select.select([self], [], [], poll_interval)
|
||||
if self in r:
|
||||
self._handle_request_noblock()
|
||||
finally:
|
||||
self.__shutdown_request = False
|
||||
self.__is_shut_down.set()
|
||||
|
||||
def shutdown(self):
|
||||
"""Stops the serve_forever loop.
|
||||
|
@ -233,7 +235,7 @@ class BaseServer:
|
|||
serve_forever() is running in another thread, or it will
|
||||
deadlock.
|
||||
"""
|
||||
self.__serving = False
|
||||
self.__shutdown_request = True
|
||||
self.__is_shut_down.wait()
|
||||
|
||||
# The distinction between handling, getting, processing and
|
||||
|
|
|
@ -243,6 +243,30 @@ class SocketServerTest(unittest.TestCase):
|
|||
# SocketServer.DatagramRequestHandler,
|
||||
# self.dgram_examine)
|
||||
|
||||
def test_shutdown(self):
|
||||
# Issue #2302: shutdown() should always succeed in making an
|
||||
# other thread leave serve_forever().
|
||||
class MyServer(SocketServer.TCPServer):
|
||||
pass
|
||||
|
||||
class MyHandler(SocketServer.StreamRequestHandler):
|
||||
pass
|
||||
|
||||
threads = []
|
||||
for i in range(20):
|
||||
s = MyServer((HOST, 0), MyHandler)
|
||||
t = threading.Thread(
|
||||
name='MyServer serving',
|
||||
target=s.serve_forever,
|
||||
kwargs={'poll_interval':0.01})
|
||||
t.daemon = True # In case this function raises.
|
||||
threads.append((t, s))
|
||||
for t, s in threads:
|
||||
t.start()
|
||||
s.shutdown()
|
||||
for t, s in threads:
|
||||
t.join()
|
||||
|
||||
|
||||
def test_main():
|
||||
if imp.lock_held():
|
||||
|
|
|
@ -33,6 +33,11 @@ Core and Builtins
|
|||
Library
|
||||
-------
|
||||
|
||||
- Issue #2302: Fix a race condition in SocketServer.BaseServer.shutdown,
|
||||
where the method could block indefinitely if called just before the
|
||||
event loop started running. This also fixes the occasional freezes
|
||||
witnessed in test_httpservers.
|
||||
|
||||
- Issue #5103: SSL handshake would ignore the socket timeout and block
|
||||
indefinitely if the other end didn't respond.
|
||||
|
||||
|
|
Loading…
Reference in New Issue