On 31.07.25 19:27, Harry Wentland wrote: > On 2025-07-30 04:09, Michel Dänzer wrote: >> From: Michel Dänzer <mdaen...@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.h...@kde.org> >> Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/3034 >> Signed-off-by: Michel Dänzer <mdaen...@redhat.com> > > Reviewed-by: Harry Wentland <harry.wentl...@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
