drm/qxl: Call qxl_bo_unref outside atomic context
authorJeremy Cline <jcline@redhat.com>
Fri, 1 Jun 2018 20:05:32 +0000 (16:05 -0400)
committerGerd Hoffmann <kraxel@redhat.com>
Mon, 4 Jun 2018 07:31:39 +0000 (09:31 +0200)
commit889ad63d41eea20184b0483e7e585e5b20fb6cfe
tree3ba6dd6621a23c7f944c1e134ac7ab021da32d07
parentc32048d9e93a5ab925d745396c63e7b912147f0a
drm/qxl: Call qxl_bo_unref outside atomic context

"qxl_bo_unref" may sleep, but calling "qxl_release_map" causes
"preempt_disable()" to be called and "preempt_enable()" isn't called
until "qxl_release_unmap" is used. Move the call to "qxl_bo_unref" out
from in between the two to avoid sleeping from an atomic context.

This issue can be demonstrated on a kernel with CONFIG_LOCKDEP=y by
creating a VM using QXL, using a desktop environment using Xorg, then
moving the cursor on or off a window.

Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1571128
Fixes: 9428088c90b6 ("drm/qxl: reapply cursor after resetting primary")
Cc: stable@vger.kernel.org
Signed-off-by: Jeremy Cline <jcline@redhat.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20180601200532.13619-1-jcline@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
drivers/gpu/drm/qxl/qxl_display.c