diff options
author | Emilio G. Cota <cota@braap.org> | 2018-08-19 05:13:35 -0400 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2018-08-23 18:46:25 +0200 |
commit | 068a5ea02f62853116788a2c42d8851a94bb7567 (patch) | |
tree | 0e7c635241a440b1aeee6dd853077fad928a40d4 /linux-user/main.c | |
parent | 04d595b300041bec645c862baccbea1c8befb6d4 (diff) | |
download | qemu-068a5ea02f62853116788a2c42d8851a94bb7567.zip qemu-068a5ea02f62853116788a2c42d8851a94bb7567.tar.gz qemu-068a5ea02f62853116788a2c42d8851a94bb7567.tar.bz2 |
qom: convert the CPU list to RCU
Iterating over the list without using atomics is undefined behaviour,
since the list can be modified concurrently by other threads (e.g.
every time a new thread is created in user-mode).
Fix it by implementing the CPU list as an RCU QTAILQ. This requires
a little bit of extra work to traverse list in reverse order (see
previous patch), but other than that the conversion is trivial.
Signed-off-by: Emilio G. Cota <cota@braap.org>
Message-Id: <20180819091335.22863-12-cota@braap.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'linux-user/main.c')
-rw-r--r-- | linux-user/main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/linux-user/main.c b/linux-user/main.c index ea00dd9..923cbb7 100644 --- a/linux-user/main.c +++ b/linux-user/main.c @@ -126,7 +126,7 @@ void fork_end(int child) Discard information about the parent threads. */ CPU_FOREACH_SAFE(cpu, next_cpu) { if (cpu != thread_cpu) { - QTAILQ_REMOVE(&cpus, cpu, node); + QTAILQ_REMOVE_RCU(&cpus, cpu, node); } } qemu_init_cpu_list(); |