From a533f3f9831081625626f96f9434a00f3079e98e Mon Sep 17 00:00:00 2001
From: Nilay Vaish <nilay@cs.wisc.edu>
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/mem/ruby')
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;
--
cgit v1.2.3