Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
Cleanup parts of malek's mess: reintroduce the bunch of changes he erased in his...
[simgrid.git] / src / xbt / context.c
index 77040cd..4629739 100644 (file)
@@ -24,9 +24,19 @@ static xbt_context_t current_context = NULL;
 static xbt_context_t init_context = NULL;
 static xbt_swag_t context_to_destroy = NULL;
 static xbt_swag_t context_living = NULL;
+
 #ifdef CONTEXT_THREADS
 static xbt_os_mutex_t creation_mutex;
 static xbt_os_cond_t creation_cond;
+static xbt_os_mutex_t master_mutex;
+static xbt_os_cond_t master_cond;
+
+static xbt_os_thread_t current_thread_id;
+
+static int is_main_thread(void) {
+   return xbt_os_thread_self() == current_thread_id;
+}
+
 #endif
 
 
@@ -36,19 +46,21 @@ static xbt_os_cond_t creation_cond;
 #ifndef CONTEXT_THREADS
 /* callback: context fetching (used only with ucontext, os_thread deal with it
                                for us otherwise) */
-static ex_ctx_t *_context_ex_ctx(void) {
+static ex_ctx_t *_context_ex_ctx(void)
+{
   return current_context->exception;
 }
 
 /* callback: termination */
-static void _context_ex_terminate(xbt_ex_t * e) {
+static void _context_ex_terminate(xbt_ex_t * e)
+{
   xbt_ex_display(e);
 
   abort();
   /* FIXME: there should be a configuration variable to 
      choose to kill everyone or only this one */
 }
-#endif 
+#endif
 
 /** \name Functions 
  *  \ingroup XBT_context
@@ -63,16 +75,23 @@ void xbt_context_init(void)
   if (!current_context) {
     current_context = init_context = xbt_new0(s_xbt_context_t, 1);
     DEBUG1("Init Context (%p)", init_context);
-
-    init_context->iwannadie = 0; /* useless but makes valgrind happy */
+    init_context->iwannadie = 0;        /* useless but makes valgrind happy */
     context_to_destroy =
-       xbt_swag_new(xbt_swag_offset(*current_context, hookup));
-    context_living =
-       xbt_swag_new(xbt_swag_offset(*current_context, hookup));
+      xbt_swag_new(xbt_swag_offset(*current_context, hookup));
+    context_living = xbt_swag_new(xbt_swag_offset(*current_context, hookup));
     xbt_swag_insert(init_context, context_living);
+
 #ifdef CONTEXT_THREADS
+    /* only used during the creation of the processes */
     creation_mutex = xbt_os_mutex_init();
     creation_cond = xbt_os_cond_init();
+
+    /* used to schedule/unschedule the processes */
+    master_mutex = xbt_os_mutex_init();
+    master_cond = xbt_os_cond_init();
+
+    current_thread_id = xbt_os_thread_self();
+
 #else
     init_context->exception = xbt_new(ex_ctx_t, 1);
     XBT_CTX_INITIALIZE(init_context->exception);
@@ -91,17 +110,22 @@ void xbt_context_exit(void)
   xbt_context_t context = NULL;
 
   xbt_context_empty_trash();
+
   while ((context = xbt_swag_extract(context_living))) {
     if (context != init_context) {
       xbt_context_kill(context);
     }
   }
+
 #ifdef CONTEXT_THREADS
   xbt_os_mutex_destroy(creation_mutex);
   xbt_os_cond_destroy(creation_cond);
-#else   
+  xbt_os_mutex_destroy(master_mutex);
+  xbt_os_cond_destroy(master_cond);
+#else
   free(init_context->exception);
 #endif
+
   free(init_context);
   init_context = current_context = NULL;
 
@@ -125,11 +149,14 @@ void xbt_context_exit(void)
  * \param argc first argument of function \a code
  * \param argv seconde argument of function \a code
  */
-xbt_context_t xbt_context_new(const char *name,xbt_main_func_t code,
-                             void_f_pvoid_t startup_func,
-                             void *startup_arg,
-                             void_f_pvoid_t cleanup_func,
-                             void *cleanup_arg, int argc, char *argv[])
+xbt_context_t
+xbt_context_new(const char *name,
+                xbt_main_func_t code,
+                void_f_pvoid_t startup_func,
+                void *startup_arg,
+                void_f_pvoid_t cleanup_func,
+                void *cleanup_arg, int argc, char *argv[]
+  )
 {
   xbt_context_t res = NULL;
 
@@ -137,27 +164,27 @@ xbt_context_t xbt_context_new(const char *name,xbt_main_func_t code,
 
   res->code = code;
   res->name = xbt_strdup(name);
+
 #ifdef CONTEXT_THREADS
   res->mutex = xbt_os_mutex_init();
   res->cond = xbt_os_cond_init();
 #else
 
   xbt_assert2(getcontext(&(res->uc)) == 0,
-             "Error in context saving: %d (%s)", errno, strerror(errno));
+              "Error in context saving: %d (%s)", errno, strerror(errno));
   res->uc.uc_link = NULL;
   /*   res->uc.uc_link = &(current_context->uc); */
   /* WARNING : when this context is over, the current_context (i.e. the 
      father), is awaken... Theorically, the wrapper should prevent using 
      this feature. */
   res->uc.uc_stack.ss_sp = pth_skaddr_makecontext(res->stack, STACK_SIZE);
-  res->uc.uc_stack.ss_size =
-      pth_sksize_makecontext(res->stack, STACK_SIZE);
-   
+  res->uc.uc_stack.ss_size = pth_sksize_makecontext(res->stack, STACK_SIZE);
+
   res->exception = xbt_new(ex_ctx_t, 1);
   XBT_CTX_INITIALIZE(res->exception);
-#endif                         /* CONTEXT_THREADS or not */
+#endif /* CONTEXT_THREADS or not */
 
-  res->iwannadie = 0; /* useless but makes valgrind happy */
+  res->iwannadie = 0;           /* useless but makes valgrind happy */
 
   res->argc = argc;
   res->argv = argv;
@@ -193,18 +220,22 @@ xbt_context_t xbt_context_new(const char *name,xbt_main_func_t code,
  */
 
 /* Argument must be stopped first -- runs in maestro context */
-static void xbt_context_free(xbt_context_t context) {
+static void xbt_context_free(xbt_context_t context)
+{
   int i;
-   
+
   if (!context)
     return;
+
   DEBUG1("Freeing %p", context);
   free(context->name);
-   
+
   DEBUG0("Freeing arguments");
+
   for (i = 0; i < context->argc; i++)
     if (context->argv[i])
       free(context->argv[i]);
+
   if (context->argv)
     free(context->argv);
 
@@ -223,9 +254,8 @@ static void xbt_context_free(xbt_context_t context) {
   if (context->exception)
     free(context->exception);
 #endif
-   
+
   free(context);
-  return;
 }
 
 /************************/
@@ -243,24 +273,23 @@ static void *__context_wrapper(void *c)
   context->thread = xbt_os_thread_self();
 
   DEBUG3("**[ctx:%p;self:%p]** Lock creation_mutex %p ****", context,
-        (void *) xbt_os_thread_self(), creation_mutex);
+         (void *) xbt_os_thread_self(), creation_mutex);
   xbt_os_mutex_lock(creation_mutex);
   xbt_os_mutex_lock(context->mutex);
 
   DEBUG4
-      ("**[ctx:%p;self:%p]** Releasing the creator (creation_cond %p,%p) ****",
-       context, (void *) xbt_os_thread_self(), creation_cond,
-       creation_mutex);
+    ("**[ctx:%p;self:%p]** Releasing the creator (creation_cond %p,%p) ****",
+     context, (void *) xbt_os_thread_self(), creation_cond, creation_mutex);
   xbt_os_cond_signal(creation_cond);
   xbt_os_mutex_unlock(creation_mutex);
 
   DEBUG4("**[ctx:%p;self:%p]** Going to Jail on lock %p and cond %p ****",
-        context, (void *) xbt_os_thread_self(), context->mutex,
-        context->cond);
+         context, (void *) xbt_os_thread_self(), context->mutex,
+         context->cond);
   xbt_os_cond_wait(context->cond, context->mutex);
 
-  DEBUG3("**[ctx:%p;self:%p]** Unlocking individual %p ****",
-        context, (void *) xbt_os_thread_self(), context->mutex);
+  DEBUG3("**[ctx:%p;self:%p]** Unlocking individual %p ****", context,
+         (void *) xbt_os_thread_self(), context->mutex);
   xbt_os_mutex_unlock(context->mutex);
 
 #endif
@@ -273,6 +302,7 @@ static void *__context_wrapper(void *c)
   xbt_context_stop((context->code) (context->argc, context->argv));
   return NULL;
 }
+
 /** 
  * \param context the context to start
  * 
@@ -283,33 +313,33 @@ void xbt_context_start(xbt_context_t context)
 {
 #ifdef CONTEXT_THREADS
   /* Launch the thread */
-   
+
   DEBUG3("**[ctx:%p;self:%p]** Locking creation_mutex %p ****", context,
-        xbt_os_thread_self(), creation_mutex);
+         xbt_os_thread_self(), creation_mutex);
   xbt_os_mutex_lock(creation_mutex);
 
   DEBUG2("**[ctx:%p;self:%p]** Thread create ****", context,
-        xbt_os_thread_self());
-  context->thread = xbt_os_thread_create(context->name,__context_wrapper, context);
+         xbt_os_thread_self());
+  context->thread =
+    xbt_os_thread_create(context->name, __context_wrapper, context);
   DEBUG3("**[ctx:%p;self:%p]** Thread created : %p ****", context,
-        xbt_os_thread_self(), context->thread);
+         xbt_os_thread_self(), context->thread);
 
   DEBUG4
-      ("**[ctx:%p;self:%p]** Going to jail on creation_cond/mutex (%p,%p) ****",
-       context, xbt_os_thread_self(), creation_cond, creation_mutex);
+    ("**[ctx:%p;self:%p]** Going to jail on creation_cond/mutex (%p,%p) ****",
+     context, xbt_os_thread_self(), creation_cond, creation_mutex);
   xbt_os_cond_wait(creation_cond, creation_mutex);
   DEBUG3("**[ctx:%p;self:%p]** Unlocking creation %p ****", context,
-        xbt_os_thread_self(), creation_mutex);
+         xbt_os_thread_self(), creation_mutex);
   xbt_os_mutex_unlock(creation_mutex);
 #else
-  makecontext(&(context->uc), (void (*)(void)) __context_wrapper, 1,
-             context);
+  makecontext(&(context->uc), (void (*)(void)) __context_wrapper, 1, context);
 #endif
-  return;
 }
 
 /* Stops current context: calls user's cleanup function, kills os thread, and yields back to maestro */
-static void xbt_context_stop(int retvalue) {
+static void xbt_context_stop(int retvalue)
+{
   DEBUG1("--------- %p is exiting ---------", current_context);
 
   if (current_context->cleanup_func) {
@@ -324,19 +354,23 @@ static void xbt_context_stop(int retvalue) {
   DEBUG0("Yielding");
 
 #ifdef CONTEXT_THREADS
-  DEBUG2("[%p] **** Locking %p ****", current_context, current_context->mutex);
-  xbt_os_mutex_lock(current_context->mutex);
-  DEBUG1("[%p] **** Releasing the prisonner ****", current_context);
-  xbt_os_cond_signal(current_context->cond);
-  DEBUG2("[%p] **** Unlocking individual %p ****", current_context,
-        current_context->mutex);
-  xbt_os_mutex_unlock(current_context->mutex);
-  DEBUG1("[%p] **** Exiting ****", current_context);
-  xbt_os_thread_exit(NULL);    // We should provide return value in case other wants it
+  /* a java thread has called this function
+   * - update the current context
+   * - signal the condition of the main thread
+   * - wait on its condition
+   * - restore thr current contex
+   */
+
+  xbt_os_mutex_lock(master_mutex);
+  xbt_os_cond_signal(master_cond);
+  xbt_os_mutex_unlock(master_mutex);
+  xbt_os_thread_exit(NULL);     /* We should provide return value in case other wants it */
+
+
 #else
   __xbt_context_yield(current_context);
 #endif
-  THROW_IMPOSSIBLE();
+  THROW_IMPOSSIBLE;
 }
 
 
@@ -349,8 +383,10 @@ static void xbt_context_stop(int retvalue) {
 void xbt_context_empty_trash(void)
 {
   xbt_context_t context = NULL;
+
   DEBUG1("Emptying trashbin (%d contexts to free)",
-        xbt_swag_size(context_to_destroy));
+         xbt_swag_size(context_to_destroy));
+
   while ((context = xbt_swag_extract(context_to_destroy)))
     xbt_context_free(context);
 }
@@ -363,35 +399,67 @@ static void __xbt_context_yield(xbt_context_t context)
 {
 #ifdef CONTEXT_THREADS
   xbt_context_t self;
-#endif  
-xbt_assert0(current_context, "You have to call context_init() first.");        
-  xbt_assert0(context,"Invalid argument");
+#endif
+
+  xbt_assert0(current_context, "You have to call context_init() first.");
+  xbt_assert0(context, "Invalid argument");
 
   if (current_context == context) {
-    DEBUG1("--------- current_context (%p) is yielding back to maestro ---------",
-          context);
+    DEBUG1
+      ("--------- current_context (%p) is yielding back to maestro ---------",
+       context);
   } else {
-    DEBUG2("--------- current_context (%p) is yielding to context(%p) ---------",
+    DEBUG2
+      ("--------- current_context (%p) is yielding to context(%p) ---------",
        current_context, context);
   }
 
 #ifdef CONTEXT_THREADS
+
   self = current_context;
-  DEBUG2("[%p] **** Locking ctx %p ****", self, context);
-  xbt_os_mutex_lock(context->mutex);
-  DEBUG1("[%p] **** Updating current_context ****", self);
-  current_context = context;
-  DEBUG1("[%p] **** Releasing the prisonner ****", self);
-  xbt_os_cond_signal(context->cond);
-  DEBUG3("[%p] **** Going to jail on individual %p/%p ****", self,
-        context->cond, context->mutex);
-  xbt_os_cond_wait(context->cond, context->mutex);
-  DEBUG2("[%p] **** Unlocking individual %p ****", self, context->mutex);
-  xbt_os_mutex_unlock(context->mutex);
-  DEBUG1("[%p] **** Updating current_context ****", self);
-  current_context = self;
 
-#else                          /* use SUSv2 contexts */
+  if (is_main_thread()) {
+    /* the main thread has called this function
+     * - update the current context
+     * - signal the condition of the process to run
+     * - wait on its condition
+     * - restore thr current contex
+     */
+
+    xbt_os_mutex_lock(master_mutex);
+    xbt_os_mutex_lock(context->mutex);
+
+    /* update the current context */
+    current_context = context;
+    xbt_os_cond_signal(context->cond);
+    xbt_os_mutex_unlock(context->mutex);
+
+    xbt_os_cond_wait(master_cond, master_mutex);
+    xbt_os_mutex_unlock(master_mutex);
+    /* retore the current context */
+    current_context = self;
+
+  } else {
+    /* a java thread has called this function
+     * - update the current context
+     * - signal the condition of the main thread
+     * - wait on its condition
+     * - restore thr current contex
+     */
+
+    xbt_os_mutex_lock(master_mutex);
+    xbt_os_mutex_lock(context->mutex);
+    /* update the current context */
+    current_context = context;
+    xbt_os_cond_signal(master_cond);
+    xbt_os_mutex_unlock(master_mutex);
+    xbt_os_cond_wait(context->cond, context->mutex);
+    xbt_os_mutex_unlock(context->mutex);
+    /* retore the current context */
+    current_context = self;
+  }
+
+#else /* use SUSv2 contexts */
   VOIRP(current_context);
   VOIRP(current_context->save);
 
@@ -403,35 +471,35 @@ xbt_assert0(current_context, "You have to call context_init() first.");
   if (context->save == NULL) {
     DEBUG1("[%p] **** Yielding to somebody else ****", current_context);
     DEBUG2("Saving current_context value (%p) to context(%p)->save",
-            current_context, context);
+           current_context, context);
     context->save = current_context;
     DEBUG1("current_context becomes  context(%p) ", context);
     current_context = context;
     DEBUG1
-         ("Current position memorized (context->save). Jumping to context (%p)",
-          context);
+      ("Current position memorized (context->save). Jumping to context (%p)",
+       context);
     return_value = swapcontext(&(context->save->uc), &(context->uc));
     xbt_assert0((return_value == 0), "Context swapping failure");
     DEBUG1("I am (%p). Coming back\n", context);
   } else {
     xbt_context_t old_context = context->save;
+
     DEBUG1("[%p] **** Back ! ****", context);
     DEBUG2("Setting current_context (%p) to context(%p)->save",
-          current_context, context);
+           current_context, context);
     current_context = context->save;
     DEBUG1("Setting context(%p)->save to NULL", context);
     context->save = NULL;
     DEBUG2("Current position memorized (%p). Jumping to context (%p)",
-          context, old_context);
+           context, old_context);
     return_value = swapcontext(&(context->uc), &(old_context->uc));
     xbt_assert0((return_value == 0), "Context swapping failure");
     DEBUG1("I am (%p). Coming back\n", context);
   }
 #endif
+
   if (current_context->iwannadie)
     xbt_context_stop(1);
-
-  return;
 }
 
 
@@ -464,7 +532,7 @@ void xbt_context_schedule(xbt_context_t context)
 {
   DEBUG1("Scheduling %p", context);
   xbt_assert0((current_context == init_context),
-             "You are not supposed to run this function here!");
+              "You are not supposed to run this function here!");
   __xbt_context_yield(context);
 }
 
@@ -479,17 +547,26 @@ void xbt_context_kill(xbt_context_t context)
   DEBUG1("Killing %p", context);
 
   context->iwannadie = 1;
+
   DEBUG1("Scheduling %p", context);
   __xbt_context_yield(context);
   DEBUG1("End of Scheduling %p", context);
-
-  return;
 }
 
 /* Java cruft I'm gonna kill in the next cleanup round */
-void  xbt_context_set_jprocess(xbt_context_t context, void *jp){}
-void* xbt_context_get_jprocess(xbt_context_t context){return NULL;}
-void  xbt_context_set_jenv(xbt_context_t context,void* je){}
-void* xbt_context_get_jenv(xbt_context_t context){return NULL;}
+void xbt_context_set_jprocess(xbt_context_t context, void *jp)
+{
+}
+void *xbt_context_get_jprocess(xbt_context_t context)
+{
+  return NULL;
+}
+void xbt_context_set_jenv(xbt_context_t context, void *je)
+{
+}
+void *xbt_context_get_jenv(xbt_context_t context)
+{
+  return NULL;
+}
 
 /* @} */