* [PATCH] drm/bridge: tc358767: fix EDID memory leak
@ 2020-08-26 13:40 ` Tomi Valkeinen
2020-08-26 15:54 ` Andrzej Hajda
2020-08-26 23:23 ` Laurent Pinchart
0 siblings, 2 replies; 4+ messages in thread
From: Tomi Valkeinen @ 2020-08-26 13:40 UTC (permalink / raw)
To: Andrzej Hajda, Neil Armstrong, Laurent Pinchart, Jonas Karlman,
Jernej Skrabec, dri-devel
Cc: Tomi Valkeinen
The current EDID allocated with drm_get_edid() is freed when the driver
gets a new EDID, but it is not freed when the driver is removed, causing
a leak.
Free the EDID (if any) on driver remove.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
---
drivers/gpu/drm/bridge/tc358767.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c
index c2777b226c75..dbb18a86beaf 100644
--- a/drivers/gpu/drm/bridge/tc358767.c
+++ b/drivers/gpu/drm/bridge/tc358767.c
@@ -1695,6 +1695,8 @@ static int tc_remove(struct i2c_client *client)
drm_bridge_remove(&tc->bridge);
drm_dp_aux_unregister(&tc->aux);
+ kfree(tc->edid);
+
return 0;
}
--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] drm/bridge: tc358767: fix EDID memory leak
2020-08-26 13:40 ` [PATCH] drm/bridge: tc358767: fix EDID memory leak Tomi Valkeinen
@ 2020-08-26 15:54 ` Andrzej Hajda
2020-08-26 23:23 ` Laurent Pinchart
1 sibling, 0 replies; 4+ messages in thread
From: Andrzej Hajda @ 2020-08-26 15:54 UTC (permalink / raw)
To: Tomi Valkeinen, Neil Armstrong, Laurent Pinchart, Jonas Karlman,
Jernej Skrabec, dri-devel
On 26.08.2020 15:40, Tomi Valkeinen wrote:
> The current EDID allocated with drm_get_edid() is freed when the driver
> gets a new EDID, but it is not freed when the driver is removed, causing
> a leak.
>
> Free the EDID (if any) on driver remove.
>
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Reviewed-by: Andrzej Hajda <a.hajda@samsung.com>
Regards
Andrzej
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] drm/bridge: tc358767: fix EDID memory leak
2020-08-26 13:40 ` [PATCH] drm/bridge: tc358767: fix EDID memory leak Tomi Valkeinen
2020-08-26 15:54 ` Andrzej Hajda
@ 2020-08-26 23:23 ` Laurent Pinchart
2020-08-27 5:18 ` Tomi Valkeinen
1 sibling, 1 reply; 4+ messages in thread
From: Laurent Pinchart @ 2020-08-26 23:23 UTC (permalink / raw)
To: Tomi Valkeinen
Cc: Andrzej Hajda, Jernej Skrabec, Jonas Karlman, dri-devel, Neil Armstrong
Hi Tomi,
Thank you for the patch.
On Wed, Aug 26, 2020 at 04:40:17PM +0300, Tomi Valkeinen wrote:
> The current EDID allocated with drm_get_edid() is freed when the driver
> gets a new EDID, but it is not freed when the driver is removed, causing
> a leak.
>
> Free the EDID (if any) on driver remove.
>
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
> ---
> drivers/gpu/drm/bridge/tc358767.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c
> index c2777b226c75..dbb18a86beaf 100644
> --- a/drivers/gpu/drm/bridge/tc358767.c
> +++ b/drivers/gpu/drm/bridge/tc358767.c
> @@ -1695,6 +1695,8 @@ static int tc_remove(struct i2c_client *client)
> drm_bridge_remove(&tc->bridge);
> drm_dp_aux_unregister(&tc->aux);
>
> + kfree(tc->edid);
> +
tc->edid is gone in drm-misc-next, problem solved already :-)
> return 0;
> }
--
Regards,
Laurent Pinchart
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] drm/bridge: tc358767: fix EDID memory leak
2020-08-26 23:23 ` Laurent Pinchart
@ 2020-08-27 5:18 ` Tomi Valkeinen
0 siblings, 0 replies; 4+ messages in thread
From: Tomi Valkeinen @ 2020-08-27 5:18 UTC (permalink / raw)
To: Laurent Pinchart
Cc: Andrzej Hajda, Jernej Skrabec, Jonas Karlman, dri-devel, Neil Armstrong
On 27/08/2020 02:23, Laurent Pinchart wrote:
> Hi Tomi,
>
> Thank you for the patch.
>
> On Wed, Aug 26, 2020 at 04:40:17PM +0300, Tomi Valkeinen wrote:
>> The current EDID allocated with drm_get_edid() is freed when the driver
>> gets a new EDID, but it is not freed when the driver is removed, causing
>> a leak.
>>
>> Free the EDID (if any) on driver remove.
>>
>> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
>> ---
>> drivers/gpu/drm/bridge/tc358767.c | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c
>> index c2777b226c75..dbb18a86beaf 100644
>> --- a/drivers/gpu/drm/bridge/tc358767.c
>> +++ b/drivers/gpu/drm/bridge/tc358767.c
>> @@ -1695,6 +1695,8 @@ static int tc_remove(struct i2c_client *client)
>> drm_bridge_remove(&tc->bridge);
>> drm_dp_aux_unregister(&tc->aux);
>>
>> + kfree(tc->edid);
>> +
>
> tc->edid is gone in drm-misc-next, problem solved already :-)
Well, indeed. Should have looked at the latest... =). The leak only happens on driver remove, so I
don't think it's worth sending for stable kernels.
Tomi
--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-08-27 5:18 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <CGME20200826134049eucas1p2de71d51b6d2fa7bcd1c29951553f5e7a@eucas1p2.samsung.com>
2020-08-26 13:40 ` [PATCH] drm/bridge: tc358767: fix EDID memory leak Tomi Valkeinen
2020-08-26 15:54 ` Andrzej Hajda
2020-08-26 23:23 ` Laurent Pinchart
2020-08-27 5:18 ` Tomi Valkeinen
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.