Patch that "fixes" compositor-x11

Bill Spitzak spitzak at gmail.com
Tue Jun 5 18:10:10 PDT 2012


On 06/05/2012 02:09 PM, Kristian Høgsberg wrote:

> Argh, you're right, thanks.  But you said that just adding the count
> and returning that alone didn't fix it, right?  I wonder if we need a
>
>          xcb_flush(compositor->conn);
>
> in x11_output_repaint() after the eglSwapBuffers() call.  Maybe just
> before returning from
> x11_compositor_handle_event()... not sure.  If you only handle one
> event at the time in x11_compositor_handle_event(), it doesn't sound
> like a missing flush issue.  Maybe try to force
> x11_compositor_next_event() to always use xcb_poll_for_event(), that
> is, don't use xcb_poll_for_queued_event()?
>
> Kristian

I tried inserting xcb_flush at the suggested places and it did not fix 
the problem.

The print statement still appears to fix the problem. I confirmed that 
redirecting stdout to /dev/null makes the problem reappear.



More information about the wayland-devel mailing list