diff options
| author | Paolo Bonzini <pbonzini@redhat.com> | 2016-01-25 13:46:43 +0100 |
|---|---|---|
| committer | Paolo Bonzini <pbonzini@redhat.com> | 2016-02-09 15:46:54 +0100 |
| commit | ac5e8acdaec7c3a97d460e76ea912fa8ea7a4570 (patch) | |
| tree | c362a51a95ec57b3027ef5fd0a638454a969d461 /hw | |
| parent | 1d1cc4d0f481b2939c7e9f6606e571b2fc81971a (diff) | |
| download | qemu-ac5e8acdaec7c3a97d460e76ea912fa8ea7a4570.zip qemu-ac5e8acdaec7c3a97d460e76ea912fa8ea7a4570.tar.gz qemu-ac5e8acdaec7c3a97d460e76ea912fa8ea7a4570.tar.bz2 | |
ipmi: do not take/drop iothread lock
This is not necessary and actually causes a hang; it was probably copied
and pasted from KVM code, that is one of the very few places that run
outside iothread lock.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'hw')
| -rw-r--r-- | hw/ipmi/ipmi.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/hw/ipmi/ipmi.c b/hw/ipmi/ipmi.c index dfab272..6adec1e 100644 --- a/hw/ipmi/ipmi.c +++ b/hw/ipmi/ipmi.c @@ -51,9 +51,7 @@ static int ipmi_do_hw_op(IPMIInterface *s, enum ipmi_op op, int checkonly) if (checkonly) { return 0; } - qemu_mutex_lock_iothread(); qmp_inject_nmi(NULL); - qemu_mutex_unlock_iothread(); return 0; case IPMI_POWERCYCLE_CHASSIS: |