[platforms/x11/standalone] Remove SGI swap control and video sync
ClosedPublic

Authored by romangg on Aug 27 2019, 8:01 PM.

Details

Summary

Current hardware should suppport either GLX_EXT_swap_control or
GLX_MESA_swap_control. To simplify code remove the usage of SGI extensions.

Test Plan

kwin_x11 tested on i915.

Diff Detail

Repository
R108 KWin
Branch
rmSGISyncSupport
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 16826
Build 16844: arc lint + arc unit
romangg created this revision.Aug 27 2019, 8:01 PM
Restricted Application added a project: KWin. · View Herald TranscriptAug 27 2019, 8:01 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
romangg requested review of this revision.Aug 27 2019, 8:01 PM
davidedmundson added inline comments.
plugins/platforms/x11/standalone/glxbackend.cpp
233

I don't understand this change in the context of this patch

plugins/platforms/x11/standalone/glxbackend.cpp
233

Edit: just seen you're removing it in another patch, so not really an issue

romangg added inline comments.Aug 27 2019, 11:27 PM
plugins/platforms/x11/standalone/glxbackend.cpp
233

Yea, so the idea why I changed it is with double buffering (normally the default case) it blocks for retrace. The swap profiler code did indicate this. But this change should have gone in the previous commit where the swap profiler code was removed.

In the end it gets removed anyway so let's ignore it for now.

zzag accepted this revision.Aug 28 2019, 5:16 PM
This revision is now accepted and ready to land.Aug 28 2019, 5:16 PM
romangg updated this revision to Diff 66533.Sep 20 2019, 10:44 AM

Rebase on master.

romangg updated this revision to Diff 68577.Oct 22 2019, 11:06 PM

Rebase on master.

romangg updated this revision to Diff 69720.Nov 14 2019, 7:41 AM

Rebase on master.

This revision was automatically updated to reflect the committed changes.