aboutsummaryrefslogtreecommitdiff
path: root/cpus-common.c
diff options
context:
space:
mode:
authorPaolo Bonzini <pbonzini@redhat.com>2016-09-02 21:02:10 +0200
committerPaolo Bonzini <pbonzini@redhat.com>2016-09-27 11:57:30 +0200
commitcf07da65f335b9a74e62f5413078f67280572f36 (patch)
tree7768c32680b0d7b04bec4cced6e0fd22ff1dbe01 /cpus-common.c
parentc978b3168727d3a76ffcb18462ea972f50b53634 (diff)
downloadqemu-cf07da65f335b9a74e62f5413078f67280572f36.zip
qemu-cf07da65f335b9a74e62f5413078f67280572f36.tar.gz
qemu-cf07da65f335b9a74e62f5413078f67280572f36.tar.bz2
cpus-common: remove redundant call to exclusive_idle()
No need to call exclusive_idle() from cpu_exec_end since it is done immediately afterwards in cpu_exec_start. Any exclusive section could run as soon as cpu_exec_end leaves, because cpu->running is false and the mutex is not taken, so the call does not add any protection either. Reviewed-by: Richard Henderson <rth@twiddle.net> Reviewed-by: Alex Bennée <alex.bennee@linaro.org> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'cpus-common.c')
-rw-r--r--cpus-common.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/cpus-common.c b/cpus-common.c
index 115f3d4..80aaf9b 100644
--- a/cpus-common.c
+++ b/cpus-common.c
@@ -221,7 +221,6 @@ void cpu_exec_end(CPUState *cpu)
qemu_cond_signal(&exclusive_cond);
}
}
- exclusive_idle();
qemu_mutex_unlock(&qemu_cpu_list_lock);
}