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 <[email protected]>
>>>
>>> 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 <[email protected]>
>>> Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/3034
>>> Signed-off-by: Michel Dänzer <[email protected]>
>>
>> Reviewed-by: Harry Wentland <[email protected]>
>
> Thanks Harry.
>
>
> It occurred to me that Cc: stable might be appropriate for this fix, what do
> you think?
>
Agreed.
Harry
>
