aboutsummaryrefslogtreecommitdiff
path: root/bsd-user
diff options
context:
space:
mode:
authorWarner Losh <imp@FreeBSD.org>2021-08-07 14:22:34 -0600
committerWarner Losh <imp@bsdimp.com>2021-09-10 14:13:06 -0600
commit63cca1067ac70df2d71c0064f7bf53e19334223c (patch)
tree4d02d6d0e6fbbec861ce27212c7d73f9bbfbd94f /bsd-user
parentf0f7f9dca986bebf2a0b88dd83f84f5e39a61462 (diff)
downloadqemu-63cca1067ac70df2d71c0064f7bf53e19334223c.zip
qemu-63cca1067ac70df2d71c0064f7bf53e19334223c.tar.gz
qemu-63cca1067ac70df2d71c0064f7bf53e19334223c.tar.bz2
bsd-user: Implement interlock for atomic operations
Implement the internlock in fork_start() and fork_end() to properly cope with atomic operations and to safely keep state for parent and child processes. Signed-off-by: Stacey Son <sson@FreeBSD.org> Signed-off-by: Warner Losh <imp@bsdimp.com> Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Diffstat (limited to 'bsd-user')
-rw-r--r--bsd-user/main.c24
1 files changed, 24 insertions, 0 deletions
diff --git a/bsd-user/main.c b/bsd-user/main.c
index 768f4ee..659dd2b 100644
--- a/bsd-user/main.c
+++ b/bsd-user/main.c
@@ -69,15 +69,39 @@ unsigned long target_dflssiz = TARGET_DFLSSIZ; /* initial data size limit */
unsigned long target_maxssiz = TARGET_MAXSSIZ; /* max stack size */
unsigned long target_sgrowsiz = TARGET_SGROWSIZ; /* amount to grow stack */
+/* Helper routines for implementing atomic operations. */
void fork_start(void)
{
+ start_exclusive();
+ cpu_list_lock();
+ mmap_fork_start();
}
void fork_end(int child)
{
if (child) {
+ CPUState *cpu, *next_cpu;
+ /*
+ * Child processes created by fork() only have a single thread. Discard
+ * information about the parent threads.
+ */
+ CPU_FOREACH_SAFE(cpu, next_cpu) {
+ if (cpu != thread_cpu) {
+ QTAILQ_REMOVE_RCU(&cpus, cpu, node);
+ }
+ }
+ mmap_fork_end(child);
+ /*
+ * qemu_init_cpu_list() takes care of reinitializing the exclusive
+ * state, so we don't need to end_exclusive() here.
+ */
+ qemu_init_cpu_list();
gdbserver_fork(thread_cpu);
+ } else {
+ mmap_fork_end(child);
+ cpu_list_unlock();
+ end_exclusive();
}
}