summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKorey Sewell <ksewell@umich.edu>2011-05-05 02:20:31 -0400
committerKorey Sewell <ksewell@umich.edu>2011-05-05 02:20:31 -0400
commita0415f2b24d1c3bcf49775dee302eb0b16f27106 (patch)
tree17fe10686f7c443bd4e3d0a83d0ae3fad3701dd7
parentfea2c26402894aa26de7b7d8e14fe71070b76296 (diff)
downloadgem5-a0415f2b24d1c3bcf49775dee302eb0b16f27106.tar.xz
ruby: use RubyMemory flag & remove setDebug() functionality
The RubyMemory flag wasnt used in the code, creating large gaps in trace output. Replace cprintfs w/dprintfs using RubyMemory in memory controller. DPRINTF also deprecate the usage of the setDebug() pure virtual function in the AbstractMemoryOrCache Class as well the m_debug/cprintf functions in MemoryControl.hh/cc
-rw-r--r--src/mem/ruby/system/AbstractMemOrCache.hh1
-rw-r--r--src/mem/ruby/system/MemoryControl.cc59
-rw-r--r--src/mem/ruby/system/MemoryControl.hh2
3 files changed, 21 insertions, 41 deletions
diff --git a/src/mem/ruby/system/AbstractMemOrCache.hh b/src/mem/ruby/system/AbstractMemOrCache.hh
index 28b446ef8..88f9f4d87 100644
--- a/src/mem/ruby/system/AbstractMemOrCache.hh
+++ b/src/mem/ruby/system/AbstractMemOrCache.hh
@@ -53,7 +53,6 @@ class AbstractMemOrCache
virtual bool areNSlotsAvailable (int n) = 0;
virtual void printConfig (std::ostream& out) = 0;
virtual void print (std::ostream& out) const = 0;
- virtual void setDebug (int debugFlag) = 0;
};
#endif // __MEM_RUBY_SYSTEM_ABSTRACTMEMORCACHE_HH__
diff --git a/src/mem/ruby/system/MemoryControl.cc b/src/mem/ruby/system/MemoryControl.cc
index 91260a907..eb27c0f78 100644
--- a/src/mem/ruby/system/MemoryControl.cc
+++ b/src/mem/ruby/system/MemoryControl.cc
@@ -176,8 +176,6 @@ MemoryControl::init()
{
m_msg_counter = 0;
- m_debug = 0;
-
assert(m_tFaw <= 62); // must fit in a uint64 shift register
m_total_banks = m_banks_per_rank * m_ranks_per_dimm * m_dimms_per_channel;
@@ -253,15 +251,13 @@ MemoryControl::enqueueMemRef(MemoryNode& memRef)
{
m_msg_counter++;
memRef.m_msg_counter = m_msg_counter;
- Time arrival_time = memRef.m_time;
- uint64 at = arrival_time;
- bool is_mem_read = memRef.m_is_mem_read;
physical_address_t addr = memRef.m_addr;
int bank = getBank(addr);
- if (m_debug) {
- cprintf("New memory request%7d: %#08x %c arrived at %10d bank = %3x\n",
- m_msg_counter, addr, is_mem_read? 'R':'W', at, bank);
- }
+
+ DPRINTF(RubyMemory, "New memory request%7d: %#08x %c arrived at %10d bank = %3x\n",
+ m_msg_counter, addr, memRef.m_is_mem_read ? 'R':'W',
+ memRef.m_time * g_eventQueue_ptr->getClock(),
+ bank);
m_profiler_ptr->profileMemReq(bank);
m_input_queue.push_back(memRef);
@@ -294,12 +290,9 @@ MemoryControl::peekNode()
{
assert(isReady());
MemoryNode req = m_response_queue.front();
- uint64 returnTime = req.m_time;
- if (m_debug) {
- cprintf("Old memory request%7d: %#08x %c peeked at %10d\n",
- req.m_msg_counter, req.m_addr, req.m_is_mem_read ? 'R':'W',
- returnTime);
- }
+ DPRINTF(RubyMemory, "Peek: memory request%7d: %#08x %c\n",
+ req.m_msg_counter, req.m_addr, req.m_is_mem_read ? 'R':'W');
+
return req;
}
@@ -353,12 +346,6 @@ MemoryControl::printConfig(ostream& out)
}
void
-MemoryControl::setDebug(int debugFlag)
-{
- m_debug = debugFlag;
-}
-
-void
MemoryControl::clearStats() const
{
m_profiler_ptr->clearStats();
@@ -379,6 +366,10 @@ MemoryControl::enqueueToDirectory(MemoryNode req, int latency)
req.m_time = arrival_time;
m_response_queue.push_back(req);
+ DPRINTF(RubyMemory, "Enqueueing msg %#08x %c back to directory at %15d\n",
+ req.m_addr, req.m_is_mem_read ? 'R':'W',
+ arrival_time * g_eventQueue_ptr->getClock());
+
// schedule the wake up
g_eventQueue_ptr->scheduleEventAbsolute(m_consumer_ptr, arrival_time);
}
@@ -413,10 +404,8 @@ MemoryControl::queueReady(int bank)
{
if ((m_bankBusyCounter[bank] > 0) && !m_mem_fixed_delay) {
m_profiler_ptr->profileMemBankBusy();
-#if 0
- if (m_debug)
- printf(" bank %x busy %d\n", bank, m_bankBusyCounter[bank]);
-#endif
+
+ DPRINTF(RubyMemory, "bank %x busy %d\n", bank, m_bankBusyCounter[bank]);
return false;
}
@@ -489,12 +478,7 @@ MemoryControl::issueRefresh(int bank)
return false;
// Issue it:
-#if 0
- if (m_debug) {
- uint64 current_time = g_eventQueue_ptr->getTime();
- printf(" Refresh bank %3x at %lld\n", bank, current_time);
- }
-#endif
+ DPRINTF(RubyMemory, "Refresh bank %3x\n", bank);
m_profiler_ptr->profileMemRefresh();
m_need_refresh--;
@@ -528,13 +512,12 @@ MemoryControl::issueRequest(int bank)
int rank = getRank(bank);
MemoryNode req = m_bankQueues[bank].front();
m_bankQueues[bank].pop_front();
- if (m_debug) {
- uint64 current_time = g_eventQueue_ptr->getTime();
- cprintf(" Mem issue request%7d: %#08x %c at %10d "
- "bank=%3x\n",
- req.m_msg_counter, req.m_addr, req.m_is_mem_read? 'R':'W',
- current_time, bank);
- }
+
+ DPRINTF(RubyMemory, "Mem issue request%7d: %#08x %c "
+ "bank=%3x\n", req.m_msg_counter, req.m_addr,
+ req.m_is_mem_read? 'R':'W',
+ bank);
+
if (req.m_msgptr) { // don't enqueue L3 writebacks
enqueueToDirectory(req, m_mem_ctl_latency + m_mem_fixed_delay);
}
diff --git a/src/mem/ruby/system/MemoryControl.hh b/src/mem/ruby/system/MemoryControl.hh
index 0d5e2c38e..2b3cca603 100644
--- a/src/mem/ruby/system/MemoryControl.hh
+++ b/src/mem/ruby/system/MemoryControl.hh
@@ -84,7 +84,6 @@ class MemoryControl :
void printConfig(std::ostream& out);
void print(std::ostream& out) const;
- void setDebug(int debugFlag);
void clearStats() const;
void printStats(std::ostream& out) const;
@@ -161,7 +160,6 @@ class MemoryControl :
int m_refresh_bank; // which bank to refresh next
int m_ageCounter; // age of old requests; to detect starvation
int m_idleCount; // watchdog timer for shutting down
- int m_debug; // turn on printf's
MemCntrlProfiler* m_profiler_ptr;
};