From: Victor Stinner <victor.stinner@gmail.com>
Date: Thu, 19 Jun 2014 10:59:15 +0000 (+0200)
Subject: Closes #21595: asyncio.BaseSelectorEventLoop._read_from_self() now reads all
X-Git-Tag: v3.4.2rc1~357
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=54c4b8e5c1e0fd11235ab0d5c848e5355293c964;p=python

Closes #21595: asyncio.BaseSelectorEventLoop._read_from_self() now reads all
available bytes from the "self pipe", not only a single byte. This change
reduces the risk of having the pipe full and so getting the innocuous
"BlockingIOError: [Errno 11] Resource temporarily unavailable" message.
---

diff --git a/Lib/asyncio/selector_events.py b/Lib/asyncio/selector_events.py
index 1f8e5c8bf6..854e815138 100644
--- a/Lib/asyncio/selector_events.py
+++ b/Lib/asyncio/selector_events.py
@@ -83,10 +83,15 @@ class BaseSelectorEventLoop(base_events.BaseEventLoop):
         self.add_reader(self._ssock.fileno(), self._read_from_self)
 
     def _read_from_self(self):
-        try:
-            self._ssock.recv(1)
-        except (BlockingIOError, InterruptedError):
-            pass
+        while True:
+            try:
+                data = self._ssock.recv(4096)
+                if not data:
+                    break
+            except InterruptedError:
+                continue
+            except BlockingIOError:
+                break
 
     def _write_to_self(self):
         # This may be called from a different thread, possibly after