Re: [PATCH] drm/amd/display: Fix memory leaks in S3 resume

2020-12-21 Thread Kazlauskas, Nicholas
Reviewed-by: Nicholas Kazlauskas Feel free to merge. Regards, Nicholas Kazlauskas On 2020-12-21 11:45 a.m., Deucher, Alexander wrote: [AMD Official Use Only - Internal Distribution Only] This leak is still present.  Can we get this applied? Acked-by: Alex Deucher -

Re: [PATCH] drm/amd/display: Fix memory leaks in S3 resume

2020-12-21 Thread Deucher, Alexander
[AMD Official Use Only - Internal Distribution Only] This leak is still present. Can we get this applied? Acked-by: Alex Deucher From: Wang, Chao-kai (Stylon) Sent: Tuesday, November 10, 2020 2:49 AM To: amd-gfx@lists.freedesktop.org Cc: Wang, Chao-kai (Stylon

Re: [PATCH] drm/amd/display: Fix memory leaks in S3 resume

2020-11-11 Thread Wang, Chao-kai (Stylon)
___ From: Deucher, Alexander Sent: November 11, 2020 12:55 AM To: Wentland, Harry ; Wang, Chao-kai (Stylon) ; amd-gfx@lists.freedesktop.org Cc: Kazlauskas, Nicholas Subject: Re: [PATCH] drm/amd/display: Fix memory leaks in S3 resume [AMD Official Use Only - Inter

Re: [PATCH] drm/amd/display: Fix memory leaks in S3 resume

2020-11-10 Thread Deucher, Alexander
: Kazlauskas, Nicholas Subject: Re: [PATCH] drm/amd/display: Fix memory leaks in S3 resume It's missing the "drm_connector_list_update" call which I assume is important. Stylon, can you review Lee Starnes's patch? Is the drm_connector_list_update call maybe not needed? Thanks, Harry

Re: [PATCH] drm/amd/display: Fix memory leaks in S3 resume

2020-11-10 Thread Harry Wentland
-- *From:* Wentland, Harry *Sent:* Tuesday, November 10, 2020 9:27 AM *To:* Wang, Chao-kai (Stylon) ; amd-gfx@lists.freedesktop.org *Cc:* Kazlauskas, Nicholas ; Deucher, Alexander *Subject:* Re: [PATCH] drm/amd/display: Fix memory leaks in S3 resume On 2020-11-10 2:49 a.m., Stylon Wang wrote: &

Re: [PATCH] drm/amd/display: Fix memory leaks in S3 resume

2020-11-10 Thread Deucher, Alexander
, Chao-kai (Stylon) ; amd-gfx@lists.freedesktop.org Cc: Kazlauskas, Nicholas ; Deucher, Alexander Subject: Re: [PATCH] drm/amd/display: Fix memory leaks in S3 resume On 2020-11-10 2:49 a.m., Stylon Wang wrote: > EDID parsing in S3 resume pushes new display modes > to probed_modes list but d

Re: [PATCH] drm/amd/display: Fix memory leaks in S3 resume

2020-11-10 Thread Harry Wentland
On 2020-11-10 2:49 a.m., Stylon Wang wrote: EDID parsing in S3 resume pushes new display modes to probed_modes list but doesn't consolidate to actual mode list. This creates a race condition when amdgpu_dm_connector_ddc_get_modes() re-initializes the list head without walking the list and results