On Sun, Jul 02, 2023 at 08:12:15AM -0500, Mario Limonciello wrote:
> Hi,
>
> A regression [1] was reported on suspend for AMD Stoney on 6.3.10.
> It bisected down to:
>
> 1ca399f127e0 ("drm/amd/display: Add wrapper to call planes and stream
> update")
>
> This was requested by me to fix a PSR problem [2], which isn't used on
> Stoney.
>
> This was also backported into 6.1.36 (as e538342002cb) and 5.15.119 (as
> 3c1aa91b37f9).
>
> It's fixed on 6.3.y by cherry-picking:
>
> 32953485c558 ("drm/amd/display: Do not update DRR while BW optimizations
> pending")
>
> It's fixed in 6.1.y by cherry-picking:
>
> 3442f4e0e555 ("drm/amd/display: Remove optimization for VRR updates")
> 32953485c558 ("drm/amd/display: Do not update DRR while BW optimizations
> pending")
All now queued up, thanks.
> On 5.15.y it's not a reasonable backport to take the fix to stable because
> there is a lot of missing Freesync code. Instead it's better to revert the
> patch series that introduced it to 5.15.y because PSR-SU isn't even
> introduced until later kernels anyway.
>
> 5a24be76af79 ("drm/amd/display: fix the system hang while disable PSR")
> 3c1aa91b37f9 ("drm/amd/display: Add wrapper to call planes and stream
> update")
> eea850c025b5 ("drm/amd/display: Use dc_update_planes_and_stream")
> 97ca308925a5 ("drm/amd/display: Add minimal pipe split transition state")
Can you please send these reverts with the discussion as to why they are
requried?
thanks,
greg k-h
