[PATCH] drm/amd/display - Don't leak memory when updating streams

Michel Dänzer michel at daenzer.net
Mon Jan 28 15:14:29 UTC 2019


On 2019-01-28 3:00 p.m., Nicholas Kazlauskas wrote:
> [Why]
> The flip and full structures were allocated but never freed.
> 
> [How]
> Free them at the end of the function. There's a small behavioral
> change here with the function returning early if the allocation fails
> but we wouldn't should be doing anything in that case anyway.
> 
> Fixes: c00e0cc0fdc0 ("drm/amd/display: Call into DC once per multiplane flip")
> Fixes: ea39594e0855 ("drm/amd/display: Perform plane updates only when needed")
> 
> Cc: Michel Dänzer <michel at daenzer.net>
> Cc: Leo Li <sunpeng.li at amd.com>
> Cc: Harry Wentland <Harry.Wentland at amd.com>
> Signed-off-by: Nicholas Kazlauskas <nicholas.kazlauskas at amd.com>

Tested-by: Michel Dänzer <michel.daenzer at amd.com>

Thanks Nicholas!


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer


More information about the amd-gfx mailing list