D18570: EGLStream DRM Backend Initial Implementation

romangg accepted this revision.
romangg added a comment.
This revision is now accepted and ready to land.


  Without going through it in detail the overall structure looks really nice now. I'm really happy with the integration in the new version, in particular that our normal AMS code path is used for mode switching. Wait for another +1 by someone who can test it on a live system. But from my side it's good to go.

INLINE COMMENTS

> egl_stream_backend.cpp:472
> +    eglSwapBuffers(eglDisplay(), o.eglSurface);
> +    if (m_backend->present(o.buffer, o.output)) { 
> +        EGLAttrib acquireAttribs[] = {

Check if the condition is not fulfilled and if this is the case return early.

REPOSITORY
  R108 KWin

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

To: ekurzinger, davidedmundson, graesslin, #kwin, romangg
Cc: eszlari, abalaji, mati865, wchilders, ognarb, abrahams, schernikov, guilhermesi, fredrik, cfeck, rapiteanu, romangg, hein, nicolasfella, zzag, kwin, jraleigh, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, alexeymin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart