Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/17453

Change subject: tests: Fix race condition in download fixture
......................................................................

tests: Fix race condition in download fixture

Change-Id: Idace0e9e71a484080fc581e232ce217b449085c1
Signed-off-by: Jason Lowe-Power <ja...@lowepower.com>
---
M tests/gem5/fixture.py
1 file changed, 6 insertions(+), 1 deletion(-)



diff --git a/tests/gem5/fixture.py b/tests/gem5/fixture.py
index 9d9319c..c4fd12f 100644
--- a/tests/gem5/fixture.py
+++ b/tests/gem5/fixture.py
@@ -229,9 +229,14 @@
         self.url = self.urlbase + self.path
     def _download(self):
         import urllib
+        import errno
         log.test_log.debug("Downloading " + self.url + " to " + self.path)
         if not os.path.exists(self.program_dir):
-            os.makedirs(self.program_dir)
+            try:
+                os.makedirs(self.program_dir)
+            except OSError as e:
+                if e.errno != errno.EEXIST:
+                    raise
         urllib.urlretrieve(self.url, self.path)

     def _getremotetime(self):

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/17453
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: Idace0e9e71a484080fc581e232ce217b449085c1
Gerrit-Change-Number: 17453
Gerrit-PatchSet: 1
Gerrit-Owner: Jason Lowe-Power <ja...@lowepower.com>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to