Srikant Bharadwaj has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/32294 )

Change subject: mem-garnet: Fix scheduling time for CDC
......................................................................

mem-garnet: Fix scheduling time for CDC

CDC does not wakeup a consumer according to its time.
Instead it was waking up the consumer according to CDC's
clock domain which creates bugs when CDCs are at set at the
source. This patch fixes that problem.

Change-Id: Id186a518c76909a4111ca524d3b392048ff3a2be
---
M src/mem/ruby/common/Consumer.hh
M src/mem/ruby/network/garnet2.0/NetworkBridge.cc
2 files changed, 8 insertions(+), 1 deletion(-)



diff --git a/src/mem/ruby/common/Consumer.hh b/src/mem/ruby/common/Consumer.hh
index 3688021..c4332cf 100644
--- a/src/mem/ruby/common/Consumer.hh
+++ b/src/mem/ruby/common/Consumer.hh
@@ -81,6 +81,13 @@
         return sem->name();
     }

+    ClockedObject *
+    getObject()
+    {
+        return em;
+    }
+
+
     void scheduleEventAbsolute(Tick timeAbs);
     void scheduleEvent(Cycles timeDelta);

diff --git a/src/mem/ruby/network/garnet2.0/NetworkBridge.cc b/src/mem/ruby/network/garnet2.0/NetworkBridge.cc
index 7438dd9..3c834d9 100644
--- a/src/mem/ruby/network/garnet2.0/NetworkBridge.cc
+++ b/src/mem/ruby/network/garnet2.0/NetworkBridge.cc
@@ -96,7 +96,7 @@
         totLatency = latency + cdcLatency;
     }

-    t_flit->set_time(clockEdge(totLatency));
+    t_flit->set_time(link_consumer->getObject()->clockEdge(totLatency));
     linkBuffer.insert(t_flit);
     link_consumer->scheduleEvent(totLatency);
 }

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/32294
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: feature-heterogarnet
Gerrit-Change-Id: Id186a518c76909a4111ca524d3b392048ff3a2be
Gerrit-Change-Number: 32294
Gerrit-PatchSet: 1
Gerrit-Owner: Srikant Bharadwaj <srikant.bharad...@amd.com>
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

Reply via email to