On Thu, Jul 09, 2015 at 11:32:39PM +0200, Daniel Vetter wrote: > Looking up an obj, immediate dropping the acquired reference and then > continuing to use it isn't how this is supposed to work. Fix this by > holding a reference for the entire function. > > While at it stop grabbing dev->struct_mutex, it doesn't protect > anything here. > > Signed-off-by: Daniel Vetter <[email protected]> > --- > drivers/gpu/drm/mgag200/mgag200_cursor.c | 22 ++++++++++------------ > 1 file changed, 10 insertions(+), 12 deletions(-)
Reviewed-by: Thierry Reding <[email protected]>
signature.asc
Description: PGP signature
_______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
