Matt Sinclair has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/67198?usp=email )
Change subject: tests: cleanup m5out directly in weekly
......................................................................
tests: cleanup m5out directly in weekly
The weekly test script was implicitly assuming that no m5out
directory existed in the folder where the script was run.
However, if a prior test ran and failed, it would not clean up
its m5out directory, causing the weekly tests to fail.
This commit resolves this by removing the m5out directory before
trying to run any tests in the weekly script. Moreover, we also
update the weekly script to explicitly remove this m5out directory
at the end of the script.
Change-Id: If10c59034528e171cc2c5dacb928b3a81d6b8c50
---
M tests/weekly.sh
1 file changed, 27 insertions(+), 1 deletion(-)
diff --git a/tests/weekly.sh b/tests/weekly.sh
index c7f834b..f218729 100755
--- a/tests/weekly.sh
+++ b/tests/weekly.sh
@@ -70,10 +70,11 @@
# GPU weekly tests start here
# before pulling gem5 resources, make sure it doesn't exist already
+# likewise, remove any lingering m5out folder
docker run --rm --volume "${gem5_root}":"${gem5_root}" -w \
"${gem5_root}" --memory="${docker_mem_limit}" \
gcr.io/gem5-test/gcn-gpu:${tag} bash -c \
- "rm -rf ${gem5_root}/gem5-resources"
+ "rm -rf ${gem5_root}/gem5-resources ${gem5_root}/m5out"
# delete Pannotia datasets and output files in case a failed regression
run left
# them around
rm -f coAuthorsDBLP.graph 1k_128k.gr result.out
@@ -383,5 +384,11 @@
"${gem5_root}" --memory="${docker_mem_limit}" hacc-test-weekly bash
-c \
"rm -rf ${gem5_root}/gem5-resources"
+# Delete the gem5 m5out folder we created -- need to do in docker because
it
+# creates
+docker run --rm --volume "${gem5_root}":"${gem5_root}" -w \
+ "${gem5_root}" --memory="${docker_mem_limit}" hacc-test-weekly bash
-c \
+ "rm -rf ${gem5_root}/m5out"
+
# delete Pannotia datasets we downloaded and output files it created
rm -f coAuthorsDBLP.graph 1k_128k.gr result.out
--
To view, visit
https://gem5-review.googlesource.com/c/public/gem5/+/67198?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: If10c59034528e171cc2c5dacb928b3a81d6b8c50
Gerrit-Change-Number: 67198
Gerrit-PatchSet: 1
Gerrit-Owner: Matt Sinclair <mattdsinclair.w...@gmail.com>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
