From d32ee94231251b8d07bb811142f6759f8655962b Mon Sep 17 00:00:00 2001 From: Nilay Vaish Date: Sun, 9 Jun 2013 07:30:00 -0500 Subject: [PATCH] ruby: remove several unused variables in Profiler This patch removes per processor cycle count, histogram for filter stats, histogram for multicasts, histogram for prefetch wait, some function prototypes that do not have definitions. --- src/mem/ruby/profiler/Profiler.cc | 25 ------------------------- src/mem/ruby/profiler/Profiler.hh | 15 --------------- 2 files changed, 40 deletions(-) diff --git a/src/mem/ruby/profiler/Profiler.cc b/src/mem/ruby/profiler/Profiler.cc index c608358f7..76c4dba74 100644 --- a/src/mem/ruby/profiler/Profiler.cc +++ b/src/mem/ruby/profiler/Profiler.cc @@ -265,18 +265,6 @@ Profiler::printStats(ostream& out, bool short_stats) out << endl; } - vector perProcCycleCount(m_num_of_sequencers); - - for (int i = 0; i < m_num_of_sequencers; i++) { - perProcCycleCount[i] = - g_system_ptr->curCycle() - m_cycles_executed_at_start[i] + 1; - // The +1 allows us to avoid division by zero - } - - out << "ruby_cycles_executed: " << perProcCycleCount << endl; - - out << endl; - if (!short_stats) { out << "Busy Controller Counts:" << endl; for (uint32_t i = 0; i < MachineType_NUM; i++) { @@ -403,8 +391,6 @@ Profiler::printStats(ostream& out, bool short_stats) printRequestProfile(out); - out << "filter_action: " << m_filter_action_histogram << endl; - if (!m_all_instructions) { m_address_profiler_ptr->printStats(out); } @@ -447,11 +433,6 @@ Profiler::clearStats() m_ruby_start = g_system_ptr->curCycle(); m_real_time_start_time = time(NULL); - m_cycles_executed_at_start.resize(m_num_of_sequencers); - for (int i = 0; i < m_num_of_sequencers; i++) { - m_cycles_executed_at_start[i] = g_system_ptr->curCycle(); - } - m_busyBankCount = 0; m_missLatencyHistograms.resize(RubyRequestType_NUM); @@ -543,12 +524,6 @@ Profiler::profileSharing(const Address& addr, AccessType type, } } -void -Profiler::profilePFWait(Cycles waitTime) -{ - m_prefetchWaitHistogram.add(waitTime); -} - void Profiler::bankBusy() { diff --git a/src/mem/ruby/profiler/Profiler.hh b/src/mem/ruby/profiler/Profiler.hh index ce11f7a7b..23efed67a 100644 --- a/src/mem/ruby/profiler/Profiler.hh +++ b/src/mem/ruby/profiler/Profiler.hh @@ -108,12 +108,6 @@ class Profiler : public SimObject m_average_latency_estimate.add(latency); } - void recordPrediction(bool wasGood, bool wasPredicted); - - void startTransaction(int cpu); - void endTransaction(int cpu); - void profilePFWait(Cycles waitTime); - void controllerBusy(MachineID machID); void bankBusy(); @@ -156,17 +150,10 @@ class Profiler : public SimObject AddressProfiler* m_address_profiler_ptr; AddressProfiler* m_inst_profiler_ptr; - std::vector m_instructions_executed_at_start; - std::vector m_cycles_executed_at_start; - Cycles m_ruby_start; time_t m_real_time_start_time; int64_t m_busyBankCount; - Histogram m_multicast_retry_histogram; - - Histogram m_filter_action_histogram; - Histogram m_tbeProfile; Histogram m_read_sharing_histogram; Histogram m_write_sharing_histogram; @@ -174,8 +161,6 @@ class Profiler : public SimObject int64 m_cache_to_cache; int64 m_memory_to_cache; - Histogram m_prefetchWaitHistogram; - std::vector m_missLatencyHistograms; std::vector m_machLatencyHistograms; std::vector< std::vector > m_missMachLatencyHistograms; -- 2.30.2