Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/13187

Change subject: systemc: Ensure the reset event is notified before reseting the target.
......................................................................

systemc: Ensure the reset event is notified before reseting the target.

The target may be the process which is currently running. In that case,
the reset method will end and never get to notifying the reset event.
To fix that, we need to notify the reset event first.

Change-Id: If3a9d87edc0999293a68d86d35989ae49eab3c07
---
M src/systemc/core/process.cc
1 file changed, 2 insertions(+), 2 deletions(-)



diff --git a/src/systemc/core/process.cc b/src/systemc/core/process.cc
index 78bbf91..d8ed5ff 100644
--- a/src/systemc/core/process.cc
+++ b/src/systemc/core/process.cc
@@ -183,14 +183,14 @@
         return;


+    _resetEvent.notify();
+
     if (_needsStart) {
         scheduler.runNow(this);
     } else {
         _isUnwinding = true;
         injectException(resetException);
     }
-
-    _resetEvent.notify();
 }

 void

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/13187
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: If3a9d87edc0999293a68d86d35989ae49eab3c07
Gerrit-Change-Number: 13187
Gerrit-PatchSet: 1
Gerrit-Owner: Gabe Black <[email protected]>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to