[vlc-devel] [PATCH 3/3] decoder: fix occasional deadlock on seek

[Thomas Guillem]

This commit fixes the following deadlock:

The input thread (from input_DecoderWait()) reads p_owner->b_idle, and wait on
the wait_acknowledge condition variable because p_owner->b_idle is false (and
because p_owner->b_has_data is false too). The decoder thread sets
p_owner->b_idle to true because the fifo is empty and wait on the fifo
condition variable. Both threads are waiting each others, hence the deadlock.

To fix this issue, signal the input thread from the decoder thread after
setting p_owner->b_idle to true.
---
 src/input/decoder.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/input/decoder.c b/src/input/decoder.c
index c739402ac5..e71cbc0302 100644
--- a/src/input/decoder.c
+++ b/src/input/decoder.c
@@ -1646,6 +1646,7 @@ static void *DecoderThread( void *p_data )
         if( p_owner->paused && p_owner->frames_countdown == 0 )
         {   /* Wait for resumption from pause */
             p_owner->b_idle = true;
+            vlc_cond_signal( &p_owner->wait_acknowledge );
             vlc_cond_wait( &p_owner->fifo.wait, &p_owner->lock );
             p_owner->b_idle = false;
             continue;
@@ -1660,6 +1661,7 @@ static void *DecoderThread( void *p_data )
             if( likely(!p_owner->b_draining) )
             {   /* Wait for a block to decode (or a request to drain) */
                 p_owner->b_idle = true;
+                vlc_cond_signal( &p_owner->wait_acknowledge );
                 vlc_cond_wait( &p_owner->fifo.wait, &p_owner->lock );
                 p_owner->b_idle = false;
                 continue;
-- 
2.11.0