D23105: WIP: [platforms/x11] Cleanup GLX backend, revise compositing

romangg added a comment.


  In D23105#514201 <https://phabricator.kde.org/D23105#514201>, @fredrik wrote:
  
  > In D23105#512099 <https://phabricator.kde.org/D23105#512099>, @romangg wrote:
  >
  > > In D23105#511957 <https://phabricator.kde.org/D23105#511957>, @fredrik wrote:
  > >
  > > > NVIDIA doesn't support the OML extensions. They can't be implemented efficiently on their hardware IIRC.
  > > >  [...]
  > >
  > >
  > > That's good to know. Thanks! I believe we can let in some of these extensions again without increasing the complexity too much as long as the SGI ones is ignored and we have no manual control of vsync. The complexity in the old code came mostly from that.
  >
  >
  > That would leave the user without a way to override the driver's default vsync setting.
  
  
  I want KWin to run v-synced always. I don't think it makes sense to make this configurable by the user. It's the default in our Wayland session and on X11 the compositor should always run v-synced as well. Or is there a valid use case for running it async in your opinion?
  
  In D23105#514203 <https://phabricator.kde.org/D23105#514203>, @fredrik wrote:
  
  > By the way, you are partially duplicating work already done in this branch:
  >  https://cgit.kde.org/kwin.git/log/?h=fredrik/swap-event-wip2
  
  
  That's good to know.

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D23105

To: romangg, #kwin, fredrik
Cc: nicolasfella, alexeymin, kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, mart