On Wed, Oct 14, 2015 at 10:52:00AM +0100, Chris Wilson wrote: > diff --git a/drivers/gpu/drm/drm_prime.c b/drivers/gpu/drm/drm_prime.c > index 27aa7183b20b..1bdd69e1ef43 100644 > --- a/drivers/gpu/drm/drm_prime.c > +++ b/drivers/gpu/drm/drm_prime.c > @@ -564,26 +564,26 @@ int drm_gem_prime_fd_to_handle(struct drm_device *dev, > uint32_t *handle)
> /* never seen this one, need to import */ > - mutex_lock(&dev->object_name_lock); I found the use of object_name_lock here confusing. I couldn't see what it was meant to protect (it is supposed to just lock access to the global flink name assigned to the object, and I'm sure what value it is adding to drm_gem_handle_create_tail() either). Anyway the handle here is protected against concurrent creation/destruction by the prime.lock, so it looked safe enough to move the object_name_lock back to to drm_gem_handle_create_tail(). -Chris -- Chris Wilson, Intel Open Source Technology Centre
