diff options
| author | Emilio G. Cota | 2015-08-24 02:23:37 +0200 |
|---|---|---|
| committer | Paolo Bonzini | 2015-09-09 15:34:54 +0200 |
| commit | 5243722376873a48e9852a58b91f4d4101ee66e4 (patch) | |
| tree | 310a56c79e1e5d6d4119ccf706abeeffdb1939c7 /util | |
| parent | Makefile.target: include top level build dir in vpath (diff) | |
| download | qemu-5243722376873a48e9852a58b91f4d4101ee66e4.tar.gz qemu-5243722376873a48e9852a58b91f4d4101ee66e4.tar.xz qemu-5243722376873a48e9852a58b91f4d4101ee66e4.zip | |
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
diff --git a/util/rcu.c b/util/rcu.c index 8ba304dc44..47c2bceac8 100644 --- a/util/rcu.c +++ b/util/rcu.c @@ -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(); } |