[gem5-dev] Change in gem5/gem5[develop]: test: Updating simple_traffic_run and adding tests

[Mahyar Samani (Gerrit) via gem5-dev]

Mahyar Samani has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/53164 )

Change subject: test: Updating simple_traffic_run and adding tests
......................................................................

test: Updating simple_traffic_run and adding tests

This change updates simple_traffic_run.py with adding clock_domain
for GUPSGen based generators. Also the tests that resulted in
error before are now fixed and added.

Change-Id: I992898a3aef509afd9dec1f9b5d14c5fd67cff37
---
M tests/gem5/traffic_gen/test_memory_traffic_gen.py
M tests/gem5/configs/simple_traffic_run.py
2 files changed, 28 insertions(+), 5 deletions(-)



diff --git a/tests/gem5/configs/simple_traffic_run.py  
b/tests/gem5/configs/simple_traffic_run.py
index 0836580..5b86ae6 100644
--- a/tests/gem5/configs/simple_traffic_run.py
+++ b/tests/gem5/configs/simple_traffic_run.py
@@ -36,9 +36,9 @@
 import importlib

 from os.path import join
-from m5.objects import Root
 from m5.stats import gem5stats
 from gem5.components.boards.test_board import TestBoard
+from m5.objects import Root, SrcClockDomain, VoltageDomain


 def generator_factory(generator_class, generator_cores, mem_size):
@@ -70,13 +70,21 @@
         from gem5.components.processors.gups_generator import GUPSGenerator

         table_size = f"{int(mem_size / 2)}B"
-        return GUPSGenerator(0, table_size, update_limit=1000)
+        clk_domain = SrcClockDomain(
+            clock="2GHz", voltage_domain=VoltageDomain()
+        )
+        return GUPSGenerator(
+            0, table_size, update_limit=1000, clock_domain=clk_domain
+        )
     elif generator_class == "GUPSGeneratorEP":
         from gem5.components.processors.gups_generator_ep import (
             GUPSGeneratorEP,
         )

         table_size = f"{int(mem_size / 2)}B"
+        clk_domain = SrcClockDomain(
+            clock="2GHz", voltage_domain=VoltageDomain()
+        )
         return GUPSGeneratorEP(
             generator_cores, 0, table_size, update_limit=1000
         )
@@ -86,12 +94,16 @@
         )

         table_size = f"{int(mem_size / 2)}B"
+        clk_domain = SrcClockDomain(
+            clock="2GHz", voltage_domain=VoltageDomain()
+        )
         return GUPSGeneratorPAR(
             generator_cores, 0, table_size, update_limit=1000
         )
     else:
         raise ValueError(f"Unknown generator class {generator_class}")

+
 def cache_factory(cache_class):
     if cache_class == "NoCache":
         from gem5.components.cachehierarchies\
diff --git a/tests/gem5/traffic_gen/test_memory_traffic_gen.py  
b/tests/gem5/traffic_gen/test_memory_traffic_gen.py
index f6824c8..0ff24d2 100644
--- a/tests/gem5/traffic_gen/test_memory_traffic_gen.py
+++ b/tests/gem5/traffic_gen/test_memory_traffic_gen.py
@@ -106,9 +106,7 @@


 def create_single_core_tests(module, memory_classes):
-    # TODO: Add GUPSGenerator to these tests after adding ClockDomain as
-    # an input parameter.
-    generator_classes = ["LinearGenerator", "RandomGenerator"]
+    generator_classes =  
["LinearGenerator", "RandomGenerator", "GUPSGenerator"]
     for generator_class in generator_classes:
         for cache_class in cache_classes:
             for memory_class in memory_classes:

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/53164
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: I992898a3aef509afd9dec1f9b5d14c5fd67cff37
Gerrit-Change-Number: 53164
Gerrit-PatchSet: 1
Gerrit-Owner: Mahyar Samani <msam...@ucdavis.edu>
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