On Fri, May 15, 2020 at 10:51:16AM +0100, Emil Velikov wrote: > From: Emil Velikov <[email protected]> > > Spelling out _unlocked for each and every driver is a annoying. > Especially if we consider how many drivers, do not know (or need to) > about the horror stories involving struct_mutex. > > Just drop the suffix. It makes the API cleaner. > > Done via the following script: > > __from=drm_gem_object_put_unlocked > __to=drm_gem_object_put > for __file in $(git grep --name-only $__from); do > sed -i "s/$__from/$__to/g" $__file; > done > > v2: > - Rebase > > Cc: Rodrigo Siqueira <[email protected]> > Cc: Haneen Mohammed <[email protected]> > Cc: Daniel Vetter <[email protected]> > Signed-off-by: Emil Velikov <[email protected]> > Acked-by: Sam Ravnborg <[email protected]> (v1)
Reviewed-by: Daniel Vetter <[email protected]> > --- > drivers/gpu/drm/vkms/vkms_gem.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/vkms/vkms_gem.c b/drivers/gpu/drm/vkms/vkms_gem.c > index c541fec57566..a017fc59905e 100644 > --- a/drivers/gpu/drm/vkms/vkms_gem.c > +++ b/drivers/gpu/drm/vkms/vkms_gem.c > @@ -141,7 +141,7 @@ int vkms_dumb_create(struct drm_file *file, struct > drm_device *dev, > args->size = gem_obj->size; > args->pitch = pitch; > > - drm_gem_object_put_unlocked(gem_obj); > + drm_gem_object_put(gem_obj); > > DRM_DEBUG_DRIVER("Created object of size %lld\n", size); > > -- > 2.25.1 > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
