]> xenbits.xen.org Git - xenclient/ioemu.git/commit
Fix race condition on access to env->interrupt_request
authoraurel32 <aurel32@c046a42c-6fe2-441c-8c8c-71466251a162>
Fri, 6 Mar 2009 21:49:37 +0000 (21:49 +0000)
committeraurel32 <aurel32@c046a42c-6fe2-441c-8c8c-71466251a162>
Fri, 6 Mar 2009 21:49:37 +0000 (21:49 +0000)
commit8a11f5ff08a2c351d6262617ee50a7a7c8d11f68
treefb891ca5b08ae5c1a7b89d222b8cef5f7561632a
parentb786b7ccbd375c052ffa9d3a818b88ee4870849c
Fix race condition on access to env->interrupt_request

env->interrupt_request is accessed as the bit level from both main code
and signal handler, making a race condition possible even on CISC CPU.
This causes freeze of QEMU under high load when running the dyntick
clock.

The patch below move the bit corresponding to CPU_INTERRUPT_EXIT in a
separate variable, declared as volatile sig_atomic_t, so it should be
work even on RISC CPU.

We may want to move the cpu_interrupt(env, CPU_INTERRUPT_EXIT) case in
its own function and get rid of CPU_INTERRUPT_EXIT. That can be done
later, I wanted to keep the patch short for easier review.

Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
git-svn-id: svn://svn.savannah.nongnu.org/qemu/branches/stable_0_10_0@6729 c046a42c-6fe2-441c-8c8c-71466251a162
cpu-defs.h
cpu-exec.c
exec.c
kvm-all.c