21 Apr
2017
21 Apr
'17
9:43 p.m.
On Thu, 20 Apr 2017 17:42:06 +0200, Takashi Iwai wrote:
On Thu, 20 Apr 2017 16:01:47 +0200, Wischer, Timo (ADITG/SW1) wrote:
Hi everyone,
I am wondering about the implementation of the new thread-safety feature [1]. There are so many issues with deadlocks (e.g. [3]) which were already solved but also which are not yet solved.
Why do you not using a recursive mutex to avoid most of this deadlocks? Using PTHREAD_MUTEX_RECURSIVE as the pthread attribute [2].
It sounds like a good idea. Although the plugin should be written not to cause deadlock, it's better to avoid such a pain by allowing the recursive lock.
Care to test and submit the proper patch?
Never mind, I committed a quick fix to git repo now. Thanks for the suggestion!
Takashi