On 2019-10-11 12:26 p.m., Nicholas Kazlauskas wrote: > [Why] > We're leaking memory by not freeing the gamma used to calculate the > transfer function for legacy gamma. > > [How] > Release the gamma after we're done with it. > > Cc: Philip Yang <Philip.Yang@xxxxxxx> > Cc: Harry Wentland <harry.wentland@xxxxxxx> > Cc: Bhawanpreet Lakha <Bhawanpreet.Lakha@xxxxxxx> > Cc: Leo Li <sunpeng.li@xxxxxxx> > Signed-off-by: Nicholas Kazlauskas <nicholas.kazlauskas@xxxxxxx Reviewed-by: Leo Li <sunpeng.li@xxxxxxx> Thanks! > --- > drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_color.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_color.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_color.c > index b43bb7f90e4e..2233d293a707 100644 > --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_color.c > +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_color.c > @@ -210,6 +210,8 @@ static int __set_legacy_tf(struct dc_transfer_func *func, > res = mod_color_calculate_regamma_params(func, gamma, true, has_rom, > NULL); > > + dc_gamma_release(&gamma); > + > return res ? 0 : -ENOMEM; > } > > _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx
