Hi Nicholas, thanks for the patches. On 2018-09-11 6:18 p.m., Nicholas Kazlauskas wrote: > These patches are part of a proposed new interface for supporting variable refresh rate via DRM properties. > > https://patchwork.freedesktop.org/series/49486/ > > When notified of a window that is FreeSync capable via X these patches help track when the window is fullscreen to manage the variable_refresh property on the CRTC. I'm afraid the Xorg driver support will have to be more or less redone from scratch for upstreaming: Whether or not a client wants variable refresh rate enabled can be tracked via the window property mechanism supported by the core X11 protocol, no need for a protocol extension. That should also allow simpler tracking of when variable refresh rate can actually be enabled: It can be enabled while a window is flipping, and its corresponding property allows it. This should be straightforward with the Present extension, because that also explicitly marks the end of a window flipping (via an "unflip"). DRI2 is trickier; it's probably okay not to support variable refresh rate with that, at least initially. I can look into this after the upcoming Xorg driver 18.1 releases. Or I can give guidance if one of you wants to look into it. -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer
