Changeset 416 for trunk/libs/malloc.c

Timestamp:

Jan 4, 2018, 10:05:47 AM (6 years ago)

Author:

alain

Message:

Improve sys_exec.

File:

• trunk/libs/malloc.c (modified) (4 diffs)

trunk/libs/malloc.c

@@ -107 +107 @@
 // It uses the mmap( MAP_REMOTE ) syscall to allocate a new vseg mapped in cluster (cxy).
 ////////////////////////////////////////////////////////////////////i//////////////////////
-// @ cxy       : target cluster identifier (fixed format).
-// @ size      : store size (bytes).
+// @ cxy        : target cluster identifier (fixed format).
+// @ store_size : store size (bytes).
 // # return without setting the initialized field in store(cxy) if failure.
 ////////////////////////////////////////////////////////////////////i//////////////////////
@@ -185 +185 @@
     }
 
-    // reset the alloc_size array
+    // DEPRECATED: we don't reset the alloc_size array 
+    // because we don't want to allocate the physical memory
+    // when the heap is created  [AG]
     // memset( (void *)alloc_base , 0 , alloc_size ); 
  
@@ -216 +218 @@
     store[cxy].initialized = MALLOC_INITIALIZED;
 
+
 #if MALLOC_DEBUG
-printf("\n[MALLOC] %s completes store[%x] initialisation\n",
+printf("\n[MALLOC] %s : completes store[%x] initialisation\n",
 __FUNCTION__, cxy );
+
+display_free_array( cxy );
 #endif
 
@@ -344 +349 @@
     unsigned char * ptr    = (unsigned char*)(store[cxy].alloc_base + offset);
 
-    // check the alloc[] array
-    if ( *ptr != 0 )
-    {
-        pthread_mutex_unlock( &store[cxy].mutex );
-        printf("\n[PANIC] in %s : allocate an already allocated block...\n",
-        __FUNCTION__ );
-        return NULL;
-    }
+    // DEPRECATED : we don't check the alloc[] array,
+    // because it has not been initialised, to avoid 
+    // physical memory allocation at heap creation [AG]
+    // if ( *ptr != 0 )
+    // {
+    //    pthread_mutex_unlock( &store[cxy].mutex );
+    //    printf("\n[PANIC] in %s : allocate an already allocated block...\n",
+    //    __FUNCTION__ );
+    //    return NULL;
+    // }
 
     // update alloc_array