diff options
| author | Victor Stinner <victor.stinner@gmail.com> | 2014-01-21 00:41:00 (GMT) |
|---|---|---|
| committer | Victor Stinner <victor.stinner@gmail.com> | 2014-01-21 00:41:00 (GMT) |
| commit | 665486e0e73f6e1cc0a54d1ac1e977b7cff7b085 (patch) | |
| tree | 4154653a1e07f67ce896221322aec091e623f65d | |
| parent | 9b7a1a1af6de71411102e2b95ee3f654cb0cc700 (diff) | |
| download | cpython-665486e0e73f6e1cc0a54d1ac1e977b7cff7b085.zip cpython-665486e0e73f6e1cc0a54d1ac1e977b7cff7b085.tar.gz cpython-665486e0e73f6e1cc0a54d1ac1e977b7cff7b085.tar.bz2 | |
Issue #20311: select.epoll.poll() now rounds the timeout away from zero,
instead of rounding towards zero. For example, a timeout of one microsecond is
now rounded to one millisecond, instead of being rounded to zero.
| -rw-r--r-- | Lib/test/test_epoll.py | 11 | ||||
| -rw-r--r-- | Misc/NEWS | 4 | ||||
| -rw-r--r-- | Modules/selectmodule.c | 4 |
3 files changed, 18 insertions, 1 deletions
diff --git a/Lib/test/test_epoll.py b/Lib/test/test_epoll.py index 7f9547f..ad545d3 100644 --- a/Lib/test/test_epoll.py +++ b/Lib/test/test_epoll.py @@ -46,6 +46,17 @@ class TestEPoll(unittest.TestCase): self.serverSocket.listen(1) self.connections = [self.serverSocket] + def test_timeout_rounding(self): + # epoll_wait() has a resolution of 1 millisecond, check if the timeout + # is correctly rounded to the upper bound + epoll = select.epoll() + self.addCleanup(epoll.close) + for timeout in (1e-2, 1e-3, 1e-4): + t0 = time.perf_counter() + epoll.poll(timeout) + dt = time.perf_counter() - t0 + self.assertGreaterEqual(dt, timeout) + def tearDown(self): for skt in self.connections: @@ -43,6 +43,10 @@ Core and Builtins Library ------- +- Issue #20311: select.epoll.poll() now rounds the timeout away from zero, + instead of rounding towards zero. For example, a timeout of one microsecond + is now rounded to one millisecond, instead of being rounded to zero. + - Issue #20262: Warnings are raised now when duplicate names are added in the ZIP file or too long ZIP file comment is truncated. diff --git a/Modules/selectmodule.c b/Modules/selectmodule.c index c492224..ab2016a 100644 --- a/Modules/selectmodule.c +++ b/Modules/selectmodule.c @@ -1379,7 +1379,9 @@ pyepoll_poll(pyEpoll_Object *self, PyObject *args, PyObject *kwds) return NULL; } else { - timeout = (int)(dtimeout * 1000.0); + /* epoll_wait() has a resolution of 1 millisecond, round away from zero + to wait *at least* dtimeout seconds. */ + timeout = (int)ceil(dtimeout * 1000.0); } if (maxevents == -1) { |