On Tue, 15 Mar 2022 at 15:58, Conor McCarthy cmccarthy@codeweavers.com wrote:
March 11, 2022 9:01 PM, "Henri Verbeet" hverbeet@gmail.com wrote:
KHR_timeline_semaphore requires raising the minimum required version of Vulkan-Headers from version 1.1.113 to version 1.2.131. That's fine, but requires an update of the README. Unfortunately this also introduces new failures in test_gpu_signal_fence() and test_fence_values() for me:
These failures occur in Mesa 20.1.10 but not 21.3.2 (I haven't checked which version eliminates them completely). Without the patches we have the "Failed to acquire Vulkan semaphore" problem, but with the patch on older Mesa versions we lack support for signalling a lower fence value in a queue. I'm not sure which is the worse problem.
I tested this on Mesa 20.3.5 / Intel SKL (i.e., current Debian stable), and those failures are still there at least. Unfortunately it also hangs in test_queue_wait() (specifically, the wait_event(event, INFINITE) call on line 33346) on the configuration, which seems a bit more concerning. Would it be hard to write a sanity check to run at initialisation to check whether the timeline semaphore implementation is usable for us?