ruby: remove several unused variables in Profiler
authorNilay Vaish <nilay@cs.wisc.edu>
Sun, 9 Jun 2013 12:30:00 +0000 (07:30 -0500)
committerNilay Vaish <nilay@cs.wisc.edu>
Sun, 9 Jun 2013 12:30:00 +0000 (07:30 -0500)
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
src/mem/ruby/profiler/Profiler.hh

index c608358f70d871c7444d4742d718d4eb049b1a2a..76c4dba74bf9389dde8c5234d47e61105fcd234b 100644 (file)
@@ -265,18 +265,6 @@ Profiler::printStats(ostream& out, bool short_stats)
         out << endl;
     }
 
-    vector<int64_t> 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()
 {
index ce11f7a7b6f38adca7d46bd272a241c7cd6f2414..23efed67a843b5d30d767b98ed42251d5481a2ee 100644 (file)
@@ -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<int64> m_instructions_executed_at_start;
-    std::vector<int64> 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<Histogram> m_missLatencyHistograms;
     std::vector<Histogram> m_machLatencyHistograms;
     std::vector< std::vector<Histogram> > m_missMachLatencyHistograms;