<div dir="ltr"><div dir="ltr">Hi Alexandros,<div><br></div><div>Sorry for a delay. I've finally got an end-to-end system to test it out. It took some time because Weston backend I wrote a while back needed serious rework to catch up with latest changes. </div><div><br></div><div>There's one thing that didn't work for me. In compositor you reject anything that isn't a DMA buffer and then in glrenderer you put an extra assertion. Why? All you do is use an EGL extension in order to import external fence_fd. There's no dmabuf dependency there. As long as the EGL implementation exposes <span style="color:rgb(0,0,0);white-space:pre-wrap">EGL_SYNC_NATIVE_FENCE_ANDROID extension this should "just work" (tm) for the GL renderer. </span><span style="color:rgb(0,0,0);white-space:pre-wrap">It certainly did for me. </span><span style="color:rgb(0,0,0);white-space:pre-wrap">CPU-based renderers can poll() to wait. </span></div><div><span style="color:rgb(0,0,0);white-space:pre-wrap"><br></span></div><div><span style="color:rgb(0,0,0);white-space:pre-wrap">The type of buffer used is an orthogonal problem. The </span><font color="#000000"><span style="white-space:pre-wrap">EGL_WL_bind_wayland_display extension takes care of GL clients' buffers in GL renderer, for anything else the renderer needs to know how to get pixels and use whatever means to put those pixels on screen.</span></font></div><div><span style="color:rgb(0,0,0);white-space:pre-wrap"><br></span></div><div><span style="color:rgb(0,0,0);white-space:pre-wrap">Cheers,</span></div><div><span style="color:rgb(0,0,0);white-space:pre-wrap">Tomek</span></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, 13 Nov 2018 at 09:33, Tomek Bury <<a href="mailto:tomek.bury@gmail.com">tomek.bury@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Thanks! </div><br><div class="gmail_quote"><div dir="ltr">On Tue, Nov 13, 2018 at 9:08 AM Alexandros Frantzis <<a href="mailto:alexandros.frantzis@collabora.com" target="_blank">alexandros.frantzis@collabora.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, Nov 12, 2018 at 12:39:58PM +0000, Tomek Bury wrote:<br>
> On Mon, Nov 12, 2018 at 11:15 AM Daniel Stone <<a href="mailto:daniel@fooishbar.org" target="_blank">daniel@fooishbar.org</a>> wrote:<br>
> <br>
> > On Fri, 9 Nov 2018 at 10:48, Pekka Paalanen <<a href="mailto:ppaalanen@gmail.com" target="_blank">ppaalanen@gmail.com</a>> wrote:<br>
> > > I can add that while pushing upstream, if there are no other changes<br>
> > > coming.<br>
> > ><br>
> > > Reviewed-by: Pekka Paalanen <<a href="mailto:pekka.paalanen@collabora.co.uk" target="_blank">pekka.paalanen@collabora.co.uk</a>><br>
> > ><br>
> > > You have ensured that the C files generated from this revision build<br>
> > > fine in Weston, right?<br>
> > ><br>
> > > David, Daniel, since your name is in the maintainers, can I have your<br>
> > > R-b, please?<br>
> ><br>
> > The protocol is:<br>
> > Reviewed-by: Daniel Stone <<a href="mailto:daniels@collabora.com" target="_blank">daniels@collabora.com</a>><br>
> ><br>
> > The Weston implementation looks pretty good so far, though there's no<br>
> > full implementation of release yet.<br>
> ><br>
> > Cheers,<br>
> > Daniel<br>
> > _______________________________________________<br>
> > wayland-devel mailing list<br>
> > <a href="mailto:wayland-devel@lists.freedesktop.org" target="_blank">wayland-devel@lists.freedesktop.org</a><br>
> > <a href="https://lists.freedesktop.org/mailman/listinfo/wayland-devel" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/wayland-devel</a><br>
> ><br>
> <br>
> HI Daniel,<br>
> <br>
> Where can I find the work-in-progress implementation? I'd like to try it<br>
> out with Broadcom driver which doesn't have implicit cross-process sync. I<br>
> can add the explicit sync protocol implementation on the driver side but<br>
> I'd need a reference to test it against.<br>
> <br>
> Cheers,<br>
> Tomek<br>
<br>
Hi Tomek,<br>
<br>
the WIP implementation can be found here [1]. I hope to push an update,<br>
including some zwp_buffer_release_v1 correctness fixes, in the following<br>
days.<br>
<br>
Thanks,<br>
Alexandros<br>
<br>
[1] <a href="https://gitlab.freedesktop.org/wayland/weston/merge_requests/32" rel="noreferrer" target="_blank">https://gitlab.freedesktop.org/wayland/weston/merge_requests/32</a><br>
</blockquote></div>
</blockquote></div>