sim,mem,dev: Merge eventq_impl.hh into eventq.hh.
authorGabe Black <gabeblack@google.com>
Mon, 25 May 2020 12:18:10 +0000 (05:18 -0700)
committerGabe Black <gabeblack@google.com>
Thu, 4 Jun 2020 01:54:19 +0000 (01:54 +0000)
Having some methods (which are supposed to be inline) defined in another
file which is only included sometimes creates a lot of opportunities for
errors. They no longer need to be separate, so merge them together.

Change-Id: I5846e55f53f59b9c2081680a6441659265a765f7
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/29409
Maintainer: Gabe Black <gabeblack@google.com>
Reviewed-by: Gabe Black <gabeblack@google.com>
Tested-by: kokoro <noreply+kokoro@google.com>
13 files changed:
src/dev/intel_8254_timer.hh
src/dev/mc146818.hh
src/mem/packet_queue.hh
src/sim/debug.cc
src/sim/dvfs_handler.cc
src/sim/eventq.cc
src/sim/eventq.hh
src/sim/eventq_impl.hh [deleted file]
src/sim/global_event.hh
src/sim/root.cc
src/sim/sim_events.cc
src/sim/sim_object.hh
src/sim/simulate.cc

index 15e0e61fce54a1cf34cc18231576cdfa3ce8aea7..2279457cbd943a033e00e8faaa4e33f50d259624 100644 (file)
@@ -36,7 +36,7 @@
 #include "base/types.hh"
 #include "base/trace.hh"
 #include "debug/Intel8254Timer.hh"
-#include "sim/eventq_impl.hh"
+#include "sim/eventq.hh"
 #include "sim/serialize.hh"
 
 /** Programmable Interval Timer (Intel 8254) */
index 0ed5636be1c026d7e41b320617e6ad5cc736cb94..038141b2f639da7de69c73654fe256c169d9c40c 100644 (file)
@@ -32,7 +32,7 @@
 #include "base/bitunion.hh"
 #include "base/logging.hh"
 #include "sim/core.hh"
-#include "sim/eventq_impl.hh"
+#include "sim/eventq.hh"
 
 /** Real-Time Clock (MC146818) */
 class MC146818 : public EventManager
index 71670e4a536271f57edf9271e3174a5935d03709..a50c858b9afd38f789dfe77a42b9008ba0f1624a 100644 (file)
@@ -53,7 +53,7 @@
 
 #include "mem/port.hh"
 #include "sim/drain.hh"
-#include "sim/eventq_impl.hh"
+#include "sim/eventq.hh"
 
 /**
  * A packet queue is a class that holds deferred packets and later
index 484d4a6702b2ec40bc6419a25ff2f35ad10e35b9..b2b3d456b1c22917a91f7cf1a9f4e21da1910cfb 100644 (file)
@@ -33,7 +33,7 @@
 
 #include "base/debug.hh"
 #include "cpu/pc_event.hh"
-#include "sim/eventq_impl.hh"
+#include "sim/eventq.hh"
 #include "sim/global_event.hh"
 #include "sim/kernel_workload.hh"
 #include "sim/sim_events.hh"
index 9c452eef437306a9dcd9967f0eb4f51d4b34e65d..9591d8f4196ebb1e16698abf6c761ef083ece5fe 100644 (file)
@@ -45,7 +45,7 @@
 #include "debug/DVFS.hh"
 #include "params/DVFSHandler.hh"
 #include "sim/clock_domain.hh"
-#include "sim/eventq_impl.hh"
+#include "sim/eventq.hh"
 #include "sim/stat_control.hh"
 #include "sim/voltage_domain.hh"
 
index 25131b3f6d3491ace5ece8e3e5b465afe5143e62..bc4864c9b5748979dde1656c4c55ed7656681c3b 100644 (file)
@@ -28,6 +28,8 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
+#include "sim/eventq.hh"
+
 #include <cassert>
 #include <iostream>
 #include <string>
@@ -39,7 +41,6 @@
 #include "cpu/smt.hh"
 #include "debug/Checkpoint.hh"
 #include "sim/core.hh"
-#include "sim/eventq_impl.hh"
 
 using namespace std;
 
index 522b394f7ebcbe0f7f46fb70ca930a617d0a553f..ddba8bd23d3a5535c6ab5e7caa748c0d389f3e68 100644 (file)
@@ -44,6 +44,7 @@
 #include <mutex>
 #include <string>
 
+#include "base/debug.hh"
 #include "base/flags.hh"
 #include "base/types.hh"
 #include "debug/Event.hh"
@@ -750,14 +751,56 @@ class EventQueue
      *
      * @ingroup api_eventq
      */
-    void schedule(Event *event, Tick when, bool global = false);
+    void
+    schedule(Event *event, Tick when, bool global=false)
+    {
+        assert(when >= getCurTick());
+        assert(!event->scheduled());
+        assert(event->initialized());
+
+        event->setWhen(when, this);
+
+        // The check below is to make sure of two things
+        // a. A thread schedules local events on other queues through the
+        //    asyncq.
+        // b. A thread schedules global events on the asyncq, whether or not
+        //    this event belongs to this eventq. This is required to maintain
+        //    a total order amongst the global events. See global_event.{cc,hh}
+        //    for more explanation.
+        if (inParallelMode && (this != curEventQueue() || global)) {
+            asyncInsert(event);
+        } else {
+            insert(event);
+        }
+        event->flags.set(Event::Scheduled);
+        event->acquire();
+
+        if (DTRACE(Event))
+            event->trace("scheduled");
+    }
 
     /**
      * Deschedule the specified event. Should be called only from the owning
      * thread.
      * @ingroup api_eventq
      */
-    void deschedule(Event *event);
+    void
+    deschedule(Event *event)
+    {
+        assert(event->scheduled());
+        assert(event->initialized());
+        assert(!inParallelMode || this == curEventQueue());
+
+        remove(event);
+
+        event->flags.clear(Event::Squashed);
+        event->flags.clear(Event::Scheduled);
+
+        if (DTRACE(Event))
+            event->trace("descheduled");
+
+        event->release();
+    }
 
     /**
      * Reschedule the specified event. Should be called only from the owning
@@ -765,7 +808,28 @@ class EventQueue
      *
      * @ingroup api_eventq
      */
-    void reschedule(Event *event, Tick when, bool always = false);
+    void
+    reschedule(Event *event, Tick when, bool always=false)
+    {
+        assert(when >= getCurTick());
+        assert(always || event->scheduled());
+        assert(event->initialized());
+        assert(!inParallelMode || this == curEventQueue());
+
+        if (event->scheduled()) {
+            remove(event);
+        } else {
+            event->acquire();
+        }
+
+        event->setWhen(when, this);
+        insert(event);
+        event->flags.clear(Event::Squashed);
+        event->flags.set(Event::Scheduled);
+
+        if (DTRACE(Event))
+            event->trace("rescheduled");
+    }
 
     Tick nextTick() const { return head->when(); }
     void setCurTick(Tick newVal) { _curTick = newVal; }
diff --git a/src/sim/eventq_impl.hh b/src/sim/eventq_impl.hh
deleted file mode 100644 (file)
index 0e4f408..0000000
+++ /dev/null
@@ -1,105 +0,0 @@
-/*
- * Copyright (c) 2012 The Regents of The University of Michigan
- * Copyright (c) 2012-2013 Mark D. Hill and David A. Wood
- * Copyright (c) 2013 Advanced Micro Devices, Inc.
- * All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions are
- * met: redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer;
- * redistributions in binary form must reproduce the above copyright
- * notice, this list of conditions and the following disclaimer in the
- * documentation and/or other materials provided with the distribution;
- * neither the name of the copyright holders nor the names of its
- * contributors may be used to endorse or promote products derived from
- * this software without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#ifndef __SIM_EVENTQ_IMPL_HH__
-#define __SIM_EVENTQ_IMPL_HH__
-
-#include "base/debug.hh"
-#include "sim/eventq.hh"
-
-inline void
-EventQueue::schedule(Event *event, Tick when, bool global)
-{
-    assert(when >= getCurTick());
-    assert(!event->scheduled());
-    assert(event->initialized());
-
-    event->setWhen(when, this);
-
-    // The check below is to make sure of two things
-    // a. a thread schedules local events on other queues through the asyncq
-    // b. a thread schedules global events on the asyncq, whether or not
-    //    this event belongs to this eventq. This is required to maintain
-    //    a total order amongst the global events. See global_event.{cc,hh}
-    //    for more explanation.
-    if (inParallelMode && (this != curEventQueue() || global)) {
-        asyncInsert(event);
-    } else {
-        insert(event);
-    }
-    event->flags.set(Event::Scheduled);
-    event->acquire();
-
-    if (DTRACE(Event))
-        event->trace("scheduled");
-}
-
-inline void
-EventQueue::deschedule(Event *event)
-{
-    assert(event->scheduled());
-    assert(event->initialized());
-    assert(!inParallelMode || this == curEventQueue());
-
-    remove(event);
-
-    event->flags.clear(Event::Squashed);
-    event->flags.clear(Event::Scheduled);
-
-    if (DTRACE(Event))
-        event->trace("descheduled");
-
-    event->release();
-}
-
-inline void
-EventQueue::reschedule(Event *event, Tick when, bool always)
-{
-    assert(when >= getCurTick());
-    assert(always || event->scheduled());
-    assert(event->initialized());
-    assert(!inParallelMode || this == curEventQueue());
-
-    if (event->scheduled()) {
-        remove(event);
-    } else {
-        event->acquire();
-    }
-
-    event->setWhen(when, this);
-    insert(event);
-    event->flags.clear(Event::Squashed);
-    event->flags.set(Event::Scheduled);
-
-    if (DTRACE(Event))
-        event->trace("rescheduled");
-}
-
-#endif // __SIM_EVENTQ_IMPL_HH__
index 0dd4c1e6da82a4dc8cbe072987a28dd1c25900af..8e4254b32797de0bfe698c27f8be059d2655abb1 100644 (file)
@@ -34,7 +34,7 @@
 #include <vector>
 
 #include "base/barrier.hh"
-#include "sim/eventq_impl.hh"
+#include "sim/eventq.hh"
 
 /**
  * @file sim/global_event.hh
index 1c396dd5e8f0c197090c25433b9a9093332f5b4f..5a174424e3cb69f3af74735dd1de6d870cf44598 100644 (file)
@@ -31,7 +31,7 @@
 #include "base/trace.hh"
 #include "config/the_isa.hh"
 #include "debug/TimeSync.hh"
-#include "sim/eventq_impl.hh"
+#include "sim/eventq.hh"
 #include "sim/full_system.hh"
 #include "sim/root.hh"
 
index bcfa06297c55bf93e8a33d38b22d9ed61086c973..5888ea06fb600d332818498088b536b9353eda07 100644 (file)
@@ -46,7 +46,7 @@
 
 #include "base/callback.hh"
 #include "base/hostinfo.hh"
-#include "sim/eventq_impl.hh"
+#include "sim/eventq.hh"
 #include "sim/sim_exit.hh"
 #include "sim/stats.hh"
 
index de89fbff2258a10f7c35c2fa5da3af29cd2c9843..27f675c64a2018f5b3dc34f341f113768188f11c 100644 (file)
@@ -53,7 +53,6 @@
 #include "params/SimObject.hh"
 #include "sim/drain.hh"
 #include "sim/eventq.hh"
-#include "sim/eventq_impl.hh"
 #include "sim/port.hh"
 #include "sim/serialize.hh"
 
index 11efd9bec4d811c7f99b8098521537ce721601e7..c0d1055fe22192025c350c35d4d51a9097500df9 100644 (file)
@@ -37,7 +37,7 @@
 #include "base/pollevent.hh"
 #include "base/types.hh"
 #include "sim/async.hh"
-#include "sim/eventq_impl.hh"
+#include "sim/eventq.hh"
 #include "sim/sim_events.hh"
 #include "sim/sim_exit.hh"
 #include "sim/stat_control.hh"