Changeset 752 for trunk/modules/vci_mem_cache/caba

Timestamp:

Jul 18, 2014, 11:38:54 AM (10 years ago)

Author:

meunier

Message:

• Added initialization to some components in order to avoid valgrind errors
• Corrected two bugs which caused the simulations to be non-deterministic:
  • one in the memcache (possibly uninitialized paddr variable used for cache access, modifying LRU bits)
  • one in the run_simus.py script (replacing a file with an identical file in a hdd image does modify its size)

Location:

trunk/modules/vci_mem_cache/caba/source/include

Files:

• mem_cache_directory.h (modified) (1 diff)
• vci_mem_cache.h (modified) (1 diff)

trunk/modules/vci_mem_cache/caba/source/include/mem_cache_directory.h

@@ -650 +650 @@
               {
                   m_cache_data[i][j] = new uint32_t [words];
+                  // Init to avoid potential errors from memory checkers
+                  std::memset(m_cache_data[i][j], 0, sizeof(uint32_t) * words);
               }
           }

trunk/modules/vci_mem_cache/caba/source/include/vci_mem_cache.h

@@ -404 +404 @@
 
       // instrumentation counters
-      uint32_t     m_cpt_cycles;         // Counter of cycles
-      uint32_t     m_cpt_reset_count;    // Cycle at which the counters were last reset
+      uint64_t     m_cpt_cycles;         // Counter of cycles
+      uint64_t     m_cpt_reset_count;    // Cycle at which the counters were last reset
 
       // Counters accessible in software (not yet but eventually)