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 daniel.vetter@intel.com
drivers/gpu/drm/mgag200/mgag200_cursor.c | 22 ++++++++++------------ 1 file changed, 10 insertions(+), 12 deletions(-)
Reviewed-by: Thierry Reding treding@nvidia.com