From 58554e14f9995fdafe2338a69a42d9192b96f712 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Fri, 26 Sep 2008 09:57:11 +0200 Subject: notify: protect notify->pending with the mutex There was a known deadlocking bug in the notify library: when the other thread set notify->pending after the according check in notify_wait(), the latter thread was deadlocked. Resolve this by synchronizing all accesses to notify->pending with the notify object's mutex. Since notify_signal_sync() was never used, we can remove it. As a consequence, we don't need notify_enter() and notify_leave() anymore; eliminate them, too. --- src/output_thread.c | 2 -- 1 file changed, 2 deletions(-) (limited to 'src/output_thread.c') diff --git a/src/output_thread.c b/src/output_thread.c index a92f0bafd..4bc5d8fcb 100644 --- a/src/output_thread.c +++ b/src/output_thread.c @@ -67,8 +67,6 @@ static void *audio_output_task(void *arg) { struct audio_output *ao = arg; - notify_enter(&ao->notify); - while (1) { switch (ao->command) { case AO_COMMAND_NONE: -- cgit v1.2.3