derek/gem5
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

systemc: Add some error checks to sc_clock.

Browse Source 
The Accellera version reports an error if the period, high or low
portions of the clock are zero.

Change-Id: I2cbb7c3a1b6abe4d969ef875ebf8754d2d1c489a
Reviewed-on: https://gem5-review.googlesource.com/c/13303
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Gabe Black <gabeblack@google.com>
This commit is contained in:
Gabe Black
2018-10-05 15:55:50 -07:00
parent c701d88756
commit 3287171e08
1 changed files with 22 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
src/systemc/channel/sc_clock.cc
View File

@@ -38,6 +38,7 @@
#include "systemc/ext/channel/sc_clock.hh"
#include "systemc/ext/core/sc_main.hh"
#include "systemc/ext/core/sc_module.hh" // for sc_gen_unique_name
#include "systemc/ext/utils/sc_report_handler.hh"
namespace sc_gem5
{
@@ -108,6 +109,27 @@ sc_clock::sc_clock(const char *name, const sc_time &period,
_period(period), _dutyCycle(duty_cycle), _startTime(start_time),
_posedgeFirst(posedge_first)
{
if (period == SC_ZERO_TIME) {
std::string msg =
"increase the period: clock '" +
std::string(name) + "'";
SC_REPORT_ERROR("(E101) sc_clock period is zero", msg.c_str());
}
if (duty_cycle * period == SC_ZERO_TIME) {
std::string msg =
"increase the period or increase the duty cycle: clock '" +
std::string(name) + "'";
SC_REPORT_ERROR("(E102) sc_clock high time is zero", msg.c_str());
}
if (duty_cycle * period == period) {
std::string msg =
"increase the period or decrease the duty cycle: clock '" +
std::string(name) + "'";
SC_REPORT_ERROR("(E103) sc_clock low time is zero", msg.c_str());
}
_gem5UpEdge = new ::sc_gem5::ClockTick(this, true, period);
_gem5DownEdge = new ::sc_gem5::ClockTick(this, false, period);
}