From a533f3f9831081625626f96f9434a00f3079e98e Mon Sep 17 00:00:00 2001 From: Nilay Vaish Date: Sat, 1 Mar 2014 23:35:21 -0600 Subject: 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. --- src/mem/ruby/profiler/Profiler.cc | 1 - src/mem/ruby/profiler/Profiler.hh | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) (limited to 'src') 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 m_InitialToForwardDelayHist; std::vector m_ForwardToFirstResponseDelayHist; std::vector m_FirstResponseToCompletionDelayHist; - std::vector m_IncompleteTimes; + Stats::Scalar m_IncompleteTimes[MachineType_NUM]; //added by SS bool m_hot_lines; -- cgit v1.2.3