Skip to content

swrast: Copy entire buffer ignoring damage regions

Lucas Fryzek requested to merge Hazematman/mesa:lfryzek/swrast_swap_all into main

This MR reverts effectively reverts some of the changes in MR !27703 (merged) as the optimizations done are not strictly correct. The damage region is supposed to only be a hint to the compositor and the driver still needs to swap the entire buffer. This change ensures that the entire buffer is swapped and the application provided damage region is passed to the compositor.

This addresses the comment chain in #11784 #11784 (comment 2545090)

This MR aims to correct the behavior, by undoing in the optimization. In the comment thread @daenzer suggested some validity checking that could be done as a different optimization. Additionally in the future, swrast could instead render to shared memory with the wm/compositor.

CC: @zmike

Merge request reports