ruby: remove unused functionalRead() function.
authorNilay Vaish <nilay@cs.wisc.edu>
Wed, 14 Oct 2015 05:29:39 +0000 (00:29 -0500)
committerNilay Vaish <nilay@cs.wisc.edu>
Wed, 14 Oct 2015 05:29:39 +0000 (00:29 -0500)
Not required since functional reads cannot rely on messages that are inflight.

src/mem/ruby/network/MessageBuffer.cc
src/mem/ruby/network/MessageBuffer.hh
src/mem/ruby/network/simple/SimpleNetwork.cc
src/mem/ruby/network/simple/Switch.cc

index 72c03457ab16339562d8f4cd285407b5cc25141b..3f15869162c33c6ed816a7220f22819c21407ad8 100644 (file)
@@ -350,32 +350,6 @@ MessageBuffer::isReady(Tick current_time) const
         (m_prio_heap.front()->getLastEnqueueTime() <= current_time));
 }
 
-bool
-MessageBuffer::functionalRead(Packet *pkt)
-{
-    // Check the priority heap and read any messages that may
-    // correspond to the address in the packet.
-    for (unsigned int i = 0; i < m_prio_heap.size(); ++i) {
-        Message *msg = m_prio_heap[i].get();
-        if (msg->functionalRead(pkt)) return true;
-    }
-
-    // Read the messages in the stall queue that correspond
-    // to the address in the packet.
-    for (StallMsgMapType::iterator map_iter = m_stall_msg_map.begin();
-         map_iter != m_stall_msg_map.end();
-         ++map_iter) {
-
-        for (std::list<MsgPtr>::iterator it = (map_iter->second).begin();
-            it != (map_iter->second).end(); ++it) {
-
-            Message *msg = (*it).get();
-            if (msg->functionalRead(pkt)) return true;
-        }
-    }
-    return false;
-}
-
 uint32_t
 MessageBuffer::functionalWrite(Packet *pkt)
 {
index 4fdf4978d5ece8789fc80c0e37e3a239cbcbd7ce..5c9e1089b8490793815eb12fa72334513c4d0fa3 100644 (file)
@@ -116,11 +116,6 @@ class MessageBuffer : public SimObject
     void setIncomingLink(int link_id) { m_input_link_id = link_id; }
     void setVnet(int net) { m_vnet_id = net; }
 
-    // Function for figuring out if any of the messages in the buffer can
-    // satisfy the read request for the address in the packet.
-    // Return value, if true, indicates that the request was fulfilled.
-    bool functionalRead(Packet *pkt);
-
     // Function for figuring out if any of the messages in the buffer need
     // to be updated with the data from the packet.
     // Return value indicates the number of messages that were updated.
index 3d76254fc09a62b6e5bd9d27e3c997d1efe23304..25d0b6f4bd16f84162556a8b7a27ff23d5c9245e 100644 (file)
@@ -189,12 +189,6 @@ SimpleNetwork::functionalRead(Packet *pkt)
         }
     }
 
-    for (unsigned int i = 0; i < m_int_link_buffers.size(); ++i) {
-        if (m_int_link_buffers[i]->functionalRead(pkt)) {
-            return true;
-        }
-    }
-
     return false;
 }
 
index 0951ef13846c1b2e2333b5cc8acc7885bda8df14..747884f165024146fa86ca60786ca2aa08304186 100644 (file)
@@ -171,12 +171,6 @@ Switch::print(std::ostream& out) const
 bool
 Switch::functionalRead(Packet *pkt)
 {
-    // Access the buffers in the switch for performing a functional read
-    for (unsigned int i = 0; i < m_port_buffers.size(); ++i) {
-        if (m_port_buffers[i]->functionalRead(pkt)) {
-            return true;
-        }
-    }
     return false;
 }