Changeset 931

Timestamp:

Jan 27, 2015, 5:58:07 PM (10 years ago)

Author:

cfuguet

Message:

reconf: introducing the segment recovery mechanism in the dspin_router

• The p_recovery_cfg port, connected through a signal to a config register in the XICU, is formatted in two fields: the 4 LSb contain the blackhole relative position and the 4 bits after contain the recovery direction.

• A request with the blackhole cluster as destination is rerouted through the defined recovery direction.

• Remove the reconfiguration parameter of the constructor. The p_recovery_cfg port is instantiated when the bind_recovery_port is called. This port is also connected during this function to the signal that is passed as an argument.

Location:

branches/reconfiguration

Files:

• modules/dspin_router/caba/source/include/dspin_router.h (modified) (3 diffs)
• modules/dspin_router/caba/source/src/dspin_router.cpp (modified) (5 diffs)
• platforms/tsar_generic_iob/tsar_iob_cluster/caba/source/src/tsar_iob_cluster.cpp (modified) (4 diffs)

branches/reconfiguration/modules/dspin_router/caba/source/include/dspin_router.h

@@ -94 +94 @@
     DspinInput<flit_width>      *p_in;
     DspinOutput<flit_width>     *p_out;
-    sc_in<uint32_t>             *p_blackhole_pos;
+
+    // reconfiguration port
+    sc_in<uint32_t>             *p_recovery_cfg;
 
     // constructor / destructor
@@ -104 +106 @@
                  const size_t    in_fifo_depth,
                  const size_t    out_fifo_depth,
-                 const bool      broadcast_supported = false,
-                 const bool      reconfigurable = false );
+                 const bool      broadcast_supported = false );
 
     ~DspinRouter();
@@ -146 +147 @@
     void    genMoore();
     int     xfirst_route( size_t xdest, size_t ydest );
-    int     recovery_route( size_t xdest, size_t ydest );
     int     route( sc_uint<flit_width> data );
     int     broadcast_route( int iter, int source, sc_uint<flit_width> data );
     bool    is_broadcast( sc_uint<flit_width> data );
 
+    // fault-recovery methods
+    bool    need_reroute( size_t xdest, size_t ydest, int bhpos );
+    int     recovery_route( size_t xdest, size_t ydest );
+
     public:
 
-    void set_disable_mask( int mask )
+    inline void set_disable_mask( int mask )
     {
         m_disable_mask = mask;
     }
+
+    void bind_recovery_port( sc_core::sc_signal<uint32_t> &s );
 
     void print_trace();

branches/reconfiguration/modules/dspin_router/caba/source/src/dspin_router.cpp

@@ -60 +60 @@
                              const size_t   in_fifo_depth,
                              const size_t   out_fifo_depth,
-                             const bool     broadcast_supported,
-                             const bool     reconfigurable )
+                             const bool     broadcast_supported)
     : soclib::caba::BaseModule(name),
 
@@ -117 +116 @@
         }
 
-        if ( reconfigurable ) p_blackhole_pos = new sc_core::sc_in<uint32_t>;
-        else                  p_blackhole_pos = NULL;
+        p_recovery_cfg = NULL;
     } //  end constructor
 
@@ -124 +122 @@
     tmpl(/**/)::~DspinRouter()
     {
-        if ( p_blackhole_pos != NULL ) delete p_blackhole_pos;
+        if ( p_recovery_cfg != NULL ) delete p_recovery_cfg;
     }
 
@@ -137 +135 @@
 
     ///////////////////////////////////////////////////
+    tmpl(void)::bind_recovery_port( sc_core::sc_signal<uint32_t> &s )
+    {
+        if (p_recovery_cfg == NULL) {
+            p_recovery_cfg = new sc_core::sc_in<uint32_t>;
+        }
+        (*p_recovery_cfg)(s);
+    }
+
+    ///////////////////////////////////////////////////
+    tmpl(bool)::need_reroute( size_t xdest, size_t ydest, int bhpos )
+    {
+        size_t xhole, yhole;
+        if (bhpos == BH_N) {
+            xhole = m_local_x;
+            yhole = m_local_y - 1;
+        }
+        else if (bhpos == BH_NW) {
+            xhole = m_local_x + 1;
+            yhole = m_local_y - 1;
+        }
+        else if (bhpos == BH_W) {
+            xhole = m_local_x + 1;
+            yhole = m_local_y;
+        }
+        else if (bhpos == BH_SW) {
+            xhole = m_local_x + 1;
+            yhole = m_local_y + 1;
+        }
+        else if (bhpos == BH_S) {
+            xhole = m_local_x;
+            yhole = m_local_y + 1;
+        }
+        else if (bhpos == BH_SE) {
+            xhole = m_local_x - 1;
+            yhole = m_local_y + 1;
+        }
+        else if (bhpos == BH_E) {
+            xhole = m_local_x - 1;
+            yhole = m_local_y;
+        }
+        else if (bhpos == BH_NE) {
+            xhole = m_local_x - 1;
+            yhole = m_local_y - 1;
+        }
+        else {
+            return false;
+        }
+
+        return ((xdest == xhole) && (ydest == yhole));
+    }
+
+    ///////////////////////////////////////////////////
     tmpl(int)::recovery_route( size_t xdest, size_t ydest )
     {
-        int bhpos = p_blackhole_pos->read();
+        int bhpos = p_recovery_cfg->read() & 0xF;
+
+        // reroute the request if its destination is the blackhole (this is to
+        // implement the segment recovery mechanism)
+        if (need_reroute(xdest, ydest, bhpos)) {
+            int recovery_direction = (p_recovery_cfg->read() >> 4) & 0xF;
+            return recovery_direction;
+        }
+
         if ( xdest > m_local_x ) {
             if ( (bhpos == BH_NE) || (bhpos == BH_E) || (bhpos == BH_SE) ||
@@ -249 +307 @@
         size_t xdest = (size_t)(data >> m_x_shift) & m_x_mask;
         size_t ydest = (size_t)(data >> m_y_shift) & m_y_mask;
-        if ( p_blackhole_pos != NULL )
-        {
-            if ( p_blackhole_pos->read() != BH_NONE )
+        if ( p_recovery_cfg != NULL )
+        {
+            if ( (p_recovery_cfg->read() & 0xF) != BH_NONE )
             {
                 return recovery_route(xdest, ydest);

branches/reconfiguration/platforms/tsar_generic_iob/tsar_iob_cluster/caba/source/src/tsar_iob_cluster.cpp

@@ -306 +306 @@
                 x_width, y_width,
                 4, 4,
-                (k == 1),
-                true);                 // use config interface
+                (k == 1));
     }
 
@@ -320 +319 @@
                 x_width, y_width,
                 4, 4,
-                false,
-                true);                 // use config interface
+                false);
     }
 
@@ -431 +429 @@
         int_router_cmd[k]->p_clk                 (this->p_clk);
         int_router_cmd[k]->p_resetn              (this->p_resetn);
-        (*int_router_cmd[k]->p_blackhole_pos)    (signal_cfg_router_cmd[k]);
+        int_router_cmd[k]->bind_recovery_port    (signal_cfg_router_cmd[k]);
         for (int i = 0; i < 4; i++)
         {
@@ -443 +441 @@
         int_router_rsp[k]->p_clk                 (this->p_clk);
         int_router_rsp[k]->p_resetn              (this->p_resetn);
-        (*int_router_rsp[k]->p_blackhole_pos)    (signal_cfg_router_rsp[k]);
+        int_router_rsp[k]->bind_recovery_port    (signal_cfg_router_rsp[k]);
         for (int i = 0; i < 4; i++)
         {