[PATCH] drm/amd/display: Add primary plane to commits for correct VRR handling

Harry Wentland harry.wentland at amd.com
Tue Aug 5 16:29:23 UTC 2025



On 2025-08-04 04:03, Michel Dänzer wrote:
> On 31.07.25 19:27, Harry Wentland wrote:
>> On 2025-07-30 04:09, Michel Dänzer wrote:
>>> From: Michel Dänzer <mdaenzer at redhat.com>
>>>
>>> amdgpu_dm_commit_planes calls update_freesync_state_on_stream only for
>>> the primary plane. If a commit affects a CRTC but not its primary plane,
>>> it would previously not trigger a refresh cycle or affect LFC, violating
>>> current UAPI semantics.
>>>
>>> Fixes e.g. atomic commits affecting only the cursor plane being limited
>>> to the minimum refresh rate.
>>>
>>> Don't do this for the legacy cursor ioctls though, it would break the
>>> UAPI semantics for those.
>>>
>>> Suggested-by: Xaver Hugl <xaver.hugl at kde.org>
>>> Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/3034
>>> Signed-off-by: Michel Dänzer <mdaenzer at redhat.com>
>>
>> Reviewed-by: Harry Wentland <harry.wentland at amd.com>
> 
> Thanks Harry.
> 
> 
> It occurred to me that Cc: stable might be appropriate for this fix, what do you think?
> 

Agreed.

Harry

> 



More information about the amd-gfx mailing list