Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
Reviewed locking in mmalloc:
[simgrid.git] / src / xbt / mmalloc / mm_legacy.c
index 82da344..fbfcd95 100644 (file)
 
 #include "mmprivate.h"
 
-static void *__mmalloc_current_heap=NULL; /* The heap we are currently using. */
+static void *__mmalloc_current_heap = NULL;     /* The heap we are currently using. */
 
 #include "xbt_modinter.h"
 
-void* mmalloc_get_current_heap(void) {
+void *mmalloc_get_current_heap(void)
+{
   return __mmalloc_current_heap;
 }
-void mmalloc_set_current_heap(void *new_heap) {
-  __mmalloc_current_heap=new_heap;
+
+void mmalloc_set_current_heap(void *new_heap)
+{
+  __mmalloc_current_heap = new_heap;
 }
 
-#ifdef MMALLOC_WANT_OVERIDE_LEGACY
-void *malloc(size_t n) {
+#if 1//def MMALLOC_WANT_OVERIDE_LEGACY
+void *malloc(size_t n)
+{
 #ifdef HAVE_MMAP
-  if (!__mmalloc_current_heap) mmalloc_preinit();
+  if (!__mmalloc_current_heap)
+    mmalloc_preinit();
 #endif
+  LOCK(__mmalloc_current_heap);
   void *ret = mmalloc(__mmalloc_current_heap, n);
+  UNLOCK(__mmalloc_current_heap);
 
   return ret;
 }
 
-void *calloc(size_t nmemb, size_t size) {
+void *calloc(size_t nmemb, size_t size)
+{
   size_t total_size = nmemb * size;
 #ifdef HAVE_MMAP
-  if (!__mmalloc_current_heap) mmalloc_preinit();
+  if (!__mmalloc_current_heap)
+    mmalloc_preinit();
 #endif
+  LOCK(__mmalloc_current_heap);
   void *ret = mmalloc(__mmalloc_current_heap, total_size);
+  UNLOCK(__mmalloc_current_heap);
 
   /* Fill the allocated memory with zeroes to mimic calloc behaviour */
-  memset(ret,'\0', total_size);
+  memset(ret, '\0', total_size);
 
   return ret;
 }
 
-void *realloc(void *p, size_t s) {
+void *realloc(void *p, size_t s)
+{
   void *ret = NULL;
 #ifdef HAVE_MMAP
-  if (!__mmalloc_current_heap) mmalloc_preinit();
+  if (!__mmalloc_current_heap)
+    mmalloc_preinit();
 #endif
+  LOCK(__mmalloc_current_heap);
   if (s) {
-       if (p)
-         ret = mrealloc(__mmalloc_current_heap, p,s);
-       else
-         ret = mmalloc(__mmalloc_current_heap,s);
+    if (p)
+      ret = mrealloc(__mmalloc_current_heap, p, s);
+    else
+      ret = mmalloc(__mmalloc_current_heap, s);
   } else {
-       if (p)
-         mfree(__mmalloc_current_heap,p);
+    if (p)
+      mfree(__mmalloc_current_heap, p);
   }
+  UNLOCK(__mmalloc_current_heap);
 
   return ret;
 }
 
-void free(void *p) {
-  return mfree(__mmalloc_current_heap, p);
+void free(void *p)
+{
+  LOCK(__mmalloc_current_heap);
+  mfree(__mmalloc_current_heap, p);
+  UNLOCK(__mmalloc_current_heap);
 }
 #endif
 
 /* Make sure it works with md==NULL */
 
-#define HEAP_OFFSET   40960000    /* Safety gap from the heap's break address */
+#define HEAP_OFFSET   (128<<20)  /* Safety gap from the heap's break address.
+                                  * Try to increase this first if you experience
+                                  * strange errors under valgrind. */
 
-void *mmalloc_get_default_md(void) {
+void *mmalloc_get_default_md(void)
+{
   xbt_assert(__mmalloc_default_mdp);
   return __mmalloc_default_mdp;
 }
 
+static void mmalloc_fork_prepare(void)
+{
+  if (__mmalloc_default_mdp)
+    LOCK(__mmalloc_default_mdp);
+}
+
+static void mmalloc_fork_finish(void)
+{
+  if (__mmalloc_default_mdp)
+    UNLOCK(__mmalloc_default_mdp);
+}
+
 /* Initialize the default malloc descriptor. */
-void mmalloc_preinit(void) {
-  if(!__mmalloc_default_mdp) __mmalloc_default_mdp = mmalloc_attach(-1, (char *)sbrk(0) + HEAP_OFFSET);
+void mmalloc_preinit(void)
+{
+  if (!__mmalloc_default_mdp) {
+    __mmalloc_default_mdp =
+        mmalloc_attach(-1, (char *) sbrk(0) + HEAP_OFFSET);
+    /* Fixme? only the default mdp in protected against forks */
+    if (xbt_os_thread_atfork(mmalloc_fork_prepare,
+                             mmalloc_fork_finish, mmalloc_fork_finish) != 0)
+      abort();
+  }
   xbt_assert(__mmalloc_default_mdp != NULL);
 }
-void mmalloc_postexit(void) {
+
+void mmalloc_postexit(void)
+{
   /* Do not detach the default mdp or ldl won't be able to free the memory it allocated since we're in memory */
   //  mmalloc_detach(__mmalloc_default_mdp);
+  mmalloc_pre_detach(__mmalloc_default_mdp);
 }