linux-stable.git/drivers/gpu/drm/drm_lock.c, branch linux-3.10.y

drm: Reject DRI1 hw lock ioctl functions for kms drivers

2015-10-22T21:37:51+00:00

commit da168d81b44898404d281d5dbe70154ab5f117c1 upstream.

I've done some extensive history digging across libdrm, mesa and
xf86-video-{intel,nouveau,ati}. The only potential user of this with
kms drivers I could find was ttmtest, which once used drmGetLock
still. But that mistake was quickly fixed up. Even the intel xvmc
library (which otherwise was really good with using dri1 stuff in kms
mode) managed to never take the hw lock for dri2 (and hence kms).

Hence it should be save to unconditionally disallow this.

Cc: Peter Antoine 
Reviewed-by: Peter Antoine 
Signed-off-by: Daniel Vetter 
Signed-off-by: Greg Kroah-Hartman

UAPI: (Scripted) Convert #include "..." to #include in drivers/gpu/

2012-10-02T17:01:07+00:00

Convert #include "..." to #include  in drivers/gpu/.

Signed-off-by: David Howells 
Acked-by: Dave Airlie 
Acked-by: Arnd Bergmann 
Acked-by: Thomas Gleixner 
Acked-by: Paul E. McKenney 
Acked-by: Dave Jones

drm: ditch strange DRIVER_DMA_QUEUE only error bail-out

2012-07-20T02:50:47+00:00

Only one driver (i810) even sets that flag. Now the actual locking
code uncoditionally promotes lock->context to an unsigned int.

Closer inspection of the userspace reveals that the drm lock context
is defined as an unsigned int (at least on linux). I suspect we just
have a strange case of signedness confusion going on.

Tested on my i815, doesn't seem to break anything.

Signed-off-by: Daniel Vetter 
Signed-off-by: Dave Airlie

drm: Don't initialize local ret variable when not needed

2012-05-22T09:32:58+00:00

Signed-off-by: Laurent Pinchart 
Reviewed-by: Alex Deucher 
Signed-off-by: Dave Airlie

drm: add missing exports for i810 driver.

2011-12-22T19:09:01+00:00

Brown paper bag of danvet.

Acked-by: Daniel Vetter 
Signed-off-by: Dave Airlie

drm: readd drm_lock_free in drm_unlock

2010-09-26T03:35:49+00:00

I've accidently killed a little bit too much in

commit 1da3f87ebb7edb3e0b829ec4bbe5fb3d9d93986f
Author: Daniel Vetter 
Date:   Mon Aug 23 22:53:24 2010 +0200

    drm: kill kernel_context_switch callbacks

Note to self: Next time also test with AIGLX disabled.

Reported-and-Tested-by: Andy Furniss 
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=30374
Signed-off-by: Daniel Vetter 
Signed-off-by: Dave Airlie

drm: don't export dri1 locking functions

2010-08-29T23:39:00+00:00

Only used by ioctl, not by any in-tree drivers.

Signed-off-by: Daniel Vetter 
Signed-off-by: Dave Airlie

drm: kill dma_ready callbacks

2010-08-29T23:38:12+00:00

Not used by any driver. So drop it.

Signed-off-by: Daniel Vetter 
Signed-off-by: Dave Airlie

drm: kill kernel_context_switch callbacks

2010-08-29T23:37:56+00:00

Not used by any in-kernel driver. So drop it.

Signed-off-by: Daniel Vetter 
Signed-off-by: Dave Airlie

drm: fix regression in drm locking since BKL removal.

2010-08-26T23:10:28+00:00

This locking path needs proper auditing but probably too late for changes at this point for 2.6.36, so lets go with the quick fix, which is to drop the lock around schedule.

Reported-by: Andreas Schwab 
Signed-off-by: Dave Airlie