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>