Browse Source

port r11506 from 1.1: Do not remove reused memchunk from the cache, it was already removed in _starpu_memchunk_cache_lookup_locked. Thanks Cyril for managing to find a case where it crashes :)

Samuel Thibault 11 years ago
parent
commit
806d128705
1 changed files with 0 additions and 8 deletions
  1. 0 8
      src/datawizard/memalloc.c

+ 0 - 8
src/datawizard/memalloc.c

@@ -408,14 +408,6 @@ static void reuse_mem_chunk(unsigned node, struct _starpu_data_replicate *new_re
 	 * of the "to free" list, and reassign it to the new
 	 * piece of data */
 
-	if (!is_already_in_mc_list)
-	{
-		uint32_t footprint = _starpu_compute_data_footprint(new_replicate->handle);
-		struct mc_cache_entry *entry;
-		HASH_FIND(hh, mc_cache[node], &footprint, sizeof(footprint), entry);
-		_starpu_mem_chunk_list_erase(entry->list, mc);
-	}
-
 	struct _starpu_data_replicate *old_replicate = mc->replicate;
 	old_replicate->allocated = 0;
 	old_replicate->automatically_allocated = 0;