From 1c0600998681295735a18690fae184b0c9a4ca51 Mon Sep 17 00:00:00 2001 From: Andrew Svetlov Date: Mon, 26 Aug 2019 12:51:08 +0300 Subject: [PATCH] bpo-34679: Restore instantiation Windows IOCP event loop from non-main thread (#15492) * Restore running proactor event loop from non-main thread Co-Authored-By: Kyle Stanley --- Lib/asyncio/proactor_events.py | 5 ++++- Lib/test/test_asyncio/test_windows_events.py | 19 +++++++++++++++++++ .../2019-08-25-18-07-48.bpo-34679.HECzL7.rst | 1 + 3 files changed, 24 insertions(+), 1 deletion(-) create mode 100644 Misc/NEWS.d/next/Library/2019-08-25-18-07-48.bpo-34679.HECzL7.rst diff --git a/Lib/asyncio/proactor_events.py b/Lib/asyncio/proactor_events.py index 9b8ae064a89..229f56e6bb9 100644 --- a/Lib/asyncio/proactor_events.py +++ b/Lib/asyncio/proactor_events.py @@ -11,6 +11,7 @@ import os import socket import warnings import signal +import threading import collections from . import base_events @@ -627,7 +628,9 @@ class BaseProactorEventLoop(base_events.BaseEventLoop): proactor.set_loop(self) self._make_self_pipe() self_no = self._csock.fileno() - signal.set_wakeup_fd(self_no) + if threading.current_thread() is threading.main_thread(): + # wakeup fd can only be installed to a file descriptor from the main thread + signal.set_wakeup_fd(self_no) def _make_socket_transport(self, sock, protocol, waiter=None, extra=None, server=None): diff --git a/Lib/test/test_asyncio/test_windows_events.py b/Lib/test/test_asyncio/test_windows_events.py index 64543268b1e..d0ba19391fa 100644 --- a/Lib/test/test_asyncio/test_windows_events.py +++ b/Lib/test/test_asyncio/test_windows_events.py @@ -59,6 +59,25 @@ class ProactorLoopCtrlC(test_utils.TestCase): thread.join() +class ProactorMultithreading(test_utils.TestCase): + def test_run_from_nonmain_thread(self): + finished = False + + async def coro(): + await asyncio.sleep(0) + + def func(): + nonlocal finished + loop = asyncio.new_event_loop() + loop.run_until_complete(coro()) + finished = True + + thread = threading.Thread(target=func) + thread.start() + thread.join() + self.assertTrue(finished) + + class ProactorTests(test_utils.TestCase): def setUp(self): diff --git a/Misc/NEWS.d/next/Library/2019-08-25-18-07-48.bpo-34679.HECzL7.rst b/Misc/NEWS.d/next/Library/2019-08-25-18-07-48.bpo-34679.HECzL7.rst new file mode 100644 index 00000000000..785b06b6482 --- /dev/null +++ b/Misc/NEWS.d/next/Library/2019-08-25-18-07-48.bpo-34679.HECzL7.rst @@ -0,0 +1 @@ +Restores instantiation of Windows IOCP event loops from the non-main thread.