[PATCH] drm/amd/display: Add primary plane to commits for correct VRR handling
Michel Dänzer
michel at daenzer.net
Mon Aug 4 08:03:58 UTC 2025
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?
--
Earthling Michel Dänzer \ GNOME / Xwayland / Mesa developer
https://redhat.com \ Libre software enthusiast
More information about the amd-gfx
mailing list