diff options
| author | Nilay Vaish <nilay@cs.wisc.edu> | 2014-03-01 23:35:21 -0600 |
|---|---|---|
| committer | Nilay Vaish <nilay@cs.wisc.edu> | 2014-03-01 23:35:21 -0600 |
| commit | a533f3f9831081625626f96f9434a00f3079e98e (patch) | |
| tree | a83c2cd3c873bc4977be9d06305dfc43083367e8 /src | |
| parent | 53f697a6166a6fe2787882f3448e73a8ebb849aa (diff) | |
| download | gem5-a533f3f9831081625626f96f9434a00f3079e98e.tar.xz | |
ruby: profiler: statically allocate stats variable
Couple of users observed segmentation fault when the simulator tries to
register the statistical variable m_IncompleteTimes. It seems that there
is some problem with the initialization of these variables when allocated
in the constructor.
Diffstat (limited to 'src')
| -rw-r--r-- | src/mem/ruby/profiler/Profiler.cc | 1 | ||||
| -rw-r--r-- | src/mem/ruby/profiler/Profiler.hh | 2 |
2 files changed, 1 insertions, 2 deletions
diff --git a/src/mem/ruby/profiler/Profiler.cc b/src/mem/ruby/profiler/Profiler.cc index 6f7da1eda..f078ef2c1 100644 --- a/src/mem/ruby/profiler/Profiler.cc +++ b/src/mem/ruby/profiler/Profiler.cc @@ -62,7 +62,6 @@ using namespace std; using m5::stl_helpers::operator<<; Profiler::Profiler(const RubySystemParams *p) - : m_IncompleteTimes(MachineType_NUM) { m_hot_lines = p->hot_lines; m_all_instructions = p->all_instructions; diff --git a/src/mem/ruby/profiler/Profiler.hh b/src/mem/ruby/profiler/Profiler.hh index 247c705b0..2a0ff71b2 100644 --- a/src/mem/ruby/profiler/Profiler.hh +++ b/src/mem/ruby/profiler/Profiler.hh @@ -125,7 +125,7 @@ class Profiler std::vector<Stats::Histogram *> m_InitialToForwardDelayHist; std::vector<Stats::Histogram *> m_ForwardToFirstResponseDelayHist; std::vector<Stats::Histogram *> m_FirstResponseToCompletionDelayHist; - std::vector<Stats::Scalar> m_IncompleteTimes; + Stats::Scalar m_IncompleteTimes[MachineType_NUM]; //added by SS bool m_hot_lines; |