diff options
author | Emilio G. Cota <cota@braap.org> | 2015-08-23 20:23:37 -0400 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2015-09-09 15:34:54 +0200 |
commit | 5243722376873a48e9852a58b91f4d4101ee66e4 (patch) | |
tree | 310a56c79e1e5d6d4119ccf706abeeffdb1939c7 /util | |
parent | 12a1ddc160cb6a73e8a6c319f3962a20da2cd22f (diff) | |
download | qemu-5243722376873a48e9852a58b91f4d4101ee66e4.zip qemu-5243722376873a48e9852a58b91f4d4101ee66e4.tar.gz qemu-5243722376873a48e9852a58b91f4d4101ee66e4.tar.bz2 |
rcu: init rcu_registry_lock after fork
We were unlocking this lock after fork, which is wrong since
only the thread that holds a mutex is allowed to unlock it.
Signed-off-by: Emilio G. Cota <cota@braap.org>
Message-Id: <1440375847-17603-9-git-send-email-cota@braap.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'util')
-rw-r--r-- | util/rcu.c | 7 |
1 files changed, 6 insertions, 1 deletions
@@ -335,6 +335,11 @@ static void rcu_init_unlock(void) qemu_mutex_unlock(&rcu_registry_lock); qemu_mutex_unlock(&rcu_sync_lock); } + +static void rcu_init_child(void) +{ + qemu_mutex_init(&rcu_registry_lock); +} #endif void rcu_after_fork(void) @@ -346,7 +351,7 @@ void rcu_after_fork(void) static void __attribute__((__constructor__)) rcu_init(void) { #ifdef CONFIG_POSIX - pthread_atfork(rcu_init_lock, rcu_init_unlock, rcu_init_unlock); + pthread_atfork(rcu_init_lock, rcu_init_unlock, rcu_init_child); #endif rcu_init_complete(); } |