Changeset 933 for branches

Timestamp:

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

Author:

cfuguet

Message:

reconf: vci_local_crossbar: locality of packets coming from the
global network is not checked.

• This is to allow a physical address space segment to be migrated from a faulty cluster to one of its neighbors. However, when this guess segment is accessed by a core of the host cluster, the request must go to the global network which will immediately return it.

File:

• branches/reconfiguration/modules/vci_local_crossbar/caba/source/src/vci_local_crossbar.cpp (modified) (1 diff)

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

@@ -241 +241 @@
                         pkt_t tmp;
                         tmp.readFrom(*input_port[in]);
-                        if ( (tmp.is_broadcast() and 
-                              r_bc_state[in].read() and 
-                              (r_bc_count[in].read() == out))   or   // broadcast request
-                             (not tmp.is_broadcast() and 
-                              not is_local( tmp ) and 
-                              (out == m_out_size-1))            or   // to global network
-                             (not tmp.is_broadcast() and
-                              is_local( tmp ) and
-                              (out == route( tmp )) ) )              // to local component
+
+                        // broadcast request
+                        bool bc_req = tmp.is_broadcast() and
+                                      r_bc_state[in].read() and (r_bc_count[in].read() == out);
+
+                        // to global request
+                        bool gl_req = not tmp.is_broadcast() and (out == m_out_size-1) and
+                                      not is_local( tmp ) and (in != m_in_size-1);
+
+                        // to local request
+                        bool lc_req = not tmp.is_broadcast() and (out == route( tmp )) and
+                                      is_local( tmp );
+
+                        // to migrated segment request (to local)
+                        bool mg_req = not tmp.is_broadcast() and (out == route( tmp )) and
+                                      not is_local( tmp ) and (in == m_in_size-1);
+
+                        if ( bc_req or gl_req or lc_req or mg_req )
                         {
                             r_allocated[out] = true;