drm: Remove utterly bogus preempt_disable() sections
authorThomas Gleixner <tglx@linutronix.de>
Sat, 23 Jul 2011 10:45:07 +0000 (12:45 +0200)
committerDave Airlie <airlied@redhat.com>
Mon, 14 Nov 2011 09:28:50 +0000 (09:28 +0000)
commitd53dab3ae1a3c6b438738c3792c98ac63f0061f4
treef4e78aae50cd45ae335dbfed81896c3d965628fe
parenta7c36fd8c5ee6dcca584137cb81aeefd785a0721
drm: Remove utterly bogus preempt_disable() sections

commit 27641c3f (drm/vblank: Add support for precise vblank
timestamping) adds preempt_disable()/enable() around a spin locked
section with the comments:

 * Disable preemption, so vblank_time_lock is held as short as
 * possible, even under a kernel with PREEMPT_RT patches.

/* Disable preemption while holding vblank_time_lock. Do
 * it explicitely to guard against PREEMPT_RT kernel.

Just that this has never been tested on a RT kernel which would have
granted that nonsense with a might_sleep() warning because
dev->vblank_time_lock is converted to a "sleeping" spinlock on RT.

So this is activly wrong on RT and superflous on mainline. Remove it.

Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Acked-by: Mario Kleiner <mario.kleiner@tuebingen.mpg.de>
Signed-off-by: Dave Airlie <airlied@redhat.com>
drivers/gpu/drm/drm_irq.c