diff options
author | Jieming Yin <jieming.yin@amd.com> | 2016-12-15 16:59:17 -0500 |
---|---|---|
committer | Jieming Yin <jieming.yin@amd.com> | 2016-12-15 16:59:17 -0500 |
commit | b9c7b8190c27ad161689934fa780859860cfd74f (patch) | |
tree | d619f13f20590cc41fbce7dd15e91758a4c0cf80 /src/mem | |
parent | cc1f5a4d1607a02246244b7a805cdb3b24bb259a (diff) | |
download | gem5-b9c7b8190c27ad161689934fa780859860cfd74f.tar.xz |
ruby: Detect garnet network-level deadlock.
This patch detects garnet network deadlock by monitoring
network interfaces. If a network interface continuously
fails to allocate virtual channels for a message, a
possible deadlock is detected.
Diffstat (limited to 'src/mem')
-rw-r--r-- | src/mem/ruby/network/garnet2.0/GarnetNetwork.py | 4 | ||||
-rw-r--r-- | src/mem/ruby/network/garnet2.0/NetworkInterface.cc | 11 | ||||
-rw-r--r-- | src/mem/ruby/network/garnet2.0/NetworkInterface.hh | 3 |
3 files changed, 17 insertions, 1 deletions
diff --git a/src/mem/ruby/network/garnet2.0/GarnetNetwork.py b/src/mem/ruby/network/garnet2.0/GarnetNetwork.py index 704532782..00213d60f 100644 --- a/src/mem/ruby/network/garnet2.0/GarnetNetwork.py +++ b/src/mem/ruby/network/garnet2.0/GarnetNetwork.py @@ -46,6 +46,8 @@ class GarnetNetwork(RubyNetwork): "0: Weight-based Table, 1: XY, 2: Custom"); enable_fault_model = Param.Bool(False, "enable network fault model"); fault_model = Param.FaultModel(NULL, "network fault model"); + garnet_deadlock_threshold = Param.UInt32(50000, + "network-level deadlock threshold") class GarnetNetworkInterface(ClockedObject): type = 'GarnetNetworkInterface' @@ -57,6 +59,8 @@ class GarnetNetworkInterface(ClockedObject): "virtual channels per virtual network") virt_nets = Param.UInt32(Parent.number_of_virtual_networks, "number of virtual networks") + garnet_deadlock_threshold = Param.UInt32(Parent.garnet_deadlock_threshold, + "network-level deadlock threshold") class GarnetRouter(BasicRouter): type = 'GarnetRouter' diff --git a/src/mem/ruby/network/garnet2.0/NetworkInterface.cc b/src/mem/ruby/network/garnet2.0/NetworkInterface.cc index 6bdaf39af..0ac55d260 100644 --- a/src/mem/ruby/network/garnet2.0/NetworkInterface.cc +++ b/src/mem/ruby/network/garnet2.0/NetworkInterface.cc @@ -50,7 +50,9 @@ using m5::stl_helpers::deletePointers; NetworkInterface::NetworkInterface(const Params *p) : ClockedObject(p), Consumer(this), m_id(p->id), m_virtual_networks(p->virt_nets), m_vc_per_vnet(p->vcs_per_vnet), - m_num_vcs(m_vc_per_vnet * m_virtual_networks) + m_num_vcs(m_vc_per_vnet * m_virtual_networks), + m_deadlock_threshold(p->garnet_deadlock_threshold), + vc_busy_counter(m_virtual_networks, 0) { m_router_id = -1; m_vc_round_robin = 0; @@ -314,9 +316,16 @@ NetworkInterface::calculateVC(int vnet) if (m_out_vc_state[(vnet*m_vc_per_vnet) + delta]->isInState( IDLE_, curCycle())) { + vc_busy_counter[vnet] = 0; return ((vnet*m_vc_per_vnet) + delta); } } + + vc_busy_counter[vnet] += 1; + panic_if(vc_busy_counter[vnet] > m_deadlock_threshold, + "%s: Possible network deadlock in vnet: %d at time: %llu \n", + name(), vnet, curTick()); + return -1; } diff --git a/src/mem/ruby/network/garnet2.0/NetworkInterface.hh b/src/mem/ruby/network/garnet2.0/NetworkInterface.hh index 85e0145af..f1d1fd505 100644 --- a/src/mem/ruby/network/garnet2.0/NetworkInterface.hh +++ b/src/mem/ruby/network/garnet2.0/NetworkInterface.hh @@ -83,6 +83,7 @@ class NetworkInterface : public ClockedObject, public Consumer int m_vc_round_robin; // For round robin scheduling flitBuffer *outFlitQueue; // For modeling link contention flitBuffer *outCreditQueue; + int m_deadlock_threshold; NetworkLink *inNetLink; NetworkLink *outNetLink; @@ -98,6 +99,8 @@ class NetworkInterface : public ClockedObject, public Consumer std::vector<MessageBuffer *> inNode_ptr; // The Message buffers that provides messages to the protocol std::vector<MessageBuffer *> outNode_ptr; + // When a vc stays busy for a long time, it indicates a deadlock + std::vector<int> vc_busy_counter; bool flitisizeMessage(MsgPtr msg_ptr, int vnet); int calculateVC(int vnet); |