drm/compositor: Only skip cursor planes, when the content didn't change

Co-authored-by: Christian Meissl <[email protected]>
Smithay · Nov 25, 2024 · 2032f0c · 2032f0c
1 parent fb5a899
commit 2032f0c
Showing 1 changed file with 10 additions and 10 deletions.
diff --git a/src/backend/drm/compositor/mod.rs b/src/backend/drm/compositor/mod.rs
@@ -2667,21 +2667,21 @@ where
             PrimaryPlaneElement::Element(primary_plane_scanout_element.unwrap())
         };
 
-        // if the update only contains a cursor update, skip it for vrr
+        // if the update only contains a cursor position update, skip it for vrr
         if self.vrr
             && allow_partial_update
             && next_frame_state.planes.iter().all(|(plane, state)| {
-                if self.planes.cursor.iter().any(|p| *plane == p.handle) {
-                    !state.skip
-                } else {
-                    state.skip
-                }
+                state.skip
+                    || (self.planes.cursor.iter().any(|p| *plane == p.handle)
+                        && state.buffer().map(|b| &b.fb)
+                            == previous_state.plane_buffer(*plane).map(|b| &b.fb))
             })
         {
-            for (plane, state) in next_frame_state.planes.iter_mut() {
-                if self.planes.cursor.iter().any(|p| *plane == p.handle) {
-                    state.skip = true;
-                }
+            for plane in self.planes.cursor.iter() {
+                let Some(state) = next_frame_state.plane_state_mut(plane.handle) else {
+                    continue;
+                };
+                state.skip = true;
             }
         }