<div dir="ltr"><span style="font-size:12.8px">I reread the mail by Pekka from the last mail chain and apparently have remembered it wrong. I thought he said that it's fine the way I did it. What I could do instead as he explained, is listen for the wl_display.sync callback after each buffer commit in xwayland-present.c.</span><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">That would mean at this point the Wayland server has processed the commit request and at some later point will eventually display the buffer on the screen (as long as Xwayland doesn't commit a new buffer before that).</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 30, 2017 at 4:34 AM, Michel Dänzer <span dir="ltr"><<a href="mailto:michel@daenzer.net" target="_blank">michel@daenzer.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 30/08/17 12:24 AM, Roman Gilg wrote:<br>
><br>
> There is a small change to the window mode in Present as well, that now<br>
> calls flip_executed at the end of processing the flip with the<br>
> calculated damage. Xwayland then commits the new buffer.<br>
<br>
</span>As discussed before, I'd prefer if Xwayland instead only called<br>
present_event_notify once it's received some kind of feedback from the<br>
Wayland compositor that it has processed the "flip". There's no such<br>
thing as an "immediate flip".<br>
<span class="HOEnZb"><font color="#888888"><br>
<br>
--<br>
Earthling Michel Dänzer               |               <a href="http://www.amd.com" rel="noreferrer" target="_blank">http://www.amd.com</a><br>
Libre software enthusiast             |             Mesa and X developer<br>
</font></span></blockquote></div><br></div></div>