loader/dri3: Improve dri3 thread-safety
authorThomas Hellstrom <thellstrom@vmware.com>
Tue, 19 Sep 2017 17:41:22 +0000 (19:41 +0200)
committerThomas Hellstrom <thellstrom@vmware.com>
Mon, 13 Nov 2017 11:43:39 +0000 (12:43 +0100)
commit54a58b2856377e18ea6a42706bea0304a8d7845e
tree4a3905f4987d24e5bf9ab750361783898004420a
parent2b72ab58e5f63f82fe64cfc1beccddb2f4844405
loader/dri3: Improve dri3 thread-safety

It turned out that with recent changes that call into dri3 from glFinish(),
it appears like different thread end up waiting for X events simultaneously,
causing deadlocks since they steal events from eachoter and update the dri3
counters behind eachothers backs.

This patch intends to improve on that. It allows at most one thread at a
time to wait on events for a single drawable. If another thread intends to
do the same, it's put to sleep until the first thread finishes waiting, and
then it rechecks counters and optionally retries the waiting. Threads that
poll for X events never pulls X events off the event queue if there are
other threads waiting for events on that drawable. Counters in the
dri3 drawable structure are protected by a mutex. Finally, the mutex we
introduce is never held while waiting for the X server to avoid
unnecessary stalls.

This does not make dri3 drawables completely thread-safe but at least it's a
first step.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102358
Fixes: d5ba75f8881 "st/dri2 Plumb the flush_swapbuffer functionality through to dri3"
Signed-off-by: Thomas Hellstrom <thellstrom@vmware.com>
Acked-by: Nicolai Hähnle <nicolai.haehnle@amd.com>
src/loader/loader_dri3_helper.c
src/loader/loader_dri3_helper.h