Bobby R. Bruce has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/50168 )
3 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the
submitted one.
Change subject: python: Remove 'is_ruby' function
......................................................................
python: Remove 'is_ruby' function
This function is not used and should not be used. It does not really
make any sense.
Change-Id: I79ee7283ddbc282b9b803df33ad266b7a88a9c67
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/50168
Tested-by: kokoro <noreply+kok...@google.com>
Reviewed-by: Jason Lowe-Power <power...@gmail.com>
Maintainer: Bobby R. Bruce <bbr...@ucdavis.edu>
---
M src/python/gem5/coherence_protocol.py
1 file changed, 1 insertion(+), 21 deletions(-)
Approvals:
Jason Lowe-Power: Looks good to me, approved
Bobby R. Bruce: Looks good to me, approved
kokoro: Regressions pass
diff --git a/src/python/gem5/coherence_protocol.py
b/src/python/gem5/coherence_protocol.py
index 06375e3..275a304 100644
--- a/src/python/gem5/coherence_protocol.py
+++ b/src/python/gem5/coherence_protocol.py
@@ -42,24 +42,4 @@
MOESI_AMD_BASE = 8
MI_EXAMPLE = 9
GPU_VIPER = 10
- CHI = 11
-
-
-def is_ruby(protocol: CoherenceProtocol) -> bool:
- """Specifies if a given protocol is Ruby or not
-
- :returns: True if the given protocol is Ruby, otherwise false
- """
- ruby = (
- CoherenceProtocol.MESI_THREE_LEVEL,
- CoherenceProtocol.MESI_THREE_LEVEL_HTM,
- CoherenceProtocol.ARM_MOESI_HAMMER,
- CoherenceProtocol.GARNET_STANDALONE,
- CoherenceProtocol.MESI_TWO_LEVEL,
- CoherenceProtocol.MOESI_CMP_DIRECTORY,
- CoherenceProtocol.MOESI_CMP_TOKEN,
- CoherenceProtocol.MOESI_AMD_BASE,
- CoherenceProtocol.GPU_VIPER,
- )
-
- return protocol in ruby
+ CHI = 11
\ No newline at end of file
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/50168
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: I79ee7283ddbc282b9b803df33ad266b7a88a9c67
Gerrit-Change-Number: 50168
Gerrit-PatchSet: 6
Gerrit-Owner: Bobby R. Bruce <bbr...@ucdavis.edu>
Gerrit-Reviewer: Andreas Sandberg <andreas.sandb...@arm.com>
Gerrit-Reviewer: Bobby R. Bruce <bbr...@ucdavis.edu>
Gerrit-Reviewer: Jason Lowe-Power <ja...@lowepower.com>
Gerrit-Reviewer: Jason Lowe-Power <power...@gmail.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