Commit 8cacc98a authored by Thomas Guillem's avatar Thomas Guillem

decoder: fix occasional deadlock on seek

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.

Since predictable scheduling behavior is not required, p_owner->lock doesn't
have to be held while signaling p_owner->wait_acknowledge.

Fixes #16100
parent da488a77
......@@ -1571,6 +1571,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_fifo_Wait( p_owner->p_fifo );
p_owner->b_idle = false;
continue;
......@@ -1585,6 +1586,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_fifo_Wait( p_owner->p_fifo );
p_owner->b_idle = false;
continue;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment