diff options
| author | Paolo Bonzini | 2018-07-20 08:31:09 +0200 |
|---|---|---|
| committer | Gerd Hoffmann | 2018-08-24 08:40:11 +0200 |
| commit | 006bf4641b553565990a6b18e0cff36972a36773 (patch) | |
| tree | 7cb7434fe9eeeaa4cddec35da1f1c6033014a05b /ui/sdl2.c | |
| parent | spice-display: access ptr_x/ptr_y under Mutex (diff) | |
| download | qemu-006bf4641b553565990a6b18e0cff36972a36773.tar.gz qemu-006bf4641b553565990a6b18e0cff36972a36773.tar.xz 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 'ui/sdl2.c')
0 files changed, 0 insertions, 0 deletions