egl/wayland: Use roundtrips when awaiting buffer release
authorKai Chen <kai.chen@intel.com>
Mon, 7 Aug 2017 15:34:51 +0000 (08:34 -0700)
committerDaniel Stone <daniels@collabora.com>
Fri, 25 Aug 2017 08:57:03 +0000 (09:57 +0100)
commit151188d1e330a7a5f110bbc8251680121a1a84a6
tree9b086d35f63fd320e1727cb90fd480f130791147
parent4da6cf6c98aeca8c143ecb5e0ada5e1eee81a0f5
egl/wayland: Use roundtrips when awaiting buffer release

In get_back_bo, we use wl_display_dispatch_queue() to block and wait for
a buffer release event. However, not all Wayland compositors flush the
client socket on posting a buffer-release event, so by only blocking
client-side, we may block indefinitely, or at least need to wait for an
input event / frame completion to arrive for the compositor to flush.

We now use dispatch_queue as a first pass, but if our entire buffer pool
is exhausted, use a roundtrip (an immediately-triggered wl_callback) to
ensure that the compositor flushes out our release event immediately.

[daniels: Modified comment and commit message.]

Signed-off-by: Kai Chen <kai.chen@intel.com>
Reviewed-by: Daniel Stone <daniels@collabora.com>
CC: <mesa-stable@lists.freedesktop.org>
src/egl/drivers/dri2/platform_wayland.c