[PATCH 4/4] drm: fix fb refcount issue with atomic modesetting

[Daniel Vetter]

On Tue, May 31, 2016 at 12:56:23PM +0200, Daniel Vetter wrote:
> On Tue, May 31, 2016 at 12:17:23PM +0300, Tomi Valkeinen wrote:
> > After commit 027b3f8ba9277410c3191d72d1ed2c6146d8a668 ("drm/modes: stop
> > handling framebuffer special") extra fb refs are left around when doing
> > atomic modesetting.
> > 
> > The problem is that the new drm_property_change_valid_get() does not
> > return anything in the '**ref' parameter, which causes
> > 'drm_property_change_valid_put' never to be called.
> > 
> > For some reason this doesn't cause problems with legacy API.
> > 
> > Also, previously the code only set the 'ref' variable for fbs, with this
> > patch the 'ref' is set for all objects.
> > 
> > So this is a bit of an RFC, as I don't understand all what's going on
> > here.
> > 
> > Signed-off-by: Tomi Valkeinen <tomi.valkeinen at ti.com>
> 
> On patches 1,2&4:
> 
> Reviewed-by: Daniel Vetter <daniel.vetter at ffwll.ch>
> 
> Imo everything but the sti patche should also grow a cc: stable, plus an
> appropriate Fixes: line for this one here.

Correction: Patch 4 seems in 4.7 only, so doens't need the cc: stable. But
patches 1&3 need it I think.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch