# HG changeset patch # Parent ab0aff639cfb0088e5010e773d873c6370a1b6f6 diff -r ab0aff639cfb Doc/library/multiprocessing.rst --- a/Doc/library/multiprocessing.rst Sun Apr 15 11:14:46 2012 +0000 +++ b/Doc/library/multiprocessing.rst Mon Apr 16 15:40:09 2012 +0100 @@ -897,6 +897,10 @@ If *lock* is specified then it should be a :class:`Lock` or :class:`RLock` object from :mod:`multiprocessing`. + .. versionchanged:: 3.3 + :meth:`wait_for` method added to match + :meth:`threading.Condition.wait_for`. + .. class:: Event() A clone of :class:`threading.Event`. @@ -1281,6 +1285,10 @@ If *lock* is supplied then it should be a proxy for a :class:`threading.Lock` or :class:`threading.RLock` object. + .. versionchanged:: 3.3 + :meth:`wait_for` method added to match + :meth:`threading.Condition.wait_for`. + .. method:: Event() Create a shared :class:`threading.Event` object and return a proxy for it. diff -r ab0aff639cfb Lib/multiprocessing/managers.py --- a/Lib/multiprocessing/managers.py Sun Apr 15 11:14:46 2012 +0000 +++ b/Lib/multiprocessing/managers.py Mon Apr 16 15:40:09 2012 +0100 @@ -48,6 +48,7 @@ from multiprocessing import Process, current_process, active_children, Pool, util, connection from multiprocessing.process import AuthenticationString from multiprocessing.forking import exit, Popen, ForkingPickler +from time import time as _time # # Register some things for pickling @@ -996,6 +997,24 @@ return self._callmethod('notify') def notify_all(self): return self._callmethod('notify_all') + def wait_for(self, predicate, timeout=None): + result = predicate() + if result: + return result + if timeout is not None: + endtime = _time() + timeout + else: + endtime = None + waittime = None + while not result: + if endtime is not None: + waittime = endtime - _time() + if waittime <= 0: + break + self.wait(waittime) + result = predicate() + return result + class EventProxy(BaseProxy): _exposed_ = ('is_set', 'set', 'clear', 'wait') diff -r ab0aff639cfb Lib/multiprocessing/synchronize.py --- a/Lib/multiprocessing/synchronize.py Sun Apr 15 11:14:46 2012 +0000 +++ b/Lib/multiprocessing/synchronize.py Mon Apr 16 15:40:09 2012 +0100 @@ -43,6 +43,7 @@ from multiprocessing.process import current_process from multiprocessing.util import register_after_fork, debug from multiprocessing.forking import assert_spawning, Popen +from time import time as _time # Try to import the mp.synchronize module cleanly, if it fails # raise ImportError for platforms lacking a working sem_open implementation. @@ -240,7 +241,7 @@ try: # wait for notification or timeout - ret = self._wait_semaphore.acquire(True, timeout) + return self._wait_semaphore.acquire(True, timeout) finally: # indicate that this thread has woken self._woken_count.release() @@ -248,7 +249,6 @@ # reacquire lock for i in range(count): self._lock.acquire() - return ret def notify(self): assert self._lock._semlock._is_mine(), 'lock is not owned' @@ -290,6 +290,24 @@ while self._wait_semaphore.acquire(False): pass + def wait_for(self, predicate, timeout=None): + result = predicate() + if result: + return result + if timeout is not None: + endtime = _time() + timeout + else: + endtime = None + waittime = None + while not result: + if endtime is not None: + waittime = endtime - _time() + if waittime <= 0: + break + self.wait(waittime) + result = predicate() + return result + # # Event # diff -r ab0aff639cfb Lib/test/test_multiprocessing.py --- a/Lib/test/test_multiprocessing.py Sun Apr 15 11:14:46 2012 +0000 +++ b/Lib/test/test_multiprocessing.py Mon Apr 16 15:40:09 2012 +0100 @@ -887,6 +887,73 @@ self.assertEqual(res, False) self.assertTimingAlmostEqual(wait.elapsed, TIMEOUT1) + @classmethod + def _test_waitfor_f(cls, cond, state): + with cond: + state.value = 0 + cond.notify() + result = cond.wait_for(lambda : state.value==4) + if not result or state.value != 4: + sys.exit(1) + + @unittest.skipUnless(HAS_SHAREDCTYPES, 'needs sharedctypes') + def test_waitfor(self): + # based on test in test/lock_tests.py + cond = self.Condition() + state = self.Value('i', -1) + + p = self.Process(target=self._test_waitfor_f, args=(cond, state)) + p.daemon = True + p.start() + + with cond: + result = cond.wait_for(lambda : state.value==0) + self.assertTrue(result) + self.assertEqual(state.value, 0) + + for i in range(4): + time.sleep(0.01) + with cond: + state.value += 1 + cond.notify() + + p.join(5) + self.assertFalse(p.is_alive()) + self.assertEqual(p.exitcode, 0) + + @classmethod + def _test_waitfor_timeout_f(cls, cond, state, success): + with cond: + expected = 0.1 + dt = time.time() + result = cond.wait_for(lambda : state.value==4, timeout=expected) + dt = time.time() - dt + # borrow logic in assertTimeout() from test/lock_tests.py + if not result and expected * 0.6 < dt < expected * 10.0: + success.value = True + + @unittest.skipUnless(HAS_SHAREDCTYPES, 'needs sharedctypes') + def test_waitfor_timeout(self): + # based on test in test/lock_tests.py + cond = self.Condition() + state = self.Value('i', 0) + success = self.Value('i', False) + + p = self.Process(target=self._test_waitfor_timeout_f, + args=(cond, state, success)) + p.daemon = True + p.start() + + # Only increment 3 times, so state == 4 is never reached. + for i in range(3): + time.sleep(0.01) + with cond: + state.value += 1 + cond.notify() + + p.join(5) + self.assertTrue(success.value) + class _TestEvent(BaseTestCase):