Changeset 599 for trunk/modules/vci_mem_cache/caba/source/src/vci_mem_cache.cpp

Timestamp:

Dec 11, 2013, 6:08:30 PM (10 years ago)

Author:

cfuguet

Message:

Improvement in vci_mem_cache component:

• No need to pass x_self and y_self constructor parameters. They are computed from the most significant bits of one of the memory segments mapped to this component.

• Erasing these two parameters from the vci_mem_cache constructor in all TSAR platforms

File:

• trunk/modules/vci_mem_cache/caba/source/src/vci_mem_cache.cpp (modified) (4 diffs)

trunk/modules/vci_mem_cache/caba/source/src/vci_mem_cache.cpp

@@ -334 +334 @@
             const IntTab        &srcid_x,          // global index on external network
             const IntTab        &tgtid_d,          // global index on direct network
-            const size_t        x_self,            // self x coordinate
-            const size_t        y_self,            // self y coordinate
             const size_t        x_width,           // number of x bits in platform
             const size_t        y_width,           // number of x bits in platform
@@ -367 +365 @@
         m_sets(nsets),
         m_words(nwords),
-        m_x_self(x_self),
-        m_y_self(y_self),
         m_x_width(x_width),
         m_y_width(y_width),
@@ -529 +525 @@
             }
 
+            assert( (m_nseg > 0) and
+                    "MEMC ERROR : At least one segment must be mapped to this component");
+
             m_seg = new soclib::common::Segment*[m_nseg];
 
@@ -537 +536 @@
                 i++;
             }
+
+            addr_t gid = m_seg[0]->baseAddress() >> (vci_param_int::N - x_width - y_width);
+            m_x_self = (gid >> m_y_width) & ((1 << m_x_width) - 1);
+            m_y_self =  gid               & ((1 << m_y_width) - 1);
 
             // Allocation for IXR_RSP FSM