Add fork hooks for pthread_atfork
authorSamuel Thibault <samuel.thibault@ens-lyon.org>
Tue, 24 Sep 2013 21:08:15 +0000 (23:08 +0200)
committerSamuel Thibault <samuel.thibault@ens-lyon.org>
Fri, 4 Oct 2013 20:29:51 +0000 (22:29 +0200)
pthread_atfork needs application callbacks to be called outside any locking.

ChangeLog
sysdeps/mach/hurd/fork.c

index 2ff5516..1c3f685 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2013-10-04  Samuel Thibault  <samuel.thibault@ens-lyon.org>
+
+       * sysdeps/mach/hurd/fork.c (_hurd_atfork_prepare_hook,
+       _hurd_atfork_child_hook, _hurd_atfork_parent_hook): New hooks.
+       (__fork): Call _hurd_atfork_prepare_hook hooks before all locking, call
+       _hurd_atfork_parent_hook or _hurd_atfork_child_hook after all unlocking.
+
 2013-10-04  Ryan S. Arnold  <ryan.arnold@linaro.org>
 
        * misc/swapon.c (swapon): Update definition, adding FLAGS parameter to
index ab11bab..321421f 100644 (file)
 symbol_set_declare (_hurd_fork_locks)
 
 
+/* Application callbacks registered through pthread_atfork.  */
+DEFINE_HOOK (_hurd_atfork_prepare_hook, (void));
+DEFINE_HOOK (_hurd_atfork_child_hook, (void));
+DEFINE_HOOK (_hurd_atfork_parent_hook, (void));
+
 /* Things that want to be called before we fork, to prepare the parent for
    task_create, when the new child task will inherit our address space.  */
 DEFINE_HOOK (_hurd_fork_prepare_hook, (void));
@@ -62,6 +67,8 @@ __fork (void)
   error_t err;
   struct hurd_sigstate *volatile ss;
 
+  RUN_HOOK (_hurd_atfork_prepare_hook, ());
+
   ss = _hurd_self_sigstate ();
   __spin_lock (&ss->critical_section_lock);
 
@@ -695,6 +702,14 @@ __fork (void)
 
   _hurd_critical_section_unlock (ss);
 
+  if (!err)
+    {
+      if (pid != 0)
+       RUN_HOOK (_hurd_atfork_parent_hook, ());
+      else
+       RUN_HOOK (_hurd_atfork_child_hook, ());
+    }
+
   return err ? __hurd_fail (err) : pid;
 }
 libc_hidden_def (__fork)