]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commit
drm/nouveau: Dis/Enable vblank irqs during suspend/resume.
authorMario Kleiner <mario.kleiner.de@gmail.com>
Tue, 29 Jul 2014 00:36:44 +0000 (02:36 +0200)
committerBen Skeggs <bskeggs@redhat.com>
Sat, 9 Aug 2014 19:12:51 +0000 (05:12 +1000)
commit9cba5efab5a8145ae6c52ea273553f069c294482
tree50e8cf26a36ad9619201574bac162fde64abe429
parent95713d4a16ad46ce9dd8ccef45cc67af0f161ae2
drm/nouveau: Dis/Enable vblank irqs during suspend/resume.

Vblank irqs don't get disabled during suspend or driver
unload, which causes irq delivery after "suspend" or
driver unload, at least until the gpu is powered off.
This could race with drm_vblank_cleanup() in the case
of nouveau and cause a use-after-free bug if the driver
is unloaded.

More annoyingly during everyday use, at least on nv50
display engine (likely also others), vblank irqs are
off after a resume from suspend, but the drm doesn't
know this, so all vblank related functionality is dead
after a resume. E.g., all windowed OpenGL clients will
hang at swapbuffers time, as well as many fullscreen
clients in many cases. This makes suspend/resume useless
if one wants to use any OpenGL apps after the resume.

In Linux 3.16, drm_vblank_on() was added, complementing
the older drm_vblank_off()  to solve these problems
elegantly, so use those calls in nouveaus suspend/resume
code.

For kernels 3.8 - 3.15, we need to cherry-pick the
drm_vblank_on() patch to support this patch.

Signed-off-by: Mario Kleiner <mario.kleiner.de@gmail.com>
Cc: <stable@vger.kernel.org> #v3.16
Cc: <stable@vger.kernel.org> #v3.8+: f275228: drm: Add drm_vblank_on()
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
drivers/gpu/drm/nouveau/nouveau_display.c