drm/i915: don't update the dri1 breadcrumb with modesetting
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Tue, 10 Dec 2013 12:20:59 +0000 (13:20 +0100)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 11 Dec 2013 13:25:58 +0000 (14:25 +0100)
commit6c719faca2aceca72f1bf5b1645c1734ed3e9447
treeeaabeca4f3923e4dcb3c473e3943dd7e9605a344
parentad071acb53110c8efd26ff1e5b5d57449b43833b
drm/i915: don't update the dri1 breadcrumb with modesetting

The update is horribly racy since it doesn't protect at all against
concurrent closing of the master fd. And it can't really since that
requires us to grab a mutex.

Instead of jumping through hoops and offloading this to a worker
thread just block this bit of code for the modesetting driver.

Note that the race is fairly easy to hit since we call the breadcrumb
function for any interrupt. So the vblank interrupt (which usually
keeps going for a bit) is enough. But even if we'd block this and only
update the breadcrumb for user interrupts from the CS we could hit
this race with kms/gem userspace: If a non-master is waiting somewhere
(and hence has interrupts enabled) and the master closes its fd
(probably due to crashing).

v2: Add a code comment to explain why fixing this for real isn't
really worth it. Also improve the commit message a bit.

v3: Fix the spelling in the comment.

Reported-by: Eugene Shatokhin <eugene.shatokhin@rosalab.ru>
Cc: Eugene Shatokhin <eugene.shatokhin@rosalab.ru>
Cc: stable@vger.kernel.org
Acked-by: Chris Wilson <chris@chris-wilson.co.uk>
Tested-by: Eugene Shatokhin <eugene.shatokhin@rosalab.ru>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/i915_dma.c