diff options
| author | Paolo Bonzini <pbonzini@redhat.com> | 2018-07-20 08:31:09 +0200 |
|---|---|---|
| committer | Gerd Hoffmann <kraxel@redhat.com> | 2018-08-24 08:40:11 +0200 |
| commit | 006bf4641b553565990a6b18e0cff36972a36773 (patch) | |
| tree | 7cb7434fe9eeeaa4cddec35da1f1c6033014a05b /qobject/json-parser-int.h | |
| parent | 2480f41e685690511f8f15bbeabb6cdc0b2ddaf0 (diff) | |
| download | focaccia-qemu-006bf4641b553565990a6b18e0cff36972a36773.tar.gz focaccia-qemu-006bf4641b553565990a6b18e0cff36972a36773.zip | |
spice-display: fix qemu_spice_cursor_refresh_bh locking
spice-display should not call the ui/console.c functions dpy_cursor_define and dpy_moues_set with the SimpleSpiceDisplay lock taken. That will cause a deadlock, because the DisplayChangeListener callbacks will take the lock again. It is also in general a bad idea to invoke generic callbacks with a lock taken, because it can cause AB-BA deadlocks in the long run. The only thing that requires care is that the cursor may disappear as soon as the mutex is released, so you need an extra cursor_get/cursor_put pair. Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com> Message-id: 20180720063109.4631-3-pbonzini@redhat.com [ kraxel: fix dpy_cursor_define() call ] Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Diffstat (limited to 'qobject/json-parser-int.h')
0 files changed, 0 insertions, 0 deletions