Re: [Intel-gfx] [PATCH 3/3] dma-buf/selftests: Add locking selftests for sw_sync

2020-07-14 Thread Bas Nieuwenhuizen
Awsome, thanks for adding the tests! Got to say I'm not that familiar with the self-test framework idioms, but from my perspective patches 2 and 3 are Reviewed-by: Bas Nieuwenhuizen as well. On Tue, Jul 14, 2020 at 10:06 PM Chris Wilson wrote: > > While sw_sync is purely a debug facility for

[Intel-gfx] [PATCH 3/3] dma-buf/selftests: Add locking selftests for sw_sync

2020-07-14 Thread Chris Wilson
While sw_sync is purely a debug facility for userspace to create fences and timelines it can control, nevertheless it has some tricky locking semantics of its own. In particular, Bas Nieuwenhuize reported that we had reintroduced a deadlock if a signal callback attempted to destroy the fence. So le