diff options
author | Gabe Black <gabeblack@google.com> | 2018-08-29 00:18:23 -0700 |
---|---|---|
committer | Gabe Black <gabeblack@google.com> | 2018-10-03 00:15:14 +0000 |
commit | 8fdda3b2d8eb12b3c26cce8f38ce6ea4f525fa69 (patch) | |
tree | 63a6a1a854a929a0aaa12b7db62532ee83a5f85c | |
parent | c599dfdfcd37934ae8756f421953ceeb37661a1f (diff) | |
download | gem5-8fdda3b2d8eb12b3c26cce8f38ce6ea4f525fa69.tar.xz |
systemc: Make Process track whether it's dynamic on its own.
Processes which are created in end_of_elaboration aren't created with
sc_spawn but still need to figure out if they're dynamic. Rather than
duplicate the check in sc_spawn, this change centralizes it in the
Process class itself.
Change-Id: I763d5a0fa89a72fbc82346b6ce2eed852ee72524
Reviewed-on: https://gem5-review.googlesource.com/c/12443
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Gabe Black <gabeblack@google.com>
-rw-r--r-- | src/systemc/core/process.cc | 12 | ||||
-rw-r--r-- | src/systemc/core/process.hh | 2 | ||||
-rw-r--r-- | src/systemc/core/process_types.hh | 11 | ||||
-rw-r--r-- | src/systemc/core/sc_spawn.cc | 8 |
4 files changed, 14 insertions, 19 deletions
diff --git a/src/systemc/core/process.cc b/src/systemc/core/process.cc index 0784ab49f..830b8c729 100644 --- a/src/systemc/core/process.cc +++ b/src/systemc/core/process.cc @@ -390,13 +390,15 @@ Process::lastReport(::sc_core::sc_report *report) ::sc_core::sc_report *Process::lastReport() const { return _lastReport.get(); } -Process::Process(const char *name, ProcessFuncWrapper *func, bool _dynamic) : +Process::Process(const char *name, ProcessFuncWrapper *func) : ::sc_core::sc_process_b(name), excWrapper(nullptr), func(func), - _needsStart(true), _dynamic(_dynamic), _isUnwinding(false), - _terminated(false), _suspended(false), _disabled(false), - _syncReset(false), refCount(0), stackSize(::Fiber::DefaultStackSize), - dynamicSensitivity(nullptr) + _needsStart(true), _isUnwinding(false), _terminated(false), + _suspended(false), _disabled(false), _syncReset(false), refCount(0), + stackSize(::Fiber::DefaultStackSize), dynamicSensitivity(nullptr) { + _dynamic = + (::sc_core::sc_get_status() > + ::sc_core::SC_BEFORE_END_OF_ELABORATION); _newest = this; } diff --git a/src/systemc/core/process.hh b/src/systemc/core/process.hh index 0cb3e3c01..0d852afdb 100644 --- a/src/systemc/core/process.hh +++ b/src/systemc/core/process.hh @@ -327,7 +327,7 @@ class Process : public ::sc_core::sc_process_b, public ListNode ::sc_core::sc_report *lastReport() const; protected: - Process(const char *name, ProcessFuncWrapper *func, bool _dynamic); + Process(const char *name, ProcessFuncWrapper *func); static Process *_newest; diff --git a/src/systemc/core/process_types.hh b/src/systemc/core/process_types.hh index 2dde4672a..5fbab8038 100644 --- a/src/systemc/core/process_types.hh +++ b/src/systemc/core/process_types.hh @@ -39,9 +39,7 @@ namespace sc_gem5 class Method : public Process { public: - Method(const char *name, ProcessFuncWrapper *func, bool _dynamic=false) : - Process(name, func, _dynamic) - {} + Method(const char *name, ProcessFuncWrapper *func) : Process(name, func) {} const char *kind() const override { return "sc_method_process"; } @@ -55,8 +53,8 @@ class Method : public Process class Thread : public Process { public: - Thread(const char *name, ProcessFuncWrapper *func, bool _dynamic=false) : - Process(name, func, _dynamic), ctx(nullptr) + Thread(const char *name, ProcessFuncWrapper *func) : + Process(name, func), ctx(nullptr) {} ~Thread() { delete ctx; } @@ -105,8 +103,7 @@ class Thread : public Process class CThread : public Thread { public: - CThread(const char *name, ProcessFuncWrapper *func, bool _dynamic=false) : - Thread(name, func, _dynamic) + CThread(const char *name, ProcessFuncWrapper *func) : Thread(name, func) { // We'll be in the initialization list now, but we shouldn't be. popListNode(); diff --git a/src/systemc/core/sc_spawn.cc b/src/systemc/core/sc_spawn.cc index 054feb8da..b0570a663 100644 --- a/src/systemc/core/sc_spawn.cc +++ b/src/systemc/core/sc_spawn.cc @@ -60,15 +60,11 @@ spawnWork(ProcessFuncWrapper *func, const char *name, name = ::sc_core::sc_gen_unique_name("thread_p"); } - bool dynamic = - (::sc_core::sc_get_status() > - ::sc_core::SC_BEFORE_END_OF_ELABORATION); - Process *proc; if (method) - proc = new Method(name, func, dynamic); + proc = new Method(name, func); else - proc = new Thread(name, func, dynamic); + proc = new Thread(name, func); if (opts) { for (auto e: opts->_events) |