Gabe Black has submitted this change and it was merged. (
https://gem5-review.googlesource.com/c/public/gem5/+/14915 )
Change subject: systemc: Update signals before initing processes.
......................................................................
systemc: Update signals before initing processes.
We were supposed to do one update phase before adding all processes
which didn't have dont_initialize() called to the run queue. We were
doing that in the opposite order.
Change-Id: I6dd4108040d455dc78029da029a2eb5e7af819cb
Reviewed-on: https://gem5-review.googlesource.com/c/14915
Reviewed-by: Matthias Jung <jun...@eit.uni-kl.de>
Maintainer: Gabe Black <gabebl...@google.com>
---
M src/systemc/core/scheduler.cc
1 file changed, 2 insertions(+), 1 deletion(-)
Approvals:
Matthias Jung: Looks good to me, approved
Gabe Black: Looks good to me, approved
diff --git a/src/systemc/core/scheduler.cc b/src/systemc/core/scheduler.cc
index a2e52f0..465da55 100644
--- a/src/systemc/core/scheduler.cc
+++ b/src/systemc/core/scheduler.cc
@@ -108,6 +108,8 @@
void
Scheduler::initPhase()
{
+ runUpdate();
+
for (Process *p = initList.getNext(); p; p = initList.getNext()) {
p->popListNode();
@@ -121,7 +123,6 @@
}
}
- runUpdate();
runDelta();
for (auto ets: eventsToSchedule)
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/14915
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: I6dd4108040d455dc78029da029a2eb5e7af819cb
Gerrit-Change-Number: 14915
Gerrit-PatchSet: 2
Gerrit-Owner: Gabe Black <gabebl...@google.com>
Gerrit-Reviewer: Andreas Sandberg <andreas.sandb...@arm.com>
Gerrit-Reviewer: Gabe Black <gabebl...@google.com>
Gerrit-Reviewer: Giacomo Travaglini <giacomo.travagl...@arm.com>
Gerrit-Reviewer: Jason Lowe-Power <ja...@lowepower.com>
Gerrit-Reviewer: Matthias Jung <jun...@eit.uni-kl.de>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev
