blob: e0c23dec66ae2d6096ad13d843b4965049171bf5 [file] [log] [blame]
Tim Petersaa222232001-05-22 09:34:27 +00001# This is a variant of the very old (early 90's) file
2# Demo/threads/bug.py. It simply provokes a number of threads into
3# trying to import the same module "at the same time".
4# There are no pleasant failure modes -- most likely is that Python
5# complains several times about module random having no attribute
6# randrange, and then Python hangs.
7
Antoine Pitrou0723d2c2010-08-22 20:43:26 +00008import os
Antoine Pitrou1f9dea02010-07-14 11:52:38 +00009import imp
10import sys
Antoine Pitrou7224d072010-08-22 10:18:36 +000011import time
Antoine Pitrou0723d2c2010-08-22 20:43:26 +000012import shutil
Georg Brandl89fad142010-03-14 10:23:39 +000013import unittest
Antoine Pitrou075050f2011-07-15 23:09:13 +020014from test.support import (
Antoine Pitrou314a16b2012-05-17 21:02:54 +020015 verbose, import_module, run_unittest, TESTFN, reap_threads, forget, unlink)
Antoine Pitrou0723d2c2010-08-22 20:43:26 +000016threading = import_module('threading')
Tim Petersaa222232001-05-22 09:34:27 +000017
Antoine Pitrou1f9dea02010-07-14 11:52:38 +000018def task(N, done, done_tasks, errors):
19 try:
Antoine Pitrou7224d072010-08-22 10:18:36 +000020 # We don't use modulefinder but still import it in order to stress
21 # importing of different modules from several threads.
22 if len(done_tasks) % 2:
23 import modulefinder
24 import random
25 else:
26 import random
27 import modulefinder
Antoine Pitrou1f9dea02010-07-14 11:52:38 +000028 # This will fail if random is not completely initialized
29 x = random.randrange(1, 3)
30 except Exception as e:
31 errors.append(e.with_traceback(None))
32 finally:
Victor Stinner2a129742011-05-30 23:02:52 +020033 done_tasks.append(threading.get_ident())
Antoine Pitrou1f9dea02010-07-14 11:52:38 +000034 finished = len(done_tasks) == N
35 if finished:
Antoine Pitroua62cbf72010-10-13 23:48:39 +000036 done.set()
Tim Petersaa222232001-05-22 09:34:27 +000037
Antoine Pitrou0723d2c2010-08-22 20:43:26 +000038# Create a circular import structure: A -> C -> B -> D -> A
39# NOTE: `time` is already loaded and therefore doesn't threaten to deadlock.
40
41circular_imports_modules = {
42 'A': """if 1:
43 import time
44 time.sleep(%(delay)s)
45 x = 'a'
46 import C
47 """,
48 'B': """if 1:
49 import time
50 time.sleep(%(delay)s)
51 x = 'b'
52 import D
53 """,
54 'C': """import B""",
55 'D': """import A""",
56}
Antoine Pitrou1f9dea02010-07-14 11:52:38 +000057
Antoine Pitrou7224d072010-08-22 10:18:36 +000058class Finder:
59 """A dummy finder to detect concurrent access to its find_module()
60 method."""
61
62 def __init__(self):
63 self.numcalls = 0
64 self.x = 0
Antoine Pitrou075050f2011-07-15 23:09:13 +020065 self.lock = threading.Lock()
Antoine Pitrou7224d072010-08-22 10:18:36 +000066
67 def find_module(self, name, path=None):
68 # Simulate some thread-unsafe behaviour. If calls to find_module()
69 # are properly serialized, `x` will end up the same as `numcalls`.
70 # Otherwise not.
Antoine Pitrou202b6062012-12-18 22:18:17 +010071 assert imp.lock_held()
Antoine Pitrou7224d072010-08-22 10:18:36 +000072 with self.lock:
73 self.numcalls += 1
74 x = self.x
Antoine Pitroue0b1c232012-12-18 23:03:42 +010075 time.sleep(0.01)
Antoine Pitrou7224d072010-08-22 10:18:36 +000076 self.x = x + 1
77
78class FlushingFinder:
79 """A dummy finder which flushes sys.path_importer_cache when it gets
80 called."""
81
82 def find_module(self, name, path=None):
83 sys.path_importer_cache.clear()
84
85
Antoine Pitrou1f9dea02010-07-14 11:52:38 +000086class ThreadedImportTests(unittest.TestCase):
87
Antoine Pitrou448acd02010-07-16 19:10:38 +000088 def setUp(self):
89 self.old_random = sys.modules.pop('random', None)
90
91 def tearDown(self):
92 # If the `random` module was already initialized, we restore the
93 # old module at the end so that pickling tests don't fail.
94 # See http://bugs.python.org/issue3657#msg110461
95 if self.old_random is not None:
96 sys.modules['random'] = self.old_random
97
Antoine Pitrou7224d072010-08-22 10:18:36 +000098 def check_parallel_module_init(self):
Antoine Pitrou1f9dea02010-07-14 11:52:38 +000099 if imp.lock_held():
100 # This triggers on, e.g., from test import autotest.
101 raise unittest.SkipTest("can't run when import lock is held")
102
Antoine Pitroua62cbf72010-10-13 23:48:39 +0000103 done = threading.Event()
Antoine Pitrou1f9dea02010-07-14 11:52:38 +0000104 for N in (20, 50) * 3:
105 if verbose:
106 print("Trying", N, "threads ...", end=' ')
Antoine Pitrou7224d072010-08-22 10:18:36 +0000107 # Make sure that random and modulefinder get reimported freshly
108 for modname in ['random', 'modulefinder']:
109 try:
110 del sys.modules[modname]
111 except KeyError:
112 pass
Antoine Pitrou1f9dea02010-07-14 11:52:38 +0000113 errors = []
114 done_tasks = []
Antoine Pitroua62cbf72010-10-13 23:48:39 +0000115 done.clear()
Antoine Pitrou1f9dea02010-07-14 11:52:38 +0000116 for i in range(N):
Antoine Pitrou075050f2011-07-15 23:09:13 +0200117 t = threading.Thread(target=task,
118 args=(N, done, done_tasks, errors,))
119 t.start()
Antoine Pitroue0b1c232012-12-18 23:03:42 +0100120 self.assertTrue(done.wait(60))
Antoine Pitrou1f9dea02010-07-14 11:52:38 +0000121 self.assertFalse(errors)
122 if verbose:
123 print("OK.")
Tim Petersaa222232001-05-22 09:34:27 +0000124
Antoine Pitrou7224d072010-08-22 10:18:36 +0000125 def test_parallel_module_init(self):
126 self.check_parallel_module_init()
127
128 def test_parallel_meta_path(self):
129 finder = Finder()
Brett Cannon3c6ea1c2012-04-27 13:52:55 -0400130 sys.meta_path.insert(0, finder)
Antoine Pitrou7224d072010-08-22 10:18:36 +0000131 try:
132 self.check_parallel_module_init()
133 self.assertGreater(finder.numcalls, 0)
134 self.assertEqual(finder.x, finder.numcalls)
135 finally:
136 sys.meta_path.remove(finder)
137
138 def test_parallel_path_hooks(self):
139 # Here the Finder instance is only used to check concurrent calls
140 # to path_hook().
141 finder = Finder()
142 # In order for our path hook to be called at each import, we need
143 # to flush the path_importer_cache, which we do by registering a
144 # dedicated meta_path entry.
145 flushing_finder = FlushingFinder()
146 def path_hook(path):
147 finder.find_module('')
148 raise ImportError
Brett Cannon8923a4d2012-04-24 22:03:46 -0400149 sys.path_hooks.insert(0, path_hook)
Antoine Pitrou7224d072010-08-22 10:18:36 +0000150 sys.meta_path.append(flushing_finder)
151 try:
152 # Flush the cache a first time
153 flushing_finder.find_module('')
154 numtests = self.check_parallel_module_init()
155 self.assertGreater(finder.numcalls, 0)
156 self.assertEqual(finder.x, finder.numcalls)
157 finally:
158 sys.meta_path.remove(flushing_finder)
159 sys.path_hooks.remove(path_hook)
160
Antoine Pitrou1f9dea02010-07-14 11:52:38 +0000161 def test_import_hangers(self):
Thomas Wouters477c8d52006-05-27 19:21:47 +0000162 # In case this test is run again, make sure the helper module
163 # gets loaded from scratch again.
Antoine Pitrou1f9dea02010-07-14 11:52:38 +0000164 try:
165 del sys.modules['test.threaded_import_hangers']
166 except KeyError:
167 pass
168 import test.threaded_import_hangers
169 self.assertFalse(test.threaded_import_hangers.errors)
Thomas Wouters477c8d52006-05-27 19:21:47 +0000170
Antoine Pitrou0723d2c2010-08-22 20:43:26 +0000171 def test_circular_imports(self):
172 # The goal of this test is to exercise implementations of the import
173 # lock which use a per-module lock, rather than a global lock.
174 # In these implementations, there is a possible deadlock with
175 # circular imports, for example:
176 # - thread 1 imports A (grabbing the lock for A) which imports B
177 # - thread 2 imports B (grabbing the lock for B) which imports A
178 # Such implementations should be able to detect such situations and
179 # resolve them one way or the other, without freezing.
180 # NOTE: our test constructs a slightly less trivial import cycle,
181 # in order to better stress the deadlock avoidance mechanism.
182 delay = 0.5
183 os.mkdir(TESTFN)
184 self.addCleanup(shutil.rmtree, TESTFN)
185 sys.path.insert(0, TESTFN)
186 self.addCleanup(sys.path.remove, TESTFN)
187 for name, contents in circular_imports_modules.items():
188 contents = contents % {'delay': delay}
189 with open(os.path.join(TESTFN, name + ".py"), "wb") as f:
190 f.write(contents.encode('utf-8'))
Antoine Pitrouea3eb882012-05-17 18:55:59 +0200191 self.addCleanup(forget, name)
Antoine Pitrou0723d2c2010-08-22 20:43:26 +0000192
193 results = []
194 def import_ab():
195 import A
196 results.append(getattr(A, 'x', None))
197 def import_ba():
198 import B
199 results.append(getattr(B, 'x', None))
200 t1 = threading.Thread(target=import_ab)
201 t2 = threading.Thread(target=import_ba)
202 t1.start()
203 t2.start()
204 t1.join()
205 t2.join()
206 self.assertEqual(set(results), {'a', 'b'})
207
Antoine Pitrouea3eb882012-05-17 18:55:59 +0200208 def test_side_effect_import(self):
209 code = """if 1:
210 import threading
211 def target():
212 import random
213 t = threading.Thread(target=target)
214 t.start()
215 t.join()"""
216 sys.path.insert(0, os.curdir)
217 self.addCleanup(sys.path.remove, os.curdir)
Antoine Pitrou314a16b2012-05-17 21:02:54 +0200218 filename = TESTFN + ".py"
219 with open(filename, "wb") as f:
Antoine Pitrouea3eb882012-05-17 18:55:59 +0200220 f.write(code.encode('utf-8'))
Antoine Pitrou314a16b2012-05-17 21:02:54 +0200221 self.addCleanup(unlink, filename)
Antoine Pitrouea3eb882012-05-17 18:55:59 +0200222 self.addCleanup(forget, TESTFN)
223 __import__(TESTFN)
224
Tim Petersaa222232001-05-22 09:34:27 +0000225
Antoine Pitrou075050f2011-07-15 23:09:13 +0200226@reap_threads
Antoine Pitrou1f9dea02010-07-14 11:52:38 +0000227def test_main():
Antoine Pitrou4f0338c2012-08-28 00:24:52 +0200228 old_switchinterval = None
229 try:
230 old_switchinterval = sys.getswitchinterval()
Stefan Krah219c7b92012-10-01 23:21:45 +0200231 sys.setswitchinterval(1e-5)
Antoine Pitrou4f0338c2012-08-28 00:24:52 +0200232 except AttributeError:
233 pass
234 try:
235 run_unittest(ThreadedImportTests)
236 finally:
237 if old_switchinterval is not None:
238 sys.setswitchinterval(old_switchinterval)
Tim Peters69232342001-08-30 05:16:13 +0000239
Tim Petersd9742212001-05-22 18:28:25 +0000240if __name__ == "__main__":
241 test_main()