[PATCH 07/15] drm/mode: reduce scope of fb_lock in framebuffer init

Daniel Vetter daniel at ffwll.ch
Thu Apr 21 08:54:29 UTC 2016


On Fri, Apr 15, 2016 at 03:10:38PM +1000, Dave Airlie wrote:
> From: Dave Airlie <airlied at redhat.com>
> 
> We don't need to hold the fb lock around the initialisation,
> only around the list manipulaton.
> 
> So do the lock hold only around the register for now.
> Signed-off-by: Dave Airlie <airlied at redhat.com>

This needs a bit more explanation added:

"Previously fb refcounting, and especially the weak reference
(kref_get_unless_zero) used in fb lookups have been protected by fb_lock.
But with the refactoring to share refcounting in the drm_mode_object base
class that switched to being protected by idr_mutex, which means fb_lock
critical sections can be reduced."

I also double-checked that we don't have any outdated comments that point
at the wrong lock, and didn't find any.

With the commit message augmented:

Reviewed-by: Daniel Vetter <daniel.vetter at ffwll.ch>

> ---
>  drivers/gpu/drm/drm_crtc.c | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c
> index 0d75517..1863879 100644
> --- a/drivers/gpu/drm/drm_crtc.c
> +++ b/drivers/gpu/drm/drm_crtc.c
> @@ -458,21 +458,22 @@ int drm_framebuffer_init(struct drm_device *dev, struct drm_framebuffer *fb,
>  {
>  	int ret;
>  
> -	mutex_lock(&dev->mode_config.fb_lock);
>  	INIT_LIST_HEAD(&fb->filp_head);
>  	fb->dev = dev;
>  	fb->funcs = funcs;
>  
>  	ret = drm_mode_object_get_reg(dev, &fb->base, DRM_MODE_OBJECT_FB,
> -				      true, drm_framebuffer_free);
> +				      false, drm_framebuffer_free);
>  	if (ret)
>  		goto out;
>  
> +	mutex_lock(&dev->mode_config.fb_lock);
>  	dev->mode_config.num_fb++;
>  	list_add(&fb->head, &dev->mode_config.fb_list);
> -out:
> -	mutex_unlock(&dev->mode_config.fb_lock);
>  
> +	drm_mode_object_register(dev, &fb->base);
> +	mutex_unlock(&dev->mode_config.fb_lock);
> +out:
>  	return ret;
>  }
>  EXPORT_SYMBOL(drm_framebuffer_init);
> -- 
> 2.5.5
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch


More information about the dri-devel mailing list