systemc: Fix the priority of the maximum time event.
authorGabe Black <gabeblack@google.com>
Thu, 16 Aug 2018 02:16:41 +0000 (19:16 -0700)
committerGabe Black <gabeblack@google.com>
Thu, 20 Sep 2018 01:49:31 +0000 (01:49 +0000)
Change-Id: I3e5e71165d06da77076f0e58012c575e24456182
Reviewed-on: https://gem5-review.googlesource.com/12209
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Gabe Black <gabeblack@google.com>

src/systemc/core/scheduler.hh

index 0c755eda47a7ac3332f3c749cf4a26aacfa4a10a..983c53fde1c7b4d75414014709d0c5114997c106 100644 (file)
@@ -129,11 +129,14 @@ typedef NodeList<Channel> ChannelList;
  * MAX RUN TIME
  *
  * When sc_start is called, it's possible to pass in a maximum time the
- * simulation should run to, at which point sc_pause is implicitly called.
- * That's implemented by scheduling an event at the max time with a priority
- * which is lower than all the others so that it happens only if time would
- * advance. When that event triggers, it calls the same function as the pause
- * event.
+ * simulation should run to, at which point sc_pause is implicitly called. The
+ * simulation is supposed to run up to the latest timed notification phase
+ * which is less than or equal to the maximum time. In other words it should
+ * run timed notifications at the maximum time, but not the subsequent evaluate
+ * phase. That's implemented by scheduling an event at the max time with a
+ * priority which is lower than all the others except the ready event. Timed
+ * notifications will happen before it fires, but it will override any ready
+ * event and prevent the evaluate phase from starting.
  */
 
 class Scheduler
@@ -279,9 +282,9 @@ class Scheduler
 
     static Priority StopPriority = DefaultPriority - 1;
     static Priority PausePriority = DefaultPriority + 1;
-    static Priority ReadyPriority = DefaultPriority + 2;
+    static Priority MaxTickPriority = DefaultPriority + 2;
+    static Priority ReadyPriority = DefaultPriority + 3;
     static Priority StarvationPriority = ReadyPriority;
-    static Priority MaxTickPriority = DefaultPriority + 3;
 
     EventQueue *eq;
     std::map<Tick, int> pendingTicks;