[PATCH 02/18] drm/cma-helper: Fix locking in drm_fb_cma_debugfs_show
Thierry Reding
thierry.reding at gmail.com
Mon Aug 10 03:41:02 PDT 2015
On Thu, Jul 09, 2015 at 11:32:34PM +0200, Daniel Vetter wrote:
> This function takes two locks, both of them the wrong ones. This
> wasn't an oversight from my fb locking rework since both patches
> landed in parallel. We really only need fb_lock when walking that
> list, since everything we can reach from that is refcounted properly
> already.
>
> Cc: Rob Clark <robdclark at gmail.com>
> Cc: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> Signed-off-by: Daniel Vetter <daniel.vetter at intel.com>
> ---
> drivers/gpu/drm/drm_fb_cma_helper.c | 16 ++--------------
> drivers/gpu/drm/drm_gem_cma_helper.c | 2 --
> 2 files changed, 2 insertions(+), 16 deletions(-)
Reviewed-by: Thierry Reding <treding at nvidia.com>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150810/98267768/attachment.sig>
More information about the dri-devel
mailing list