Matthew Poremba has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/53703 )
Change subject: configs,mem-ruby: Remove reference to old GPU ptls
......................................................................
configs,mem-ruby: Remove reference to old GPU ptls
GPU_VIPER_Baseline, GPU_VIPER_Region, and GPU_RfO were removed some time
ago.
Change-Id: If873b0cfe8cc2b3096cbe97d4e13a8e02d2ec567
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/53703
Reviewed-by: Bobby Bruce <bbr...@ucdavis.edu>
Maintainer: Bobby Bruce <bbr...@ucdavis.edu>
Reviewed-by: Matt Sinclair <mattdsincl...@gmail.com>
Maintainer: Matt Sinclair <mattdsincl...@gmail.com>
Tested-by: kokoro <noreply+kok...@google.com>
---
M configs/ruby/AMD_Base_Constructor.py
M src/mem/ruby/protocol/SConsopts
2 files changed, 19 insertions(+), 6 deletions(-)
Approvals:
Matt Sinclair: Looks good to me, approved; Looks good to me, approved
Bobby Bruce: Looks good to me, approved; Looks good to me, approved
kokoro: Regressions pass
diff --git a/configs/ruby/AMD_Base_Constructor.py
b/configs/ruby/AMD_Base_Constructor.py
index 1abc6d7..7f4fb43 100644
--- a/configs/ruby/AMD_Base_Constructor.py
+++ b/configs/ruby/AMD_Base_Constructor.py
@@ -102,9 +102,7 @@
parser.add_argument("--cpu-to-dir-latency", type=int, default=15)
def construct(options, system, ruby_system):
- if (buildEnv['PROTOCOL'] != 'GPU_VIPER' or
- buildEnv['PROTOCOL'] != 'GPU_VIPER_Region' or
- buildEnv['PROTOCOL'] != 'GPU_VIPER_Baseline'):
+ if buildEnv['PROTOCOL'] != 'GPU_VIPER':
panic("This script requires VIPER based protocols \
to be built.")
cpu_sequencers = []
diff --git a/src/mem/ruby/protocol/SConsopts
b/src/mem/ruby/protocol/SConsopts
index ad3bd2f..03b87b4 100644
--- a/src/mem/ruby/protocol/SConsopts
+++ b/src/mem/ruby/protocol/SConsopts
@@ -32,9 +32,6 @@
main.Append(ALL_PROTOCOLS=[
'GPU_VIPER',
- 'GPU_VIPER_Baseline',
- 'GPU_VIPER_Region',
- 'GPU_RfO',
'MOESI_AMD_Base',
'MESI_Two_Level',
'MESI_Three_Level',
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/53703
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: If873b0cfe8cc2b3096cbe97d4e13a8e02d2ec567
Gerrit-Change-Number: 53703
Gerrit-PatchSet: 2
Gerrit-Owner: Matthew Poremba <matthew.pore...@amd.com>
Gerrit-Reviewer: Bobby Bruce <bbr...@ucdavis.edu>
Gerrit-Reviewer: Jason Lowe-Power <ja...@lowepower.com>
Gerrit-Reviewer: Jason Lowe-Power <power...@gmail.com>
Gerrit-Reviewer: Matt Sinclair <mattdsincl...@gmail.com>
Gerrit-Reviewer: Matthew Poremba <matthew.pore...@amd.com>
Gerrit-Reviewer: kokoro <noreply+kok...@google.com>
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