On Fri, Aug 23, 2013 at 11:46:02PM +0300, Dan Carpenter wrote: > There is a typo so deadlocks on error instead of unlocking. > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > > diff --git a/drivers/gpu/drm/drm_prime.c b/drivers/gpu/drm/drm_prime.c > index 7ae2bfc..276d470 100644 > --- a/drivers/gpu/drm/drm_prime.c > +++ b/drivers/gpu/drm/drm_prime.c > @@ -552,7 +552,7 @@ fail: > */ > drm_gem_handle_delete(file_priv, *handle); > out_unlock: > - mutex_lock(&dev->object_name_lock); > + mutex_unlock(&dev->object_name_lock); Duh. Unfortunately exercising error paths is pretty hard :( So thanks for catching this. Reviewed-by: Daniel Vetter <daniel.vetter@xxxxxxxx> -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
