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

Change subject: mem-garnet: Enable bridge rescheduling for DVFS
......................................................................

mem-garnet: Enable bridge rescheduling for DVFS

The Bridge SimObject, which hosts the CDC and SerDes functionality,
could be woken up out-of-order when changing frequencies in DVFS.
This change ensures there is no packet drop during the transition
times.

Change-Id: I40240dcb3e957217abd2d7ad22cc4f78809f7b49
---
M src/mem/ruby/network/garnet/NetworkBridge.cc
1 file changed, 5 insertions(+), 2 deletions(-)



diff --git a/src/mem/ruby/network/garnet/NetworkBridge.cc b/src/mem/ruby/network/garnet/NetworkBridge.cc
index ddb6f4e..1ad34d3 100644
--- a/src/mem/ruby/network/garnet/NetworkBridge.cc
+++ b/src/mem/ruby/network/garnet/NetworkBridge.cc
@@ -257,7 +257,10 @@
         t_flit = link_srcQueue->getTopFlit();
         DPRINTF(RubyNetwork, "Recieved flit %s\n", *t_flit);
         flitisizeAndSend(t_flit);
-        return;
     }
-    assert(!link_srcQueue->getSize());
+
+    // Reschedule in case there is a waiting flit.
+    if (!link_srcQueue->isEmpty()) {
+        scheduleEvent(Cycles(1));;
+    }
 }

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/44287
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: I40240dcb3e957217abd2d7ad22cc4f78809f7b49
Gerrit-Change-Number: 44287
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