From a0415f2b24d1c3bcf49775dee302eb0b16f27106 Mon Sep 17 00:00:00 2001 From: Korey Sewell Date: Thu, 5 May 2011 02:20:31 -0400 Subject: 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 --- src/mem/ruby/system/AbstractMemOrCache.hh | 1 - src/mem/ruby/system/MemoryControl.cc | 59 +++++++++++-------------------- src/mem/ruby/system/MemoryControl.hh | 2 -- 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; } @@ -352,12 +345,6 @@ MemoryControl::printConfig(ostream& out) out << " Arbitration randomness: " << m_mem_random_arbitrate << endl; } -void -MemoryControl::setDebug(int debugFlag) -{ - m_debug = debugFlag; -} - void MemoryControl::clearStats() const { @@ -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; }; -- cgit v1.2.3