[gem5-dev] Change in gem5/gem5[develop]: tests: Add HeteroSync to nightly regression

2021-09-29 Thread Matt Sinclair (Gerrit) via gem5-dev
Matt Sinclair has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/50951 )


Change subject: tests: Add HeteroSync to nightly regression
..

tests: Add HeteroSync to nightly regression

HeteroSync does a good job of testing the GPU memory system and
atomics support, without requiring a long runtime.  Thus, this
commit adds a mutex and barrier test from HeteroSync to the
nightly regression to ensure these components are tested.

Change-Id: I65998a0a63d41dd3ba165c3a000cee7e42e9034a
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/50951
Maintainer: Matt Sinclair 
Tested-by: kokoro 
Reviewed-by: Bobby R. Bruce 
---
M tests/nightly.sh
1 file changed, 40 insertions(+), 0 deletions(-)

Approvals:
  Bobby R. Bruce: Looks good to me, approved
  Matt Sinclair: Looks good to me, approved
  kokoro: Regressions pass




diff --git a/tests/nightly.sh b/tests/nightly.sh
index 91b19f5..6631bb0 100755
--- a/tests/nightly.sh
+++ b/tests/nightly.sh
@@ -102,3 +102,25 @@
 docker run --rm -u $UID:$GUID --volume "${gem5_root}":"${gem5_root}" -w \
 "${gem5_root}" gcr.io/gem5-test/gcn-gpu:latest build/GCN3_X86/gem5.opt  
\

 configs/example/apu_se.py -n3 -c square
+
+# get HeteroSync
+wget -qN  
http://dist.gem5.org/dist/develop/test-progs/heterosync/gcn3/allSyncPrims-1kernel

+
+# run HeteroSync sleepMutex -- 16 WGs (4 per CU in default config), each  
doing
+# 10 Ld/St per thread and 4 iterations of the critical section is a  
reasonable
+# moderate contention case for the default 4 CU GPU config and help ensure  
GPU

+# atomics are tested.
+docker run --rm -u $UID:$GUID --volume "${gem5_root}":"${gem5_root}" -w \
+"${gem5_root}" gcr.io/gem5-test/gcn-gpu:latest build/GCN3_X86/gem5.opt  
\

+configs/example/apu_se.py -n3 -callSyncPrims-1kernel \
+--options="sleepMutex 10 16 4"
+
+# run HeteroSync LFBarr -- similar setup to sleepMutex above -- 16 WGs
+# accessing unique data and then joining a lock-free barrier, 10 Ld/St per
+# thread, 4 iterations of critical section.  Again this is representative  
of a
+# moderate contention case for the default 4 CU GPU config and help ensure  
GPU

+# atomics are tested.
+docker run --rm -u $UID:$GUID --volume "${gem5_root}":"${gem5_root}" -w \
+"${gem5_root}" gcr.io/gem5-test/gcn-gpu:latest build/GCN3_X86/gem5.opt  
\

+configs/example/apu_se.py -n3 -callSyncPrims-1kernel \
+--options="lfTreeBarrUniq 10 16 4"

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/50951
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: I65998a0a63d41dd3ba165c3a000cee7e42e9034a
Gerrit-Change-Number: 50951
Gerrit-PatchSet: 3
Gerrit-Owner: Matt Sinclair 
Gerrit-Reviewer: Bobby R. Bruce 
Gerrit-Reviewer: Jason Lowe-Power 
Gerrit-Reviewer: Kyle Roarty 
Gerrit-Reviewer: Matt Sinclair 
Gerrit-Reviewer: Matthew Poremba 
Gerrit-Reviewer: kokoro 
Gerrit-MessageType: merged
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

[gem5-dev] Change in gem5/gem5[develop]: tests: Add HeteroSync to nightly regression

2021-09-25 Thread Matt Sinclair (Gerrit) via gem5-dev
Matt Sinclair has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/50951 )



Change subject: tests: Add HeteroSync to nightly regression
..

tests: Add HeteroSync to nightly regression

HeteroSync does a good job of testing the GPU memory system and
atomics support, without requiring a long runtime.  Thus, this
commit adds a mutex and barrier test from HeteroSync to the
nightly regression to ensure these components are tested.

Change-Id: I65998a0a63d41dd3ba165c3a000cee7e42e9034a
---
M tests/nightly.sh
1 file changed, 22 insertions(+), 0 deletions(-)



diff --git a/tests/nightly.sh b/tests/nightly.sh
index 91b19f5..3f115e0 100755
--- a/tests/nightly.sh
+++ b/tests/nightly.sh
@@ -102,3 +102,25 @@
 docker run --rm -u $UID:$GUID --volume "${gem5_root}":"${gem5_root}" -w \
 "${gem5_root}" gcr.io/gem5-test/gcn-gpu:latest build/GCN3_X86/gem5.opt  
\

 configs/example/apu_se.py -n3 -c square
+
+# get HeteroSync
+wget -qN  
http://dist.gem5.org/dist/v21-1/test-progs/heterosync/gcn3/allSyncPrims-1kernel

+
+# run HeteroSync sleepMutex -- 16 WGs (4 per CU in default config), each  
doing
+# 10 Ld/St per thread and 4 iterations of the critical section is a  
reasonable
+# moderate contention case for the default 4 CU GPU config and help ensure  
GPU

+# atomics are tested.
+docker run --rm -u $UID:$GUID --volume "${gem5_root}":"${gem5_root}" -w \
+"${gem5_root}" gcr.io/gem5-test/gcn-gpu:latest build/GCN3_X86/gem5.opt  
\

+configs/example/apu_se.py -n3 -callSyncPrims-1kernel \
+--options="sleepMutex 10 16 4"
+
+# run HeteroSync LFBarr -- similar setup to sleepMutex above -- 16 WGs
+# accessing unique data and then joining a lock-free barrier, 10 Ld/St per
+# thread, 4 iterations of critical section.  Again this is representative  
of a
+# moderate contention case for the default 4 CU GPU config and help ensure  
GPU

+# atomics are tested.
+docker run --rm -u $UID:$GUID --volume "${gem5_root}":"${gem5_root}" -w \
+"${gem5_root}" gcr.io/gem5-test/gcn-gpu:latest build/GCN3_X86/gem5.opt  
\

+configs/example/apu_se.py -n3 -callSyncPrims-1kernel \
+--options="lfTreeBarrUniq 10 16 4"

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/50951
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: I65998a0a63d41dd3ba165c3a000cee7e42e9034a
Gerrit-Change-Number: 50951
Gerrit-PatchSet: 1
Gerrit-Owner: Matt Sinclair 
Gerrit-MessageType: newchange
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s