X-Git-Url: https://git.lttng.org/?p=urcu.git;a=blobdiff_plain;f=urcu-bp.c;h=04bb6756a2bbe5aa8603c0ed148eae8c95a6be23;hp=db2aef0c57b8ef397095cd71d6c12ded119bb9ec;hb=85b577030e0dd244ed2d42aa5196a088e1e64dbb;hpb=d123233afd72d7556d8eec3a1cf07b0af7b4b3d2 diff --git a/urcu-bp.c b/urcu-bp.c index db2aef0..04bb675 100644 --- a/urcu-bp.c +++ b/urcu-bp.c @@ -76,6 +76,9 @@ struct registry_arena { static struct registry_arena registry_arena; +/* Saved fork signal mask, protected by rcu_gp_lock */ +static sigset_t saved_fork_signal_mask; + static void rcu_gc_registry(void); static void mutex_lock(pthread_mutex_t *mutex) @@ -331,3 +334,44 @@ void rcu_bp_exit() { munmap(registry_arena.p, registry_arena.len); } + +/* + * Holding the rcu_gp_lock across fork will make sure we fork() don't race with + * a concurrent thread executing with this same lock held. This ensures that the + * registry is in a coherent state in the child. + */ +void rcu_bp_before_fork(void) +{ + sigset_t newmask, oldmask; + int ret; + + ret = sigemptyset(&newmask); + assert(!ret); + ret = pthread_sigmask(SIG_SETMASK, &newmask, &oldmask); + assert(!ret); + mutex_lock(&rcu_gp_lock); + saved_fork_signal_mask = oldmask; +} + +void rcu_bp_after_fork_parent(void) +{ + sigset_t oldmask; + int ret; + + oldmask = saved_fork_signal_mask; + mutex_unlock(&rcu_gp_lock); + ret = pthread_sigmask(SIG_SETMASK, &oldmask, NULL); + assert(!ret); +} + +void rcu_bp_after_fork_child(void) +{ + sigset_t oldmask; + int ret; + + rcu_gc_registry(); + oldmask = saved_fork_signal_mask; + mutex_unlock(&rcu_gp_lock); + ret = pthread_sigmask(SIG_SETMASK, &oldmask, NULL); + assert(!ret); +}