]> git.sesse.net Git - vlc/blobdiff - src/input/decoder.c
Cosmetic.
[vlc] / src / input / decoder.c
index db14ff5cb4f8a5357f575c9c7c35642f23a59f34..15428581e211fbef08f74ec2c5476635743b3228 100644 (file)
@@ -31,7 +31,7 @@
 #endif
 #include <assert.h>
 
-#include <vlc/vlc.h>
+#include <vlc_common.h>
 
 #include <vlc_block.h>
 #include <vlc_vout.h>
 #include "audio_output/aout_internal.h"
 #include "stream_output/stream_output.h"
 #include "input_internal.h"
+#include "clock.h"
+#include "decoder.h"
 
-static decoder_t * CreateDecoder( input_thread_t *, es_format_t *, int );
-static void        DeleteDecoder( decoder_t * );
+#include "../video_output/vout_control.h"
 
-static int         DecoderThread( decoder_t * );
-static int         DecoderDecode( decoder_t * p_dec, block_t *p_block );
+static decoder_t *CreateDecoder( input_thread_t *, es_format_t *, int, sout_instance_t *p_sout );
+static void       DeleteDecoder( decoder_t * );
+
+static void      *DecoderThread( vlc_object_t * );
+static int        DecoderProcess( decoder_t *, block_t * );
+static void       DecoderOutputChangePause( decoder_t *, bool b_paused, mtime_t i_date );
+static void       DecoderFlush( decoder_t * );
+static void       DecoderSignalBuffering( decoder_t *, bool );
+static void       DecoderFlushBuffering( decoder_t * );
+
+static void       DecoderUnsupportedCodec( decoder_t *, vlc_fourcc_t );
 
 /* Buffers allocation callbacks for the decoders */
 static aout_buffer_t *aout_new_buffer( decoder_t *, int );
@@ -67,19 +77,15 @@ static es_format_t null_es_format;
 
 struct decoder_owner_sys_t
 {
-    vlc_bool_t      b_own_thread;
-
     int64_t         i_preroll_end;
 
     input_thread_t  *p_input;
-
-    aout_instance_t *p_aout;
-    aout_input_t    *p_aout_input;
-
-    vout_thread_t   *p_vout;
+    input_clock_t   *p_clock;
+    int             i_last_rate;
 
     vout_thread_t   *p_spu_vout;
     int              i_spu_channel;
+    int64_t          i_spu_order;
 
     sout_instance_t         *p_sout;
     sout_packetizer_input_t *p_sout_input;
@@ -95,23 +101,66 @@ struct decoder_owner_sys_t
     /* fifo */
     block_fifo_t *p_fifo;
 
+    /* Lock for communication with decoder thread */
+    vlc_mutex_t lock;
+    vlc_cond_t  wait;
+
+    /* -- These variables need locking on write(only) -- */
+    aout_instance_t *p_aout;
+    aout_input_t    *p_aout_input;
+
+    vout_thread_t   *p_vout;
+
+    /* -- Theses variables need locking on read *and* write -- */
+    /* */
+    /* Pause */
+    bool b_paused;
+    struct
+    {
+        mtime_t i_date;
+        int     i_ignore;
+    } pause;
+
+    /* Buffering */
+    bool b_buffering;
+    struct
+    {
+        bool b_first;
+        bool b_full;
+        int  i_count;
+
+        picture_t     *p_picture;
+        picture_t     **pp_picture_next;
+
+        subpicture_t  *p_subpic;
+        subpicture_t  **pp_subpic_next;
+
+        aout_buffer_t *p_audio;
+        aout_buffer_t **pp_audio_next;
+    } buffer;
+
+    /* Flushing */
+    bool b_flushing;
+
     /* CC */
-    vlc_bool_t b_cc_supported;
-    vlc_mutex_t lock_cc;
-    vlc_bool_t pb_cc_present[4];
-    decoder_t *pp_cc[4];
+    struct
+    {
+        bool b_supported;
+        bool pb_present[4];
+        decoder_t *pp_decoder[4];
+    } cc;
+
+    /* Delay */
+    mtime_t i_ts_delay;
 };
 
-/* */
-static void DecoderUnsupportedCodec( decoder_t *p_dec, vlc_fourcc_t codec )
-{
-    msg_Err( p_dec, "no suitable decoder module for fourcc `%4.4s'.\n"
-             "VLC probably does not support this sound or video format.",
-             (char*)&codec );
-    intf_UserFatal( p_dec, VLC_FALSE, _("No suitable decoder module"), 
-                    _("VLC does not support the audio or video format \"%4.4s\". "
-                      "Unfortunately there is no way for you to fix this."), (char*)&codec );
-}
+#define DECODER_MAX_BUFFERING_COUNT (4)
+#define DECODER_MAX_BUFFERING_AUDIO_DURATION (AOUT_MAX_PREPARE_TIME)
+#define DECODER_MAX_BUFFERING_VIDEO_DURATION (1*CLOCK_FREQ)
+
+/*****************************************************************************
+ * Public functions
+ *****************************************************************************/
 
 /* decoder_GetInputAttachment:
  */
@@ -136,7 +185,27 @@ int decoder_GetInputAttachments( decoder_t *p_dec,
  */
 mtime_t decoder_GetDisplayDate( decoder_t *p_dec, mtime_t i_ts )
 {
-    return i_ts;
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+
+    vlc_mutex_lock( &p_owner->lock );
+    if( p_owner->b_buffering || p_owner->b_paused )
+        i_ts = 0;
+    vlc_mutex_unlock( &p_owner->lock );
+
+    if( !p_owner->p_clock || !i_ts )
+        return i_ts;
+
+    return input_clock_GetTS( p_owner->p_clock, NULL, p_owner->p_input->i_pts_delay, i_ts );
+}
+/* decoder_GetDisplayRate:
+ */
+int decoder_GetDisplayRate( decoder_t *p_dec )
+{
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+
+    if( !p_owner->p_clock )
+        return INPUT_RATE_DEFAULT;
+    return input_clock_GetRate( p_owner->p_clock );
 }
 
 /**
@@ -147,32 +216,34 @@ mtime_t decoder_GetDisplayDate( decoder_t *p_dec, mtime_t i_ts )
  * \return the spawned decoder object
  */
 decoder_t *input_DecoderNew( input_thread_t *p_input,
-                             es_format_t *fmt, vlc_bool_t b_force_decoder )
+                             es_format_t *fmt, input_clock_t *p_clock, sout_instance_t *p_sout  )
 {
-    decoder_t   *p_dec = NULL;
-    vlc_value_t val;
+    decoder_t *p_dec = NULL;
+    int i_priority;
 
+#ifdef ENABLE_SOUT
     /* If we are in sout mode, search for packetizer module */
-    if( p_input->p->p_sout && !b_force_decoder )
+    if( p_sout )
     {
         /* Create the decoder configuration structure */
-        p_dec = CreateDecoder( p_input, fmt, VLC_OBJECT_PACKETIZER );
+        p_dec = CreateDecoder( p_input, fmt, VLC_OBJECT_PACKETIZER, p_sout );
         if( p_dec == NULL )
         {
             msg_Err( p_input, "could not create packetizer" );
-            intf_UserFatal( p_input, VLC_FALSE, _("Streaming / Transcoding failed"),
+            intf_UserFatal( p_input, false, _("Streaming / Transcoding failed"),
                             _("VLC could not open the packetizer module.") );
             return NULL;
         }
     }
     else
+#endif
     {
         /* Create the decoder configuration structure */
-        p_dec = CreateDecoder( p_input, fmt, VLC_OBJECT_DECODER );
+        p_dec = CreateDecoder( p_input, fmt, VLC_OBJECT_DECODER, p_sout );
         if( p_dec == NULL )
         {
             msg_Err( p_input, "could not create decoder" );
-            intf_UserFatal( p_input, VLC_FALSE, _("Streaming / Transcoding failed"),
+            intf_UserFatal( p_input, false, _("Streaming / Transcoding failed"),
                             _("VLC could not open the decoder module.") );
             return NULL;
         }
@@ -187,36 +258,22 @@ decoder_t *input_DecoderNew( input_thread_t *p_input,
         return NULL;
     }
 
-    if( p_input->p->p_sout && p_input->p->input.b_can_pace_control &&
-        !b_force_decoder )
-    {
-        msg_Dbg( p_input, "stream out mode -> no decoder thread" );
-        p_dec->p_owner->b_own_thread = VLC_FALSE;
-    }
+    p_dec->p_owner->p_clock = p_clock;
+
+    if( fmt->i_cat == AUDIO_ES )
+        i_priority = VLC_THREAD_PRIORITY_AUDIO;
     else
-    {
-        var_Get( p_input, "minimize-threads", &val );
-        p_dec->p_owner->b_own_thread = !val.b_bool;
-    }
+        i_priority = VLC_THREAD_PRIORITY_VIDEO;
 
-    if( p_dec->p_owner->b_own_thread )
+    /* Spawn the decoder thread */
+    if( vlc_thread_create( p_dec, "decoder", DecoderThread,
+                           i_priority, false ) )
     {
-        int i_priority;
-        if( fmt->i_cat == AUDIO_ES )
-            i_priority = VLC_THREAD_PRIORITY_AUDIO;
-        else
-            i_priority = VLC_THREAD_PRIORITY_VIDEO;
-
-        /* Spawn the decoder thread */
-        if( vlc_thread_create( p_dec, "decoder", DecoderThread,
-                               i_priority, VLC_FALSE ) )
-        {
-            msg_Err( p_dec, "cannot spawn decoder thread" );
-            module_Unneed( p_dec, p_dec->p_module );
-            DeleteDecoder( p_dec );
-            vlc_object_release( p_dec );
-            return NULL;
-        }
+        msg_Err( p_dec, "cannot spawn decoder thread" );
+        module_unneed( p_dec, p_dec->p_module );
+        DeleteDecoder( p_dec );
+        vlc_object_release( p_dec );
+        return NULL;
     }
 
     return p_dec;
@@ -231,34 +288,34 @@ decoder_t *input_DecoderNew( input_thread_t *p_input,
  */
 void input_DecoderDelete( decoder_t *p_dec )
 {
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+
     vlc_object_kill( p_dec );
 
-    if( p_dec->p_owner->b_own_thread )
+    /* Make sure we aren't paused anymore */
+    vlc_mutex_lock( &p_owner->lock );
+    if( p_owner->b_paused || p_owner->b_buffering )
     {
-        /* Make sure the thread leaves the function by
-         * sending it an empty block. */
-        block_t *p_block = block_New( p_dec, 0 );
-        input_DecoderDecode( p_dec, p_block );
+        p_owner->b_paused = false;
+        p_owner->b_buffering = false;
+        vlc_cond_signal( &p_owner->wait );
+    }
+    vlc_mutex_unlock( &p_owner->lock );
 
-        vlc_thread_join( p_dec );
+    /* Make sure the thread leaves the function */
+    block_FifoWake( p_owner->p_fifo );
 
-        /* Don't module_Unneed() here because of the dll loader that wants
-         * close() in the same thread than open()/decode() */
-    }
-    else
-    {
-        /* Flush */
-        input_DecoderDecode( p_dec, NULL );
+    vlc_thread_join( p_dec );
 
-        module_Unneed( p_dec, p_dec->p_module );
-    }
+    /* Don't module_unneed() here because of the dll loader that wants
+     * close() in the same thread than open()/decode() */
 
     /* */
-    if( p_dec->p_owner->b_cc_supported )
+    if( p_dec->p_owner->cc.b_supported )
     {
         int i;
         for( i = 0; i < 4; i++ )
-            input_DecoderSetCcState( p_dec, VLC_FALSE, i );
+            input_DecoderSetCcState( p_dec, false, i );
     }
 
     /* Delete decoder configuration */
@@ -274,88 +331,56 @@ void input_DecoderDelete( decoder_t *p_dec )
  * \param p_dec the decoder object
  * \param p_block the data block
  */
-void input_DecoderDecode( decoder_t * p_dec, block_t *p_block )
+void input_DecoderDecode( decoder_t *p_dec, block_t *p_block )
 {
-    if( p_dec->p_owner->b_own_thread )
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+
+    if( p_owner->p_input->p->b_out_pace_control )
     {
-        if( p_dec->p_owner->p_input->p->b_out_pace_control )
-        {
-            /* FIXME !!!!! */
-            while( !p_dec->b_die && !p_dec->b_error &&
-                   block_FifoCount( p_dec->p_owner->p_fifo ) > 10 )
-            {
-                msleep( 1000 );
-            }
-        }
-        else if( block_FifoSize( p_dec->p_owner->p_fifo ) > 50000000 /* 50 MB */ )
+        /* FIXME !!!!! */
+        while( vlc_object_alive( p_dec ) &&
+               block_FifoCount( p_owner->p_fifo ) > 10 )
         {
-            /* FIXME: ideally we would check the time amount of data
-             * in the fifo instead of its size. */
-            msg_Warn( p_dec, "decoder/packetizer fifo full (data not "
-                      "consumed quickly enough), resetting fifo!" );
-            block_FifoEmpty( p_dec->p_owner->p_fifo );
+            msleep( 1000 );
         }
-
-        block_FifoPut( p_dec->p_owner->p_fifo, p_block );
     }
-    else
+    else if( block_FifoSize( p_owner->p_fifo ) > 50000000 /* 50 MB */ )
     {
-        if( p_dec->b_error || (p_block && p_block->i_buffer <= 0) )
-        {
-            if( p_block ) block_Release( p_block );
-        }
-        else
-        {
-            DecoderDecode( p_dec, p_block );
-        }
+        /* FIXME: ideally we would check the time amount of data
+         * in the fifo instead of its size. */
+        msg_Warn( p_dec, "decoder/packetizer fifo full (data not "
+                  "consumed quickly enough), resetting fifo!" );
+        block_FifoEmpty( p_owner->p_fifo );
     }
-}
-
-void input_DecoderDiscontinuity( decoder_t * p_dec, vlc_bool_t b_flush )
-{
-    block_t *p_null;
-
-    /* Empty the fifo */
-    if( p_dec->p_owner->b_own_thread && b_flush )
-        block_FifoEmpty( p_dec->p_owner->p_fifo );
 
-    /* Send a special block */
-    p_null = block_New( p_dec, 128 );
-    p_null->i_flags |= BLOCK_FLAG_DISCONTINUITY;
-    /* FIXME check for p_packetizer or b_packitized from es_format_t of input ? */
-    if( p_dec->p_owner->p_packetizer && b_flush )
-        p_null->i_flags |= BLOCK_FLAG_CORRUPTED;
-    memset( p_null->p_buffer, 0, p_null->i_buffer );
-
-    input_DecoderDecode( p_dec, p_null );
+    block_FifoPut( p_owner->p_fifo, p_block );
 }
 
-vlc_bool_t input_DecoderEmpty( decoder_t * p_dec )
+bool input_DecoderIsEmpty( decoder_t * p_dec )
 {
-    if( p_dec->p_owner->b_own_thread
-     && block_FifoCount( p_dec->p_owner->p_fifo ) > 0 )
-    {
-        return VLC_FALSE;
-    }
-    return VLC_TRUE;
+    assert( !p_dec->p_owner->b_buffering );
+
+    /* FIXME that's not really true */
+    return block_FifoCount( p_dec->p_owner->p_fifo ) <= 0;
 }
 
-void input_DecoderIsCcPresent( decoder_t *p_dec, vlc_bool_t pb_present[4] )
+void input_DecoderIsCcPresent( decoder_t *p_dec, bool pb_present[4] )
 {
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
     int i;
 
-    vlc_mutex_lock( &p_dec->p_owner->lock_cc );
+    vlc_mutex_lock( &p_owner->lock );
     for( i = 0; i < 4; i++ )
-        pb_present[i] =  p_dec->p_owner->pb_cc_present[i];
-    vlc_mutex_unlock( &p_dec->p_owner->lock_cc );
+        pb_present[i] =  p_owner->cc.pb_present[i];
+    vlc_mutex_unlock( &p_owner->lock );
 }
-int input_DecoderSetCcState( decoder_t *p_dec, vlc_bool_t b_decode, int i_channel )
+int input_DecoderSetCcState( decoder_t *p_dec, bool b_decode, int i_channel )
 {
     decoder_owner_sys_t *p_owner = p_dec->p_owner;
 
     //msg_Warn( p_dec, "input_DecoderSetCcState: %d @%d", b_decode, i_channel );
 
-    if( i_channel < 0 || i_channel >= 4 || !p_owner->pb_cc_present[i_channel] )
+    if( i_channel < 0 || i_channel >= 4 || !p_owner->cc.pb_present[i_channel] )
         return VLC_EGENERIC;
 
     if( b_decode )
@@ -370,11 +395,11 @@ int input_DecoderSetCcState( decoder_t *p_dec, vlc_bool_t b_decode, int i_channe
         es_format_t fmt;
 
         es_format_Init( &fmt, SPU_ES, fcc[i_channel] );
-        p_cc = CreateDecoder( p_owner->p_input, &fmt, VLC_OBJECT_DECODER );
+        p_cc = CreateDecoder( p_owner->p_input, &fmt, VLC_OBJECT_DECODER, p_owner->p_sout );
         if( !p_cc )
         {
             msg_Err( p_dec, "could not create decoder" );
-            intf_UserFatal( p_dec, VLC_FALSE, _("Streaming / Transcoding failed"),
+            intf_UserFatal( p_dec, false, _("Streaming / Transcoding failed"),
                             _("VLC could not open the decoder module.") );
             return VLC_EGENERIC;
         }
@@ -385,44 +410,169 @@ int input_DecoderSetCcState( decoder_t *p_dec, vlc_bool_t b_decode, int i_channe
             vlc_object_release( p_cc );
             return VLC_EGENERIC;
         }
+        p_cc->p_owner->p_clock = p_owner->p_clock;
 
-        vlc_mutex_lock( &p_owner->lock_cc );
-        p_dec->p_owner->pp_cc[i_channel] = p_cc;
-        vlc_mutex_unlock( &p_owner->lock_cc );
+        vlc_mutex_lock( &p_owner->lock );
+        p_owner->cc.pp_decoder[i_channel] = p_cc;
+        vlc_mutex_unlock( &p_owner->lock );
     }
     else
     {
         decoder_t *p_cc;
 
-        vlc_mutex_lock( &p_owner->lock_cc );
-        p_cc = p_dec->p_owner->pp_cc[i_channel];
-        p_dec->p_owner->pp_cc[i_channel] = NULL;
-        vlc_mutex_unlock( &p_dec->p_owner->lock_cc );
+        vlc_mutex_lock( &p_owner->lock );
+        p_cc = p_owner->cc.pp_decoder[i_channel];
+        p_owner->cc.pp_decoder[i_channel] = NULL;
+        vlc_mutex_unlock( &p_owner->lock );
 
         if( p_cc )
         {
             vlc_object_kill( p_cc );
-            module_Unneed( p_cc, p_cc->p_module );
+            module_unneed( p_cc, p_cc->p_module );
             DeleteDecoder( p_cc );
             vlc_object_release( p_cc );
         }
     }
     return VLC_SUCCESS;
 }
-int input_DecoderGetCcState( decoder_t *p_dec, vlc_bool_t *pb_decode, int i_channel )
+int input_DecoderGetCcState( decoder_t *p_dec, bool *pb_decode, int i_channel )
 {
     decoder_owner_sys_t *p_owner = p_dec->p_owner;
 
-    *pb_decode = VLC_FALSE;
-    if( i_channel < 0 || i_channel >= 4 || !p_owner->pb_cc_present[i_channel] )
+    *pb_decode = false;
+    if( i_channel < 0 || i_channel >= 4 || !p_owner->cc.pb_present[i_channel] )
         return VLC_EGENERIC;
 
-    vlc_mutex_lock( &p_owner->lock_cc );
-    *pb_decode = p_dec->p_owner->pp_cc[i_channel] != NULL;
-    vlc_mutex_unlock( &p_dec->p_owner->lock_cc );
+    vlc_mutex_lock( &p_owner->lock );
+    *pb_decode = p_owner->cc.pp_decoder[i_channel] != NULL;
+    vlc_mutex_unlock( &p_owner->lock );
     return VLC_EGENERIC;
 }
 
+void input_DecoderChangePause( decoder_t *p_dec, bool b_paused, mtime_t i_date )
+{
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+
+    vlc_mutex_lock( &p_owner->lock );
+
+    assert( !p_owner->b_paused || !b_paused );
+
+    p_owner->b_paused = b_paused;
+    p_owner->pause.i_date = i_date;
+    p_owner->pause.i_ignore = 0;
+    vlc_cond_signal( &p_owner->wait );
+
+    DecoderOutputChangePause( p_dec, b_paused, i_date );
+
+    vlc_mutex_unlock( &p_owner->lock );
+}
+
+void input_DecoderChangeDelay( decoder_t *p_dec, mtime_t i_delay )
+{
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+
+    vlc_mutex_lock( &p_owner->lock );
+    p_owner->i_ts_delay = i_delay;
+    vlc_mutex_unlock( &p_owner->lock );
+}
+
+void input_DecoderStartBuffering( decoder_t *p_dec )
+{
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+
+    vlc_mutex_lock( &p_owner->lock );
+
+    DecoderFlush( p_dec );
+
+    p_owner->buffer.b_first = true;
+    p_owner->buffer.b_full = false;
+    p_owner->buffer.i_count = 0;
+
+    assert( !p_owner->buffer.p_picture && !p_owner->buffer.p_subpic && !p_owner->buffer.p_audio );
+
+    p_owner->buffer.p_picture = NULL;
+    p_owner->buffer.pp_picture_next = &p_owner->buffer.p_picture;
+
+    p_owner->buffer.p_subpic = NULL;
+    p_owner->buffer.pp_subpic_next = &p_owner->buffer.p_subpic;
+
+    p_owner->buffer.p_audio = NULL;
+    p_owner->buffer.pp_audio_next = &p_owner->buffer.p_audio;
+
+    p_owner->b_buffering = true;
+
+    vlc_cond_signal( &p_owner->wait );
+
+    vlc_mutex_unlock( &p_owner->lock );
+}
+
+void input_DecoderStopBuffering( decoder_t *p_dec )
+{
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+
+    vlc_mutex_lock( &p_owner->lock );
+
+    p_owner->b_buffering = false;
+
+    vlc_cond_signal( &p_owner->wait );
+
+    vlc_mutex_unlock( &p_owner->lock );
+}
+
+void input_DecoderWaitBuffering( decoder_t *p_dec )
+{
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+
+    vlc_mutex_lock( &p_owner->lock );
+
+    while( vlc_object_alive( p_dec ) && p_owner->b_buffering && !p_owner->buffer.b_full )
+    {
+        block_FifoWake( p_owner->p_fifo );
+        vlc_cond_wait( &p_owner->wait, &p_owner->lock );
+    }
+
+    vlc_mutex_unlock( &p_owner->lock );
+}
+void input_DecoderFrameNext( decoder_t *p_dec, mtime_t *pi_duration )
+{
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+
+    *pi_duration = 0;
+
+    vlc_mutex_lock( &p_owner->lock );
+    if( p_dec->fmt_in.i_cat == VIDEO_ES )
+    {
+        if( p_owner->b_paused && p_owner->p_vout )
+        {
+            vout_NextPicture( p_owner->p_vout, pi_duration );
+            p_owner->pause.i_ignore++;
+            vlc_cond_signal( &p_owner->wait );
+        }
+    }
+    else
+    {
+        /* TODO subtitle should not be flushed */
+        DecoderFlush( p_dec );
+    }
+    vlc_mutex_unlock( &p_owner->lock );
+}
+
+/*****************************************************************************
+ * Internal functions
+ *****************************************************************************/
+
+/* */
+static void DecoderUnsupportedCodec( decoder_t *p_dec, vlc_fourcc_t codec )
+{
+    msg_Err( p_dec, "no suitable decoder module for fourcc `%4.4s'.\n"
+             "VLC probably does not support this sound or video format.",
+             (char*)&codec );
+    intf_UserFatal( p_dec, false, _("No suitable decoder module"), 
+                    _("VLC does not support the audio or video format \"%4.4s\". "
+                      "Unfortunately there is no way for you to fix this."), (char*)&codec );
+}
+
+
 /**
  * Create a decoder object
  *
@@ -432,7 +582,7 @@ int input_DecoderGetCcState( decoder_t *p_dec, vlc_bool_t *pb_decode, int i_chan
  * \return the decoder object
  */
 static decoder_t * CreateDecoder( input_thread_t *p_input,
-                                  es_format_t *fmt, int i_object_type )
+                                  es_format_t *fmt, int i_object_type, sout_instance_t *p_sout )
 {
     decoder_t *p_dec;
     decoder_owner_sys_t *p_owner;
@@ -440,16 +590,13 @@ static decoder_t * CreateDecoder( input_thread_t *p_input,
 
     p_dec = vlc_object_create( p_input, i_object_type );
     if( p_dec == NULL )
-    {
-        msg_Err( p_input, "out of memory" );
         return NULL;
-    }
 
-    p_dec->pf_decode_audio = 0;
-    p_dec->pf_decode_video = 0;
-    p_dec->pf_decode_sub = 0;
-    p_dec->pf_get_cc = 0;
-    p_dec->pf_packetize = 0;
+    p_dec->pf_decode_audio = NULL;
+    p_dec->pf_decode_video = NULL;
+    p_dec->pf_decode_sub = NULL;
+    p_dec->pf_get_cc = NULL;
+    p_dec->pf_packetize = NULL;
 
     /* Initialize the decoder fifo */
     p_dec->p_module = NULL;
@@ -462,25 +609,27 @@ static decoder_t * CreateDecoder( input_thread_t *p_input,
     p_dec->p_owner = p_owner = malloc( sizeof( decoder_owner_sys_t ) );
     if( p_dec->p_owner == NULL )
     {
-        msg_Err( p_dec, "out of memory" );
+        vlc_object_release( p_dec );
         return NULL;
     }
-    p_dec->p_owner->b_own_thread = VLC_TRUE;
     p_dec->p_owner->i_preroll_end = -1;
+    p_dec->p_owner->i_last_rate = INPUT_RATE_DEFAULT;
     p_dec->p_owner->p_input = p_input;
     p_dec->p_owner->p_aout = NULL;
     p_dec->p_owner->p_aout_input = NULL;
     p_dec->p_owner->p_vout = NULL;
     p_dec->p_owner->p_spu_vout = NULL;
     p_dec->p_owner->i_spu_channel = 0;
-    p_dec->p_owner->p_sout = p_input->p->p_sout;
+    p_dec->p_owner->i_spu_order = 0;
+    p_dec->p_owner->p_sout = p_sout;
     p_dec->p_owner->p_sout_input = NULL;
     p_dec->p_owner->p_packetizer = NULL;
 
     /* decoder fifo */
-    if( ( p_dec->p_owner->p_fifo = block_FifoNew( p_dec ) ) == NULL )
+    if( ( p_dec->p_owner->p_fifo = block_FifoNew() ) == NULL )
     {
-        msg_Err( p_dec, "out of memory" );
+        free( p_dec->p_owner );
+        vlc_object_release( p_dec );
         return NULL;
     }
 
@@ -498,9 +647,9 @@ static decoder_t * CreateDecoder( input_thread_t *p_input,
 
     /* Find a suitable decoder/packetizer module */
     if( i_object_type == VLC_OBJECT_DECODER )
-        p_dec->p_module = module_Need( p_dec, "decoder", "$codec", 0 );
+        p_dec->p_module = module_need( p_dec, "decoder", "$codec", 0 );
     else
-        p_dec->p_module = module_Need( p_dec, "packetizer", "$packetizer", 0 );
+        p_dec->p_module = module_need( p_dec, "packetizer", "$packetizer", 0 );
 
     /* Check if decoder requires already packetized data */
     if( i_object_type == VLC_OBJECT_DECODER &&
@@ -519,7 +668,7 @@ static decoder_t * CreateDecoder( input_thread_t *p_input,
             vlc_object_attach( p_dec->p_owner->p_packetizer, p_input );
 
             p_dec->p_owner->p_packetizer->p_module =
-                module_Need( p_dec->p_owner->p_packetizer,
+                module_need( p_dec->p_owner->p_packetizer,
                              "packetizer", "$packetizer", 0 );
 
             if( !p_dec->p_owner->p_packetizer->p_module )
@@ -548,22 +697,41 @@ static decoder_t * CreateDecoder( input_thread_t *p_input,
             }
         }
     }
+
+    /* */
+    vlc_mutex_init( &p_owner->lock );
+    vlc_cond_init( &p_owner->wait );
+
+    p_owner->b_paused = false;
+    p_owner->pause.i_date = 0;
+    p_owner->pause.i_ignore = 0;
+
+    p_owner->b_buffering = false;
+    p_owner->buffer.b_first = true;
+    p_owner->buffer.b_full = false;
+    p_owner->buffer.i_count = 0;
+    p_owner->buffer.p_picture = NULL;
+    p_owner->buffer.p_subpic = NULL;
+    p_owner->buffer.p_audio = NULL;
+
+    p_owner->b_flushing = false;
+
     /* */
-    p_owner->b_cc_supported = VLC_FALSE;
+    p_owner->cc.b_supported = false;
     if( i_object_type == VLC_OBJECT_DECODER )
     {
         if( p_owner->p_packetizer && p_owner->p_packetizer->pf_get_cc )
-            p_owner->b_cc_supported = VLC_TRUE;
+            p_owner->cc.b_supported = true;
         if( p_dec->pf_get_cc )
-            p_owner->b_cc_supported = VLC_TRUE;
+            p_owner->cc.b_supported = true;
     }
 
-    vlc_mutex_init( p_dec, &p_owner->lock_cc );
     for( i = 0; i < 4; i++ )
     {
-        p_owner->pb_cc_present[i] = VLC_FALSE;
-        p_owner->pp_cc[i] = NULL;
+        p_owner->cc.pb_present[i] = false;
+        p_owner->cc.pp_decoder[i] = NULL;
     }
+    p_owner->i_ts_delay = 0;
     return p_dec;
 }
 
@@ -571,389 +739,1074 @@ static decoder_t * CreateDecoder( input_thread_t *p_input,
  * The decoding main loop
  *
  * \param p_dec the decoder
- * \return 0
  */
-static int DecoderThread( decoder_t * p_dec )
+static void *DecoderThread( vlc_object_t *p_this )
 {
-    block_t *p_block;
+    decoder_t *p_dec = (decoder_t *)p_this;
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+
+    int canc = vlc_savecancel();
 
     /* The decoder's main loop */
-    while( !p_dec->b_die && !p_dec->b_error )
+    while( vlc_object_alive( p_dec ) )
     {
-        if( ( p_block = block_FifoGet( p_dec->p_owner->p_fifo ) ) == NULL )
-        {
-            p_dec->b_error = 1;
-            break;
-        }
-        if( DecoderDecode( p_dec, p_block ) != VLC_SUCCESS )
+        block_t *p_block = block_FifoGet( p_owner->p_fifo );
+
+        DecoderSignalBuffering( p_dec, p_block == NULL );
+
+        if( p_block )
         {
-            break;
+            if( p_dec->b_error )
+            {   /* Trash all received PES packets */
+                block_Release( p_block );
+            }
+            else if( DecoderProcess( p_dec, p_block ) != VLC_SUCCESS )
+            {
+                break;
+            }
         }
     }
 
-    while( !p_dec->b_die )
-    {
-        /* Trash all received PES packets */
-        p_block = block_FifoGet( p_dec->p_owner->p_fifo );
-        if( p_block ) block_Release( p_block );
-    }
+    DecoderSignalBuffering( p_dec, true );
 
     /* We do it here because of the dll loader that wants close() in the
      * same thread than open()/decode() */
-    module_Unneed( p_dec, p_dec->p_module );
-
-    return 0;
+    module_unneed( p_dec, p_dec->p_module );
+    vlc_restorecancel( canc );
+    return NULL;
 }
 
-static inline void DecoderUpdatePreroll( int64_t *pi_preroll, const block_t *p )
-{
-    if( p->i_flags & (BLOCK_FLAG_PREROLL|BLOCK_FLAG_DISCONTINUITY) )
-        *pi_preroll = INT64_MAX;
-    else if( p->i_pts > 0 )
-        *pi_preroll = __MIN( *pi_preroll, p->i_pts );
-    else if( p->i_dts > 0 )
-        *pi_preroll = __MIN( *pi_preroll, p->i_dts );
-}
-static void DecoderDecodeAudio( decoder_t *p_dec, block_t *p_block )
+static void DecoderFlush( decoder_t *p_dec )
 {
-    input_thread_t *p_input = p_dec->p_owner->p_input;
-    const int i_rate = p_block->i_rate;
-    aout_buffer_t *p_aout_buf;
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+    block_t *p_null;
 
-    while( (p_aout_buf = p_dec->pf_decode_audio( p_dec, &p_block )) )
-    {
-        vlc_mutex_lock( &p_input->p->counters.counters_lock );
-        stats_UpdateInteger( p_dec, p_input->p->counters.p_decoded_audio, 1, NULL );
-        vlc_mutex_unlock( &p_input->p->counters.counters_lock );
+    vlc_assert_locked( &p_owner->lock );
 
-        if( p_aout_buf->start_date < p_dec->p_owner->i_preroll_end )
-        {
-            aout_DecDeleteBuffer( p_dec->p_owner->p_aout,
-                                  p_dec->p_owner->p_aout_input, p_aout_buf );
-            continue;
-        }
+    /* Empty the fifo */
+    block_FifoEmpty( p_owner->p_fifo );
 
-        if( p_dec->p_owner->i_preroll_end > 0 )
-        {
-            /* FIXME TODO flush audio output (don't know how to do that) */
-            msg_Dbg( p_dec, "End of audio preroll" );
-            p_dec->p_owner->i_preroll_end = -1;
-        }
-        aout_DecPlay( p_dec->p_owner->p_aout,
-                      p_dec->p_owner->p_aout_input,
-                      p_aout_buf, i_rate );
-    }
+    /* Monitor for flush end */
+    p_owner->b_flushing = true;
+    vlc_cond_signal( &p_owner->wait );
+
+    /* Send a special block */
+    p_null = block_New( p_dec, 128 );
+    if( !p_null )
+        return;
+    p_null->i_flags |= BLOCK_FLAG_DISCONTINUITY;
+    p_null->i_flags |= BLOCK_FLAG_CORE_FLUSH;
+    if( !p_dec->fmt_in.b_packetized )
+        p_null->i_flags |= BLOCK_FLAG_CORRUPTED;
+    memset( p_null->p_buffer, 0, p_null->i_buffer );
+
+    input_DecoderDecode( p_dec, p_null );
+
+    /* */
+    while( vlc_object_alive( p_dec ) && p_owner->b_flushing )
+        vlc_cond_wait( &p_owner->wait, &p_owner->lock );
 }
-static void DecoderGetCc( decoder_t *p_dec, decoder_t *p_dec_cc )
+
+static void DecoderSignalFlushed( decoder_t *p_dec )
 {
-    block_t *p_cc;
-    vlc_bool_t pb_present[4];
-    int i;
-    int i_cc_decoder;
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
 
-    assert( p_dec_cc->pf_get_cc != NULL );
+    vlc_mutex_lock( &p_owner->lock );
 
-    /* Do not try retreiving CC if not wanted (sout) or cannot be retreived */
-    if( !p_dec->p_owner->b_cc_supported )
-        return;
+    if( p_owner->b_flushing )
+    {
+        p_owner->b_flushing = false;
+        vlc_cond_signal( &p_owner->wait );
+    }
 
-    p_cc = p_dec_cc->pf_get_cc( p_dec_cc, pb_present );
-    if( !p_cc )
-        return;
+    vlc_mutex_unlock( &p_owner->lock );
+}
 
-    vlc_mutex_lock( &p_dec->p_owner->lock_cc );
-    for( i = 0, i_cc_decoder = 0; i < 4; i++ )
+static void DecoderSignalBuffering( decoder_t *p_dec, bool b_full )
+{
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+
+    vlc_mutex_lock( &p_owner->lock );
+
+    if( p_owner->b_buffering )
     {
-        p_dec->p_owner->pb_cc_present[i] |= pb_present[i];
-        if( p_dec->p_owner->pp_cc[i] )
-            i_cc_decoder++;
+        if( b_full )
+            p_owner->buffer.b_full = true;
+        vlc_cond_signal( &p_owner->wait );
     }
 
-    for( i = 0; i < 4; i++ )
-    {
-        if( !p_dec->p_owner->pp_cc[i] )
-            continue;
+    vlc_mutex_unlock( &p_owner->lock );
+}
 
-        if( i_cc_decoder > 1 )
-            DecoderDecode( p_dec->p_owner->pp_cc[i], block_Duplicate( p_cc ) );
+static bool DecoderIsFlushing( decoder_t *p_dec )
+{
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+    bool b_flushing;
+
+    vlc_mutex_lock( &p_owner->lock );
+
+    b_flushing = p_owner->b_flushing;
+
+    vlc_mutex_unlock( &p_owner->lock );
+
+    return b_flushing;
+}
+
+static void DecoderWaitUnblock( decoder_t *p_dec, bool *pb_reject )
+{
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+
+    vlc_assert_locked( &p_owner->lock );
+
+    while( !p_owner->b_flushing )
+    {
+        if( p_owner->b_paused )
+        {
+            if( p_owner->b_buffering && !p_owner->buffer.b_full )
+                break;
+            if( p_owner->pause.i_ignore > 0 )
+            {
+                p_owner->pause.i_ignore--;
+                break;
+            }
+        }
         else
-            DecoderDecode( p_dec->p_owner->pp_cc[i], p_cc );
-        i_cc_decoder--;
+        {
+            if( !p_owner->b_buffering || !p_owner->buffer.b_full )
+                break;
+        }
+        vlc_cond_wait( &p_owner->wait, &p_owner->lock );
     }
-    vlc_mutex_unlock( &p_dec->p_owner->lock_cc );
+
+    if( pb_reject )
+        *pb_reject = p_owner->b_flushing;
 }
-static void VoutDisplayedPicture( vout_thread_t *p_vout, picture_t *p_pic )
+
+static void DecoderOutputChangePause( decoder_t *p_dec, bool b_paused, mtime_t i_date )
 {
-    vlc_mutex_lock( &p_vout->picture_lock );
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+
+    vlc_assert_locked( &p_owner->lock );
 
-    if( p_pic->i_status == READY_PICTURE )
+    /* XXX only audio and video output have to be paused.
+     * - for sout it is useless
+     * - for subs, it is done by the vout
+     */
+    if( p_dec->fmt_in.i_cat == AUDIO_ES )
     {
-        /* Grr cannot destroy ready picture by myself so be sure vout won't like it */
-        p_pic->date = 1;
+        if( p_owner->p_aout && p_owner->p_aout_input )
+            aout_DecChangePause( p_owner->p_aout, p_owner->p_aout_input,
+                                 b_paused, i_date );
     }
-    else if( p_pic->i_refcount > 0 )
+    else if( p_dec->fmt_in.i_cat == VIDEO_ES )
     {
-        p_pic->i_status = DISPLAYED_PICTURE;
+        if( p_owner->p_vout )
+            vout_ChangePause( p_owner->p_vout, b_paused, i_date );
     }
-    else
+}
+static inline void DecoderUpdatePreroll( int64_t *pi_preroll, const block_t *p )
+{
+    if( p->i_flags & (BLOCK_FLAG_PREROLL|BLOCK_FLAG_DISCONTINUITY) )
+        *pi_preroll = INT64_MAX;
+    else if( p->i_pts > 0 )
+        *pi_preroll = __MIN( *pi_preroll, p->i_pts );
+    else if( p->i_dts > 0 )
+        *pi_preroll = __MIN( *pi_preroll, p->i_dts );
+}
+
+static mtime_t DecoderTeletextFixTs( mtime_t i_ts, mtime_t i_ts_delay )
+{
+    mtime_t current_date = mdate();
+
+    /* FIXME I don't really like that, es_out SHOULD do it using the video pts */
+    if( !i_ts || i_ts > current_date + 10000000 || i_ts < current_date )
+    {
+        /* ETSI EN 300 472 Annex A : do not take into account the PTS
+         * for teletext streams. */
+        return current_date + 400000 + i_ts_delay;
+    }
+    return i_ts;
+}
+
+static void DecoderFixTs( decoder_t *p_dec, mtime_t *pi_ts0, mtime_t *pi_ts1,
+                          mtime_t *pi_duration, int *pi_rate, mtime_t *pi_delay, bool b_telx )
+{
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+    input_clock_t   *p_clock = p_owner->p_clock;
+    int i_rate = 0;
+
+    vlc_assert_locked( &p_owner->lock );
+
+    const mtime_t i_ts_delay = p_owner->p_input->i_pts_delay;
+    const mtime_t i_es_delay = p_owner->i_ts_delay;
+
+    if( p_clock )
+    {
+        const bool b_ephemere = pi_ts1 && *pi_ts0 == *pi_ts1;
+
+        if( *pi_ts0 > 0 )
+            *pi_ts0 = input_clock_GetTS( p_clock, &i_rate, i_ts_delay,
+                                         *pi_ts0 + i_es_delay );
+        if( pi_ts1 && *pi_ts1 > 0 )
+            *pi_ts1 = input_clock_GetTS( p_clock, &i_rate, i_ts_delay,
+                                         *pi_ts1 + i_es_delay );
+
+        /* Do not create ephemere data because of rounding errors */
+        if( !b_ephemere && pi_ts1 && *pi_ts0 == *pi_ts1 )
+            *pi_ts1 += 1;
+
+        if( i_rate <= 0 )
+            i_rate = input_clock_GetRate( p_clock ); 
+
+        if( pi_duration )
+            *pi_duration = ( *pi_duration * i_rate +
+                                    INPUT_RATE_DEFAULT-1 ) / INPUT_RATE_DEFAULT;
+
+        if( pi_rate )
+            *pi_rate = i_rate;
+
+        if( b_telx )
+        {
+            *pi_ts0 = DecoderTeletextFixTs( *pi_ts0, i_ts_delay );
+            if( pi_ts1 && *pi_ts1 <= 0 )
+                *pi_ts1 = *pi_ts0;
+        }
+    }
+    if( pi_delay )
+    {
+        const int r = i_rate > 0 ? i_rate : INPUT_RATE_DEFAULT;
+        *pi_delay = i_ts_delay + i_es_delay * r / INPUT_RATE_DEFAULT;
+    }
+}
+
+static void DecoderPlayAudio( decoder_t *p_dec, aout_buffer_t *p_audio,
+                              int *pi_played_sum, int *pi_lost_sum )
+{
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+    aout_instance_t *p_aout = p_owner->p_aout;
+    aout_input_t    *p_aout_input = p_owner->p_aout_input;
+
+    /* */
+    if( p_audio->start_date <= 0 )
     {
-        p_pic->i_status = DESTROYED_PICTURE;
-        p_vout->i_heap_size--;
+        msg_Warn( p_dec, "non-dated audio buffer received" );
+        *pi_lost_sum += 1;
+        aout_BufferFree( p_audio );
+        return;
     }
 
-    vlc_mutex_unlock( &p_vout->picture_lock );
+    /* */
+    vlc_mutex_lock( &p_owner->lock );
+
+    if( p_owner->b_buffering || p_owner->buffer.p_audio )
+    {
+        p_audio->p_next = NULL;
+
+        *p_owner->buffer.pp_audio_next = p_audio;
+        p_owner->buffer.pp_audio_next = &p_audio->p_next;
+
+        p_owner->buffer.i_count++;
+        if( p_owner->buffer.i_count > DECODER_MAX_BUFFERING_COUNT ||
+            p_audio->start_date - p_owner->buffer.p_audio->start_date > DECODER_MAX_BUFFERING_AUDIO_DURATION )
+        {
+            p_owner->buffer.b_full = true;
+            vlc_cond_signal( &p_owner->wait );
+        }
+    }
+
+    for( ;; )
+    {
+        bool b_has_more = false;
+        bool b_reject;
+        DecoderWaitUnblock( p_dec, &b_reject );
+
+        if( p_owner->b_buffering )
+        {
+            vlc_mutex_unlock( &p_owner->lock );
+            return;
+        }
+
+        /* */
+        if( p_owner->buffer.p_audio )
+        {
+            p_audio = p_owner->buffer.p_audio;
+
+            p_owner->buffer.p_audio = p_audio->p_next;
+            p_owner->buffer.i_count--;
+
+            b_has_more = p_owner->buffer.p_audio != NULL;
+            if( !b_has_more )
+                p_owner->buffer.pp_audio_next = &p_owner->buffer.p_audio;
+        }
+
+        /* */
+        int i_rate = INPUT_RATE_DEFAULT;
+        mtime_t i_delay;
+
+        DecoderFixTs( p_dec, &p_audio->start_date, &p_audio->end_date, NULL,
+                      &i_rate, &i_delay, false );
+
+        vlc_mutex_unlock( &p_owner->lock );
+
+        /* */
+        const mtime_t i_max_date = mdate() + i_delay + AOUT_MAX_ADVANCE_TIME;
+
+        if( !p_aout || !p_aout_input ||
+            p_audio->start_date <= 0 || p_audio->start_date > i_max_date ||
+            i_rate < INPUT_RATE_DEFAULT/AOUT_MAX_INPUT_RATE ||
+            i_rate > INPUT_RATE_DEFAULT*AOUT_MAX_INPUT_RATE )
+            b_reject = true;
+
+        if( !b_reject )
+        {
+            /* Wait if we are too early
+             * FIXME that's plain ugly to do it here */
+            mwait( p_audio->start_date - AOUT_MAX_PREPARE_TIME );
+
+            if( !aout_DecPlay( p_aout, p_aout_input, p_audio, i_rate ) )
+                *pi_played_sum += 1;
+            *pi_lost_sum += aout_DecGetResetLost( p_aout, p_aout_input );
+        }
+        else
+        {
+            if( p_audio->start_date <= 0 )
+            {
+                msg_Warn( p_dec, "non-dated audio buffer received" );
+            }
+            else if( p_audio->start_date > i_max_date )
+            {
+                msg_Warn( p_aout, "received buffer in the future (%"PRId64")",
+                          p_audio->start_date - mdate() );
+            }
+            *pi_lost_sum += 1;
+            aout_BufferFree( p_audio );
+        }
+
+        if( !b_has_more )
+            break;
+
+        vlc_mutex_lock( &p_owner->lock );
+        if( !p_owner->buffer.p_audio )
+        {
+            vlc_mutex_unlock( &p_owner->lock );
+            break;
+        }
+    }
 }
-static void VoutFlushPicture( vout_thread_t *p_vout )
+
+static void DecoderDecodeAudio( decoder_t *p_dec, block_t *p_block )
 {
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+    input_thread_t  *p_input = p_owner->p_input;
+    aout_buffer_t   *p_aout_buf;
+    int i_decoded = 0;
+    int i_lost = 0;
+    int i_played = 0;
+
+    while( (p_aout_buf = p_dec->pf_decode_audio( p_dec, &p_block )) )
+    {
+        aout_instance_t *p_aout = p_owner->p_aout;
+        aout_input_t    *p_aout_input = p_owner->p_aout_input;
+        int i_lost = 0;
+        int i_played;
+
+        if( p_dec->b_die )
+        {
+            /* It prevent freezing VLC in case of broken decoder */
+            aout_DecDeleteBuffer( p_aout, p_aout_input, p_aout_buf );
+            if( p_block )
+                block_Release( p_block );
+            break;
+        }
+        i_decoded++;
+
+        if( p_aout_buf->start_date < p_owner->i_preroll_end )
+        {
+            aout_DecDeleteBuffer( p_aout, p_aout_input, p_aout_buf );
+            continue;
+        }
+
+        if( p_owner->i_preroll_end > 0 )
+        {
+            msg_Dbg( p_dec, "End of audio preroll" );
+            if( p_owner->p_aout && p_owner->p_aout_input )
+                aout_DecFlush( p_owner->p_aout, p_owner->p_aout_input );
+            /* */
+            p_owner->i_preroll_end = -1;
+        }
+
+        DecoderPlayAudio( p_dec, p_aout_buf, &i_played, &i_lost );
+    }
+
+    /* Update ugly stat */
+    if( i_decoded > 0 || i_lost > 0 || i_played > 0 )
+    {
+        vlc_mutex_lock( &p_input->p->counters.counters_lock);
+
+        stats_UpdateInteger( p_dec, p_input->p->counters.p_lost_abuffers,
+                             i_lost, NULL );
+        stats_UpdateInteger( p_dec, p_input->p->counters.p_played_abuffers,
+                             i_played, NULL );
+        stats_UpdateInteger( p_dec, p_input->p->counters.p_decoded_audio,
+                             i_decoded, NULL );
+
+        vlc_mutex_unlock( &p_input->p->counters.counters_lock);
+    }
+}
+static void DecoderGetCc( decoder_t *p_dec, decoder_t *p_dec_cc )
+{
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+    block_t *p_cc;
+    bool pb_present[4];
     int i;
-    vlc_mutex_lock( &p_vout->picture_lock );
-    for( i = 0; i < p_vout->render.i_pictures; i++ )
+    int i_cc_decoder;
+
+    assert( p_dec_cc->pf_get_cc != NULL );
+
+    /* Do not try retreiving CC if not wanted (sout) or cannot be retreived */
+    if( !p_owner->cc.b_supported )
+        return;
+
+    p_cc = p_dec_cc->pf_get_cc( p_dec_cc, pb_present );
+    if( !p_cc )
+        return;
+
+    vlc_mutex_lock( &p_owner->lock );
+    for( i = 0, i_cc_decoder = 0; i < 4; i++ )
+    {
+        p_owner->cc.pb_present[i] |= pb_present[i];
+        if( p_owner->cc.pp_decoder[i] )
+            i_cc_decoder++;
+    }
+
+    for( i = 0; i < 4; i++ )
     {
-        picture_t *p_pic = p_vout->render.pp_picture[i];
+        if( !p_owner->cc.pp_decoder[i] )
+            continue;
+
+        if( i_cc_decoder > 1 )
+            DecoderProcess( p_owner->cc.pp_decoder[i], block_Duplicate( p_cc ) );
+        else
+            DecoderProcess( p_owner->cc.pp_decoder[i], p_cc );
+        i_cc_decoder--;
+    }
+    vlc_mutex_unlock( &p_owner->lock );
+}
+
+static void DecoderPlayVideo( decoder_t *p_dec, picture_t *p_picture,
+                              int *pi_played_sum, int *pi_lost_sum )
+{
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+    vout_thread_t  *p_vout = p_owner->p_vout;
+    bool b_first_buffered;
 
-        if( p_pic->i_status == READY_PICTURE ||
-            p_pic->i_status == DISPLAYED_PICTURE )
+    if( p_picture->date <= 0 )
+    {
+        msg_Warn( p_vout, "non-dated video buffer received" );
+        *pi_lost_sum += 1;
+        vout_DropPicture( p_vout, p_picture );
+        return;
+    }
+
+    /* */
+    vlc_mutex_lock( &p_owner->lock );
+
+    if( ( p_owner->b_buffering && !p_owner->buffer.b_first ) || p_owner->buffer.p_picture )
+    {
+        p_picture->p_next = NULL;
+
+        *p_owner->buffer.pp_picture_next = p_picture;
+        p_owner->buffer.pp_picture_next = &p_picture->p_next;
+
+        p_owner->buffer.i_count++;
+        if( p_owner->buffer.i_count > DECODER_MAX_BUFFERING_COUNT ||
+            p_picture->date - p_owner->buffer.p_picture->date > DECODER_MAX_BUFFERING_VIDEO_DURATION )
         {
-            /* We cannot change picture status if it is in READY_PICTURE state,
-             * Just make sure they won't be displayed */
-            p_pic->date = 1;
+            p_owner->buffer.b_full = true;
+            vlc_cond_signal( &p_owner->wait );
+        }
+    }
+    b_first_buffered = p_owner->buffer.p_picture != NULL;
+
+    for( ;; b_first_buffered = false )
+    {
+        bool b_has_more = false;
+
+        bool b_reject;
+
+        DecoderWaitUnblock( p_dec, &b_reject );
+
+        if( p_owner->b_buffering && !p_owner->buffer.b_first )
+        {
+            vlc_mutex_unlock( &p_owner->lock );
+            return;
+        }
+        bool b_buffering_first = p_owner->b_buffering;
+
+        /* */
+        if( p_owner->buffer.p_picture )
+        {
+            p_picture = p_owner->buffer.p_picture;
+
+            p_owner->buffer.p_picture = p_picture->p_next;
+            p_owner->buffer.i_count--;
+
+            b_has_more = p_owner->buffer.p_picture != NULL;
+            if( !b_has_more )
+                p_owner->buffer.pp_picture_next = &p_owner->buffer.p_picture;
+        }
+
+        /* */
+        int i_rate = INPUT_RATE_DEFAULT;
+        mtime_t i_delay;
+
+        if( b_buffering_first )
+        {
+            assert( p_owner->buffer.b_first );
+            assert( !p_owner->buffer.i_count );
+            msg_Dbg( p_dec, "Received first picture" );
+            p_owner->buffer.b_first = false;
+            p_picture->b_force = true;
+            i_delay = 0;
+            if( p_owner->p_clock )
+                i_rate = input_clock_GetRate( p_owner->p_clock );
+        }
+        DecoderFixTs( p_dec, &p_picture->date, NULL, NULL,
+                      &i_rate, &i_delay, false );
+
+        vlc_mutex_unlock( &p_owner->lock );
+
+        /* */
+        const mtime_t i_max_date = mdate() + i_delay + VOUT_BOGUS_DELAY;
+
+        if( !p_picture->b_force && ( p_picture->date <= 0 || p_picture->date >= i_max_date ) )
+            b_reject = true;
+
+        if( !b_reject )
+        {
+            if( i_rate != p_owner->i_last_rate || b_first_buffered )
+            {
+                /* Be sure to not display old picture after our own */
+                vout_Flush( p_vout, p_picture->date );
+                p_owner->i_last_rate = i_rate;
+            }
+            vout_DisplayPicture( p_vout, p_picture );
+        }
+        else
+        {
+            if( p_picture->date <= 0 )
+            {
+                msg_Warn( p_vout, "non-dated video buffer received" );
+            }
+            else
+            {
+                msg_Warn( p_vout, "early picture skipped (%"PRId64")",
+                          p_picture->date - mdate() );
+            }
+            *pi_lost_sum += 1;
+            vout_DropPicture( p_vout, p_picture );
+        }
+        int i_tmp_display;
+        int i_tmp_lost;
+        vout_GetResetStatistic( p_vout, &i_tmp_display, &i_tmp_lost );
+
+        *pi_played_sum += i_tmp_display;
+        *pi_lost_sum += i_tmp_lost;
+
+        if( !b_has_more || b_buffering_first )
+            break;
+
+        vlc_mutex_lock( &p_owner->lock );
+        if( !p_owner->buffer.p_picture )
+        {
+            vlc_mutex_unlock( &p_owner->lock );
+            break;
         }
     }
-    vlc_mutex_unlock( &p_vout->picture_lock );
 }
+
 static void DecoderDecodeVideo( decoder_t *p_dec, block_t *p_block )
 {
-    input_thread_t *p_input = p_dec->p_owner->p_input;
-    picture_t *p_pic;
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+    input_thread_t *p_input = p_owner->p_input;
+    picture_t      *p_pic;
+    int i_lost = 0;
+    int i_decoded = 0;
+    int i_displayed = 0;
 
     while( (p_pic = p_dec->pf_decode_video( p_dec, &p_block )) )
     {
-        vlc_mutex_lock( &p_input->p->counters.counters_lock );
-        stats_UpdateInteger( p_dec, p_input->p->counters.p_decoded_video, 1, NULL );
-        vlc_mutex_unlock( &p_input->p->counters.counters_lock );
+        vout_thread_t  *p_vout = p_owner->p_vout;
+        if( p_dec->b_die )
+        {
+            /* It prevent freezing VLC in case of broken decoder */
+            vout_DropPicture( p_vout, p_pic );
+            if( p_block )
+                block_Release( p_block );
+            break;
+        }
 
-        if( p_pic->date < p_dec->p_owner->i_preroll_end )
+        i_decoded++;
+
+        if( p_pic->date < p_owner->i_preroll_end )
         {
-            VoutDisplayedPicture( p_dec->p_owner->p_vout, p_pic );
+            vout_DropPicture( p_vout, p_pic );
             continue;
         }
 
-        if( p_dec->p_owner->i_preroll_end > 0 )
+        if( p_owner->i_preroll_end > 0 )
         {
             msg_Dbg( p_dec, "End of video preroll" );
-            if( p_dec->p_owner->p_vout )
-                VoutFlushPicture( p_dec->p_owner->p_vout );
+            if( p_vout )
+                vout_Flush( p_vout, 1 );
             /* */
-            p_dec->p_owner->i_preroll_end = -1;
+            p_owner->i_preroll_end = -1;
         }
 
-        if( ( !p_dec->p_owner->p_packetizer || !p_dec->p_owner->p_packetizer->pf_get_cc ) && p_dec->pf_get_cc )
+        if( p_dec->pf_get_cc &&
+            ( !p_owner->p_packetizer || !p_owner->p_packetizer->pf_get_cc ) )
             DecoderGetCc( p_dec, p_dec );
 
-        vout_DatePicture( p_dec->p_owner->p_vout, p_pic,
-                          p_pic->date );
-        vout_DisplayPicture( p_dec->p_owner->p_vout, p_pic );
+        DecoderPlayVideo( p_dec, p_pic, &i_displayed, &i_lost );
+    }
+    if( i_decoded > 0 || i_lost > 0 || i_displayed > 0 )
+    {
+        vlc_mutex_lock( &p_input->p->counters.counters_lock );
+
+        stats_UpdateInteger( p_dec, p_input->p->counters.p_decoded_video,
+                             i_decoded, NULL );
+        stats_UpdateInteger( p_dec, p_input->p->counters.p_lost_pictures,
+                             i_lost , NULL);
+
+        stats_UpdateInteger( p_dec, p_input->p->counters.p_displayed_pictures,
+                             i_displayed, NULL);
+
+        vlc_mutex_unlock( &p_input->p->counters.counters_lock );
     }
 }
 
-/**
- * Decode a block
- *
- * \param p_dec the decoder object
- * \param p_block the block to decode
- * \return VLC_SUCCESS or an error code
- */
-static int DecoderDecode( decoder_t *p_dec, block_t *p_block )
+static void DecoderPlaySpu( decoder_t *p_dec, subpicture_t *p_subpic,
+                            bool b_telx )
 {
-    decoder_owner_sys_t *p_sys = (decoder_owner_sys_t *)p_dec->p_owner;
-    const int i_rate = p_block ? p_block->i_rate : INPUT_RATE_DEFAULT;
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+    vout_thread_t *p_vout = p_owner->p_spu_vout;
 
-    if( p_block && p_block->i_buffer <= 0 )
+    /* */
+    if( p_subpic->i_start <= 0 )
     {
-        block_Release( p_block );
-        return VLC_SUCCESS;
+        msg_Warn( p_dec, "non-dated spu buffer received" );
+        subpicture_Delete( p_subpic );
+        return;
     }
 
-    if( p_dec->i_object_type == VLC_OBJECT_PACKETIZER )
+    /* */
+    vlc_mutex_lock( &p_owner->lock );
+
+    if( p_owner->b_buffering || p_owner->buffer.p_subpic )
     {
-        block_t *p_sout_block;
+        p_subpic->p_next = NULL;
+
+        *p_owner->buffer.pp_subpic_next = p_subpic;
+        p_owner->buffer.pp_subpic_next = &p_subpic->p_next;
 
-        while( ( p_sout_block =
-                     p_dec->pf_packetize( p_dec, p_block ? &p_block : 0 ) ) )
+        p_owner->buffer.i_count++;
+        /* XXX it is important to be full after the first one */
+        if( p_owner->buffer.i_count > 0 )
         {
-            if( !p_dec->p_owner->p_sout_input )
-            {
-                es_format_Copy( &p_dec->p_owner->sout, &p_dec->fmt_out );
+            p_owner->buffer.b_full = true;
+            vlc_cond_signal( &p_owner->wait );
+        }
+    }
 
-                p_dec->p_owner->sout.i_group = p_dec->fmt_in.i_group;
-                p_dec->p_owner->sout.i_id = p_dec->fmt_in.i_id;
-                if( p_dec->fmt_in.psz_language )
-                {
-                    if( p_dec->p_owner->sout.psz_language )
-                        free( p_dec->p_owner->sout.psz_language );
-                    p_dec->p_owner->sout.psz_language =
-                        strdup( p_dec->fmt_in.psz_language );
-                }
+    for( ;; )
+    {
+        bool b_has_more = false;
+        bool b_reject;
+        DecoderWaitUnblock( p_dec, &b_reject );
 
-                p_dec->p_owner->p_sout_input =
-                    sout_InputNew( p_dec->p_owner->p_sout,
-                                   &p_dec->p_owner->sout );
+        if( p_owner->b_buffering )
+        {
+            vlc_mutex_unlock( &p_owner->lock );
+            return;
+        }
 
-                if( p_dec->p_owner->p_sout_input == NULL )
-                {
-                    msg_Err( p_dec, "cannot create packetizer output (%4.4s)",
-                             (char *)&p_dec->p_owner->sout.i_codec );
-                    p_dec->b_error = VLC_TRUE;
-
-                    while( p_sout_block )
-                    {
-                        block_t *p_next = p_sout_block->p_next;
-                        block_Release( p_sout_block );
-                        p_sout_block = p_next;
-                    }
-                    break;
-                }
-            }
+        /* */
+        if( p_owner->buffer.p_subpic )
+        {
+            p_subpic = p_owner->buffer.p_subpic;
 
-            while( p_sout_block )
-            {
-                block_t *p_next = p_sout_block->p_next;
+            p_owner->buffer.p_subpic = p_subpic->p_next;
+            p_owner->buffer.i_count--;
 
-                p_sout_block->p_next = NULL;
-                p_sout_block->i_rate = i_rate;
+            b_has_more = p_owner->buffer.p_subpic != NULL;
+            if( !b_has_more )
+                p_owner->buffer.pp_subpic_next = &p_owner->buffer.p_subpic;
+        }
 
-                sout_InputSendBuffer( p_dec->p_owner->p_sout_input,
-                                      p_sout_block );
+        /* */
+        DecoderFixTs( p_dec, &p_subpic->i_start, &p_subpic->i_stop, NULL,
+                      NULL, NULL, b_telx );
 
-                p_sout_block = p_next;
-            }
+        vlc_mutex_unlock( &p_owner->lock );
 
-            /* For now it's enough, as only sout inpact on this flag */
-            if( p_dec->p_owner->p_sout->i_out_pace_nocontrol > 0 &&
-                p_dec->p_owner->p_input->p->b_out_pace_control )
-            {
-                msg_Dbg( p_dec, "switching to sync mode" );
-                p_dec->p_owner->p_input->p->b_out_pace_control = VLC_FALSE;
-            }
-            else if( p_dec->p_owner->p_sout->i_out_pace_nocontrol <= 0 &&
-                     !p_dec->p_owner->p_input->p->b_out_pace_control )
-            {
-                msg_Dbg( p_dec, "switching to async mode" );
-                p_dec->p_owner->p_input->p->b_out_pace_control = VLC_TRUE;
-            }
+        if( !b_reject )
+            spu_DisplaySubpicture( p_vout->p_spu, p_subpic );
+        else
+            subpicture_Delete( p_subpic );
+
+        if( !b_has_more )
+            break;
+        vlc_mutex_lock( &p_owner->lock );
+        if( !p_owner->buffer.p_subpic )
+        {
+            vlc_mutex_unlock( &p_owner->lock );
+            break;
         }
     }
-    else if( p_dec->fmt_in.i_cat == AUDIO_ES )
+}
+
+static void DecoderPlaySout( decoder_t *p_dec, block_t *p_sout_block,
+                             bool b_telx )
+{
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+
+    assert( p_owner->p_clock );
+
+    vlc_mutex_lock( &p_owner->lock );
+
+    bool b_reject;
+    DecoderWaitUnblock( p_dec, &b_reject );
+
+    DecoderFixTs( p_dec, &p_sout_block->i_dts, &p_sout_block->i_pts, &p_sout_block->i_length,
+                  &p_sout_block->i_rate, NULL, b_telx );
+
+    vlc_mutex_unlock( &p_owner->lock );
+
+    sout_InputSendBuffer( p_owner->p_sout_input, p_sout_block );
+}
+
+/* */
+static void DecoderFlushBuffering( decoder_t *p_dec )
+{
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+
+    vlc_assert_locked( &p_owner->lock );
+
+    while( p_owner->buffer.p_picture )
+    {
+        picture_t *p_picture = p_owner->buffer.p_picture;
+
+        p_owner->buffer.p_picture = p_picture->p_next;
+        p_owner->buffer.i_count--;
+
+        if( p_owner->p_vout )
+            vout_DropPicture( p_owner->p_vout, p_picture );
+
+        if( !p_owner->buffer.p_picture )
+            p_owner->buffer.pp_picture_next = &p_owner->buffer.p_picture;
+    }
+    while( p_owner->buffer.p_audio )
     {
-        if( p_block )
-            DecoderUpdatePreroll( &p_dec->p_owner->i_preroll_end, p_block );
+        aout_buffer_t *p_audio = p_owner->buffer.p_audio;
 
-        if( p_dec->p_owner->p_packetizer )
+        p_owner->buffer.p_audio = p_audio->p_next;
+        p_owner->buffer.i_count--;
+
+        aout_BufferFree( p_audio );
+
+        if( !p_owner->buffer.p_audio )
+            p_owner->buffer.pp_audio_next = &p_owner->buffer.p_audio;
+    }
+    while( p_owner->buffer.p_subpic )
+    {
+        subpicture_t *p_subpic = p_owner->buffer.p_subpic;
+
+        p_owner->buffer.p_subpic = p_subpic->p_next;
+        p_owner->buffer.i_count--;
+
+        subpicture_Delete( p_subpic );
+
+        if( !p_owner->buffer.p_subpic )
+            p_owner->buffer.pp_subpic_next = &p_owner->buffer.p_subpic;
+    }
+}
+
+/* This function process a block for sout
+ */
+static void DecoderProcessSout( decoder_t *p_dec, block_t *p_block )
+{
+    decoder_owner_sys_t *p_owner = (decoder_owner_sys_t *)p_dec->p_owner;
+    const bool b_telx = p_dec->fmt_in.i_codec == VLC_FOURCC('t','e','l','x');
+    block_t *p_sout_block;
+
+    while( ( p_sout_block =
+                 p_dec->pf_packetize( p_dec, p_block ? &p_block : NULL ) ) )
+    {
+        if( !p_owner->p_sout_input )
         {
-            block_t *p_packetized_block;
-            decoder_t *p_packetizer = p_dec->p_owner->p_packetizer;
+            es_format_Copy( &p_owner->sout, &p_dec->fmt_out );
 
-            while( (p_packetized_block =
-                    p_packetizer->pf_packetize( p_packetizer, p_block ? &p_block : NULL )) )
+            p_owner->sout.i_group = p_dec->fmt_in.i_group;
+            p_owner->sout.i_id = p_dec->fmt_in.i_id;
+            if( p_dec->fmt_in.psz_language )
             {
-                if( p_packetizer->fmt_out.i_extra && !p_dec->fmt_in.i_extra )
-                {
-                    es_format_Clean( &p_dec->fmt_in );
-                    es_format_Copy( &p_dec->fmt_in, &p_packetizer->fmt_out );
-                }
+                if( p_owner->sout.psz_language )
+                    free( p_owner->sout.psz_language );
+                p_owner->sout.psz_language =
+                    strdup( p_dec->fmt_in.psz_language );
+            }
 
-                while( p_packetized_block )
-                {
-                    block_t *p_next = p_packetized_block->p_next;
-                    p_packetized_block->p_next = NULL;
-                    p_packetized_block->i_rate = i_rate;
+            p_owner->p_sout_input =
+                sout_InputNew( p_owner->p_sout,
+                               &p_owner->sout );
 
-                    DecoderDecodeAudio( p_dec, p_packetized_block );
+            if( p_owner->p_sout_input == NULL )
+            {
+                msg_Err( p_dec, "cannot create packetizer output (%4.4s)",
+                         (char *)&p_owner->sout.i_codec );
+                p_dec->b_error = true;
 
-                    p_packetized_block = p_next;
+                while( p_sout_block )
+                {
+                    block_t *p_next = p_sout_block->p_next;
+                    block_Release( p_sout_block );
+                    p_sout_block = p_next;
                 }
+                break;
             }
         }
-        else if( p_block )
+
+        while( p_sout_block )
         {
-            DecoderDecodeAudio( p_dec, p_block );
+            block_t *p_next = p_sout_block->p_next;
+
+            p_sout_block->p_next = NULL;
+
+            DecoderPlaySout( p_dec, p_sout_block, b_telx );
+
+            p_sout_block = p_next;
+        }
+
+        /* For now it's enough, as only sout impact on this flag */
+        if( p_owner->p_sout->i_out_pace_nocontrol > 0 &&
+            p_owner->p_input->p->b_out_pace_control )
+        {
+            msg_Dbg( p_dec, "switching to sync mode" );
+            p_owner->p_input->p->b_out_pace_control = false;
+        }
+        else if( p_owner->p_sout->i_out_pace_nocontrol <= 0 &&
+                 !p_owner->p_input->p->b_out_pace_control )
+        {
+            msg_Dbg( p_dec, "switching to async mode" );
+            p_owner->p_input->p->b_out_pace_control = true;
         }
     }
-    else if( p_dec->fmt_in.i_cat == VIDEO_ES )
+}
+
+/* This function process a video block
+ */
+static void DecoderProcessVideo( decoder_t *p_dec, block_t *p_block, bool b_flush )
+{
+    decoder_owner_sys_t *p_owner = (decoder_owner_sys_t *)p_dec->p_owner;
+
+    if( p_owner->p_packetizer )
     {
-        if( p_block )
-            DecoderUpdatePreroll( &p_dec->p_owner->i_preroll_end, p_block );
+        block_t *p_packetized_block;
+        decoder_t *p_packetizer = p_owner->p_packetizer;
 
-        if( p_dec->p_owner->p_packetizer )
+        while( (p_packetized_block =
+                p_packetizer->pf_packetize( p_packetizer, p_block ? &p_block : NULL )) )
         {
-            block_t *p_packetized_block;
-            decoder_t *p_packetizer = p_dec->p_owner->p_packetizer;
-
-            while( (p_packetized_block =
-                    p_packetizer->pf_packetize( p_packetizer, p_block ? &p_block : NULL )) )
+            if( p_packetizer->fmt_out.i_extra && !p_dec->fmt_in.i_extra )
             {
-                if( p_packetizer->fmt_out.i_extra && !p_dec->fmt_in.i_extra )
-                {
-                    es_format_Clean( &p_dec->fmt_in );
-                    es_format_Copy( &p_dec->fmt_in, &p_packetizer->fmt_out );
-                }
-                if( p_packetizer->pf_get_cc )
-                    DecoderGetCc( p_dec, p_packetizer );
+                es_format_Clean( &p_dec->fmt_in );
+                es_format_Copy( &p_dec->fmt_in, &p_packetizer->fmt_out );
+            }
+            if( p_packetizer->pf_get_cc )
+                DecoderGetCc( p_dec, p_packetizer );
 
-                while( p_packetized_block )
-                {
-                    block_t *p_next = p_packetized_block->p_next;
-                    p_packetized_block->p_next = NULL;
-                    p_packetized_block->i_rate = i_rate;
+            while( p_packetized_block )
+            {
+                block_t *p_next = p_packetized_block->p_next;
+                p_packetized_block->p_next = NULL;
 
-                    DecoderDecodeVideo( p_dec, p_packetized_block );
+                DecoderDecodeVideo( p_dec, p_packetized_block );
 
-                    p_packetized_block = p_next;
-                }
+                p_packetized_block = p_next;
             }
         }
-        else if( p_block )
+    }
+    else if( p_block )
+    {
+        DecoderDecodeVideo( p_dec, p_block );
+    }
+
+    if( b_flush && p_owner->p_vout )
+        vout_Flush( p_owner->p_vout, 1 );
+}
+
+/* This function process a audio block
+ */
+static void DecoderProcessAudio( decoder_t *p_dec, block_t *p_block, bool b_flush )
+{
+    decoder_owner_sys_t *p_owner = (decoder_owner_sys_t *)p_dec->p_owner;
+
+    if( p_owner->p_packetizer )
+    {
+        block_t *p_packetized_block;
+        decoder_t *p_packetizer = p_owner->p_packetizer;
+
+        while( (p_packetized_block =
+                p_packetizer->pf_packetize( p_packetizer, p_block ? &p_block : NULL )) )
         {
-            DecoderDecodeVideo( p_dec, p_block );
+            if( p_packetizer->fmt_out.i_extra && !p_dec->fmt_in.i_extra )
+            {
+                es_format_Clean( &p_dec->fmt_in );
+                es_format_Copy( &p_dec->fmt_in, &p_packetizer->fmt_out );
+            }
+
+            while( p_packetized_block )
+            {
+                block_t *p_next = p_packetized_block->p_next;
+                p_packetized_block->p_next = NULL;
+
+                DecoderDecodeAudio( p_dec, p_packetized_block );
+
+                p_packetized_block = p_next;
+            }
         }
     }
-    else if( p_dec->fmt_in.i_cat == SPU_ES )
+    else if( p_block )
     {
-        input_thread_t *p_input = p_dec->p_owner->p_input;
-        vout_thread_t *p_vout;
-        subpicture_t *p_spu;
+        DecoderDecodeAudio( p_dec, p_block );
+    }
 
-        if( p_block )
-            DecoderUpdatePreroll( &p_dec->p_owner->i_preroll_end, p_block );
+    if( b_flush && p_owner->p_aout && p_owner->p_aout_input )
+        aout_DecFlush( p_owner->p_aout, p_owner->p_aout_input );
+}
 
-        while( (p_spu = p_dec->pf_decode_sub( p_dec, p_block ? &p_block : NULL ) ) )
-        {
-            vlc_mutex_lock( &p_input->p->counters.counters_lock );
-            stats_UpdateInteger( p_dec, p_input->p->counters.p_decoded_sub, 1, NULL );
-            vlc_mutex_unlock( &p_input->p->counters.counters_lock );
+/* This function process a subtitle block
+ */
+static void DecoderProcessSpu( decoder_t *p_dec, block_t *p_block, bool b_flush )
+{
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+    const bool b_telx = p_dec->fmt_in.i_codec == VLC_FOURCC('t','e','l','x');
+
+    input_thread_t *p_input = p_owner->p_input;
+    vout_thread_t *p_vout;
+    subpicture_t *p_spu;
+
+    while( (p_spu = p_dec->pf_decode_sub( p_dec, p_block ? &p_block : NULL ) ) )
+    {
+        vlc_mutex_lock( &p_input->p->counters.counters_lock );
+        stats_UpdateInteger( p_dec, p_input->p->counters.p_decoded_sub, 1, NULL );
+        vlc_mutex_unlock( &p_input->p->counters.counters_lock );
 
-            p_vout = vlc_object_find( p_dec, VLC_OBJECT_VOUT, FIND_ANYWHERE );
-            if( p_vout && p_sys->p_spu_vout == p_vout )
+        p_vout = vlc_object_find( p_dec, VLC_OBJECT_VOUT, FIND_ANYWHERE );
+        if( p_vout && p_owner->p_spu_vout == p_vout )
+        {
+            /* Preroll does not work very well with subtitle */
+            if( p_spu->i_start > 0 &&
+                p_spu->i_start < p_owner->i_preroll_end &&
+                ( p_spu->i_stop <= 0 || p_spu->i_stop < p_owner->i_preroll_end ) )
             {
-                /* Prerool does not work very well with subtitle */
-                if( p_spu->i_start < p_dec->p_owner->i_preroll_end &&
-                    ( p_spu->i_stop <= 0 || p_spu->i_stop < p_dec->p_owner->i_preroll_end ) )
-                    spu_DestroySubpicture( p_vout->p_spu, p_spu );
-                else
-                    spu_DisplaySubpicture( p_vout->p_spu, p_spu );
+                subpicture_Delete( p_spu );
             }
             else
             {
-                msg_Warn( p_dec, "no vout found, leaking subpicture" );
+                DecoderPlaySpu( p_dec, p_spu, b_telx );
             }
-            if( p_vout )
-                vlc_object_release( p_vout );
         }
+        else
+        {
+            subpicture_Delete( p_spu );
+        }
+        if( p_vout )
+            vlc_object_release( p_vout );
+    }
+
+    if( b_flush && p_owner->p_spu_vout )
+    {
+        p_vout = vlc_object_find( p_dec, VLC_OBJECT_VOUT, FIND_ANYWHERE );
+
+        if( p_vout && p_owner->p_spu_vout == p_vout )
+            spu_Control( p_vout->p_spu, SPU_CHANNEL_CLEAR,
+                         p_owner->i_spu_channel );
+
+        if( p_vout )
+            vlc_object_release( p_vout );
+    }
+}
+
+/**
+ * Decode a block
+ *
+ * \param p_dec the decoder object
+ * \param p_block the block to decode
+ * \return VLC_SUCCESS or an error code
+ */
+static int DecoderProcess( decoder_t *p_dec, block_t *p_block )
+{
+    decoder_owner_sys_t *p_owner = (decoder_owner_sys_t *)p_dec->p_owner;
+    const bool b_flush_request = p_block && (p_block->i_flags & BLOCK_FLAG_CORE_FLUSH);
+
+    if( p_block && p_block->i_buffer <= 0 )
+    {
+        assert( !b_flush_request );
+        block_Release( p_block );
+        return VLC_SUCCESS;
+    }
+
+#ifdef ENABLE_SOUT
+    if( p_dec->i_object_type == VLC_OBJECT_PACKETIZER )
+    {
+        if( p_block )
+            p_block->i_flags &= ~BLOCK_FLAG_CORE_PRIVATE_MASK;
+
+        DecoderProcessSout( p_dec, p_block );
     }
     else
+#endif
     {
-        msg_Err( p_dec, "unknown ES format" );
-        p_dec->b_error = 1;
+        bool b_flush = false;
+
+        if( p_block )
+        {
+            const bool b_flushing = p_owner->i_preroll_end == INT64_MAX;
+            DecoderUpdatePreroll( &p_owner->i_preroll_end, p_block );
+
+            b_flush = !b_flushing && b_flush_request;
+
+            p_block->i_flags &= ~BLOCK_FLAG_CORE_PRIVATE_MASK;
+        }
+
+        if( p_dec->fmt_in.i_cat == AUDIO_ES )
+        {
+            DecoderProcessAudio( p_dec, p_block, b_flush );
+        }
+        else if( p_dec->fmt_in.i_cat == VIDEO_ES )
+        {
+            DecoderProcessVideo( p_dec, p_block, b_flush );
+        }
+        else if( p_dec->fmt_in.i_cat == SPU_ES )
+        {
+            DecoderProcessSpu( p_dec, p_block, b_flush );
+        }
+        else
+        {
+            msg_Err( p_dec, "unknown ES format" );
+            p_dec->b_error = true;
+        }
+    }
+
+    /* */
+    if( b_flush_request )
+    {
+        vlc_mutex_lock( &p_owner->lock );
+        DecoderFlushBuffering( p_dec );
+        vlc_mutex_unlock( &p_owner->lock );
+
+        DecoderSignalFlushed( p_dec );
     }
 
     return p_dec->b_error ? VLC_EGENERIC : VLC_SUCCESS;
@@ -967,53 +1820,56 @@ static int DecoderDecode( decoder_t *p_dec, block_t *p_block )
  */
 static void DeleteDecoder( decoder_t * p_dec )
 {
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+
     msg_Dbg( p_dec, "killing decoder fourcc `%4.4s', %u PES in FIFO",
              (char*)&p_dec->fmt_in.i_codec,
-             (unsigned)block_FifoCount( p_dec->p_owner->p_fifo ) );
+             (unsigned)block_FifoCount( p_owner->p_fifo ) );
 
     /* Free all packets still in the decoder fifo. */
-    block_FifoEmpty( p_dec->p_owner->p_fifo );
-    block_FifoRelease( p_dec->p_owner->p_fifo );
+    block_FifoEmpty( p_owner->p_fifo );
+    block_FifoRelease( p_owner->p_fifo );
 
-    /* Cleanup */
-    if( p_dec->p_owner->p_aout_input )
-        aout_DecDelete( p_dec->p_owner->p_aout, p_dec->p_owner->p_aout_input );
+    /* */
+    vlc_mutex_lock( &p_owner->lock );
+    DecoderFlushBuffering( p_dec );
+    vlc_mutex_unlock( &p_owner->lock );
 
-    if( p_dec->p_owner->p_vout )
+    /* Cleanup */
+    if( p_owner->p_aout_input )
+        aout_DecDelete( p_owner->p_aout, p_owner->p_aout_input );
+    if( p_owner->p_aout )
+    {
+        vlc_object_release( p_owner->p_aout );
+        p_owner->p_aout = NULL;
+    }
+    if( p_owner->p_vout )
     {
-        int i_pic;
-
-#define p_pic p_dec->p_owner->p_vout->render.pp_picture[i_pic]
         /* Hack to make sure all the the pictures are freed by the decoder */
-        for( i_pic = 0; i_pic < p_dec->p_owner->p_vout->render.i_pictures;
-             i_pic++ )
-        {
-            if( p_pic->i_status == RESERVED_PICTURE )
-                vout_DestroyPicture( p_dec->p_owner->p_vout, p_pic );
-            if( p_pic->i_refcount > 0 )
-                vout_UnlinkPicture( p_dec->p_owner->p_vout, p_pic );
-        }
-#undef p_pic
+        vout_FixLeaks( p_owner->p_vout, true );
 
         /* We are about to die. Reattach video output to p_vlc. */
-        vout_Request( p_dec, p_dec->p_owner->p_vout, 0 );
+        vout_Request( p_dec, p_owner->p_vout, NULL );
+        var_SetBool( p_owner->p_input, "intf-change-vout", true );
     }
 
-    if( p_dec->p_owner->p_sout_input )
+#ifdef ENABLE_SOUT
+    if( p_owner->p_sout_input )
     {
-        sout_InputDelete( p_dec->p_owner->p_sout_input );
-        es_format_Clean( &p_dec->p_owner->sout );
+        sout_InputDelete( p_owner->p_sout_input );
+        es_format_Clean( &p_owner->sout );
     }
+#endif
 
     if( p_dec->fmt_in.i_cat == SPU_ES )
     {
         vout_thread_t *p_vout;
 
         p_vout = vlc_object_find( p_dec, VLC_OBJECT_VOUT, FIND_ANYWHERE );
-        if( p_vout )
+        if( p_vout && p_owner->p_spu_vout == p_vout )
         {
             spu_Control( p_vout->p_spu, SPU_CHANNEL_CLEAR,
-                         p_dec->p_owner->i_spu_channel );
+                         p_owner->i_spu_channel );
             vlc_object_release( p_vout );
         }
     }
@@ -1021,21 +1877,22 @@ static void DeleteDecoder( decoder_t * p_dec )
     es_format_Clean( &p_dec->fmt_in );
     es_format_Clean( &p_dec->fmt_out );
 
-    if( p_dec->p_owner->p_packetizer )
+    if( p_owner->p_packetizer )
     {
-        module_Unneed( p_dec->p_owner->p_packetizer,
-                       p_dec->p_owner->p_packetizer->p_module );
-        es_format_Clean( &p_dec->p_owner->p_packetizer->fmt_in );
-        es_format_Clean( &p_dec->p_owner->p_packetizer->fmt_out );
-        vlc_object_detach( p_dec->p_owner->p_packetizer );
-        vlc_object_release( p_dec->p_owner->p_packetizer );
+        module_unneed( p_owner->p_packetizer,
+                       p_owner->p_packetizer->p_module );
+        es_format_Clean( &p_owner->p_packetizer->fmt_in );
+        es_format_Clean( &p_owner->p_packetizer->fmt_out );
+        vlc_object_detach( p_owner->p_packetizer );
+        vlc_object_release( p_owner->p_packetizer );
     }
 
-    vlc_mutex_destroy( &p_dec->p_owner->lock_cc );
+    vlc_cond_destroy( &p_owner->wait );
+    vlc_mutex_destroy( &p_owner->lock );
 
     vlc_object_detach( p_dec );
 
-    free( p_dec->p_owner );
+    free( p_owner );
 }
 
 /*****************************************************************************
@@ -1043,30 +1900,40 @@ static void DeleteDecoder( decoder_t * p_dec )
  *****************************************************************************/
 static aout_buffer_t *aout_new_buffer( decoder_t *p_dec, int i_samples )
 {
-    decoder_owner_sys_t *p_sys = (decoder_owner_sys_t *)p_dec->p_owner;
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
     aout_buffer_t *p_buffer;
 
-    if( p_sys->p_aout_input != NULL &&
-        ( p_dec->fmt_out.audio.i_rate != p_sys->audio.i_rate ||
+    if( p_owner->p_aout_input != NULL &&
+        ( p_dec->fmt_out.audio.i_rate != p_owner->audio.i_rate ||
           p_dec->fmt_out.audio.i_original_channels !=
-              p_sys->audio.i_original_channels ||
+              p_owner->audio.i_original_channels ||
           p_dec->fmt_out.audio.i_bytes_per_frame !=
-              p_sys->audio.i_bytes_per_frame ) )
+              p_owner->audio.i_bytes_per_frame ) )
     {
+        aout_input_t *p_aout_input = p_owner->p_aout_input;
+
         /* Parameters changed, restart the aout */
-        aout_DecDelete( p_sys->p_aout, p_sys->p_aout_input );
-        p_sys->p_aout_input = NULL;
+        vlc_mutex_lock( &p_owner->lock );
+
+        DecoderFlushBuffering( p_dec );
+
+        p_owner->p_aout_input = NULL;
+        aout_DecDelete( p_owner->p_aout, p_aout_input );
+
+        vlc_mutex_unlock( &p_owner->lock );
     }
 
-    if( p_sys->p_aout_input == NULL )
+    if( p_owner->p_aout_input == NULL )
     {
+        const int i_force_dolby = config_GetInt( p_dec, "force-dolby-surround" );
         audio_sample_format_t format;
-        int i_force_dolby = config_GetInt( p_dec, "force-dolby-surround" );
+        aout_input_t *p_aout_input;
+        aout_instance_t *p_aout;
 
         p_dec->fmt_out.audio.i_format = p_dec->fmt_out.i_codec;
-        p_sys->audio = p_dec->fmt_out.audio;
+        p_owner->audio = p_dec->fmt_out.audio;
 
-        memcpy( &format, &p_sys->audio, sizeof( audio_sample_format_t ) );
+        memcpy( &format, &p_owner->audio, sizeof( audio_sample_format_t ) );
         if ( i_force_dolby && (format.i_original_channels&AOUT_CHAN_PHYSMASK)
                                     == (AOUT_CHAN_LEFT|AOUT_CHAN_RIGHT) )
         {
@@ -1082,40 +1949,53 @@ static aout_buffer_t *aout_new_buffer( decoder_t *p_dec, int i_samples )
             }
         }
 
-        p_sys->p_aout_input =
-            aout_DecNew( p_dec, &p_sys->p_aout, &format, &p_dec->fmt_out.audio_replay_gain );
-        if( p_sys->p_aout_input == NULL )
+        p_aout = p_owner->p_aout;
+        p_aout_input = aout_DecNew( p_dec, &p_aout,
+                                    &format, &p_dec->fmt_out.audio_replay_gain );
+
+        vlc_mutex_lock( &p_owner->lock );
+        p_owner->p_aout = p_aout;
+        p_owner->p_aout_input = p_aout_input;
+        vlc_mutex_unlock( &p_owner->lock );
+
+        if( p_owner->p_aout_input == NULL )
         {
             msg_Err( p_dec, "failed to create audio output" );
-            p_dec->b_error = VLC_TRUE;
+            p_dec->b_error = true;
             return NULL;
         }
         p_dec->fmt_out.audio.i_bytes_per_frame =
-            p_sys->audio.i_bytes_per_frame;
+            p_owner->audio.i_bytes_per_frame;
     }
 
-    p_buffer = aout_DecNewBuffer( p_sys->p_aout_input, i_samples );
+    p_buffer = aout_DecNewBuffer( p_owner->p_aout_input, i_samples );
 
     return p_buffer;
 }
 
 static void aout_del_buffer( decoder_t *p_dec, aout_buffer_t *p_buffer )
 {
-    aout_DecDeleteBuffer( p_dec->p_owner->p_aout,
-                          p_dec->p_owner->p_aout_input, p_buffer );
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
+
+    aout_DecDeleteBuffer( p_owner->p_aout,
+                          p_owner->p_aout_input, p_buffer );
 }
 
+
+int vout_CountPictureAvailable( vout_thread_t *p_vout );
+
 static picture_t *vout_new_buffer( decoder_t *p_dec )
 {
-    decoder_owner_sys_t *p_sys = (decoder_owner_sys_t *)p_dec->p_owner;
-    picture_t *p_pic;
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
 
-    if( p_sys->p_vout == NULL ||
-        p_dec->fmt_out.video.i_width != p_sys->video.i_width ||
-        p_dec->fmt_out.video.i_height != p_sys->video.i_height ||
-        p_dec->fmt_out.video.i_chroma != p_sys->video.i_chroma ||
-        p_dec->fmt_out.video.i_aspect != p_sys->video.i_aspect )
+    if( p_owner->p_vout == NULL ||
+        p_dec->fmt_out.video.i_width != p_owner->video.i_width ||
+        p_dec->fmt_out.video.i_height != p_owner->video.i_height ||
+        p_dec->fmt_out.video.i_chroma != p_owner->video.i_chroma ||
+        p_dec->fmt_out.video.i_aspect != p_owner->video.i_aspect )
     {
+        vout_thread_t *p_vout;
+
         if( !p_dec->fmt_out.video.i_width ||
             !p_dec->fmt_out.video.i_height )
         {
@@ -1168,78 +2048,76 @@ static picture_t *vout_new_buffer( decoder_t *p_dec )
                      p_dec->fmt_out.video.i_sar_den, 50000 );
 
         p_dec->fmt_out.video.i_chroma = p_dec->fmt_out.i_codec;
-        p_sys->video = p_dec->fmt_out.video;
+        p_owner->video = p_dec->fmt_out.video;
+
+        vlc_mutex_lock( &p_owner->lock );
+
+        DecoderFlushBuffering( p_dec );
+
+        p_vout = p_owner->p_vout;
+        p_owner->p_vout = NULL;
+        vlc_mutex_unlock( &p_owner->lock );
 
-        p_sys->p_vout = vout_Request( p_dec, p_sys->p_vout,
-                                      &p_dec->fmt_out.video );
-        if( p_sys->p_vout == NULL )
+        p_vout = vout_Request( p_dec, p_vout, &p_dec->fmt_out.video );
+
+        vlc_mutex_lock( &p_owner->lock );
+        p_owner->p_vout = p_vout;
+        vlc_mutex_unlock( &p_owner->lock );
+
+        var_SetBool( p_owner->p_input, "intf-change-vout", true );
+        if( p_vout == NULL )
         {
             msg_Err( p_dec, "failed to create video output" );
-            p_dec->b_error = VLC_TRUE;
+            p_dec->b_error = true;
             return NULL;
         }
 
-        if( p_sys->video.i_rmask )
-            p_sys->p_vout->render.i_rmask = p_sys->video.i_rmask;
-        if( p_sys->video.i_gmask )
-            p_sys->p_vout->render.i_gmask = p_sys->video.i_gmask;
-        if( p_sys->video.i_bmask )
-            p_sys->p_vout->render.i_bmask = p_sys->video.i_bmask;
+        if( p_owner->video.i_rmask )
+            p_owner->p_vout->render.i_rmask = p_owner->video.i_rmask;
+        if( p_owner->video.i_gmask )
+            p_owner->p_vout->render.i_gmask = p_owner->video.i_gmask;
+        if( p_owner->video.i_bmask )
+            p_owner->p_vout->render.i_bmask = p_owner->video.i_bmask;
     }
 
-    /* Get a new picture */
-    while( !(p_pic = vout_CreatePicture( p_sys->p_vout, 0, 0, 0 ) ) )
+    /* Get a new picture
+     */
+    for( ;; )
     {
-        int i_pic, i_ready_pic = 0;
+        picture_t *p_picture;
 
         if( p_dec->b_die || p_dec->b_error )
-        {
             return NULL;
-        }
 
-#define p_pic p_dec->p_owner->p_vout->render.pp_picture[i_pic]
-        /* Check the decoder doesn't leak pictures */
-        for( i_pic = 0; i_pic < p_dec->p_owner->p_vout->render.i_pictures;
-             i_pic++ )
+        /* The video filter chain required that there is always 1 free buffer
+         * that it will use as temporary one. It will release the temporary
+         * buffer once its work is done, so this check is safe even if we don't
+         * lock around both count() and create().
+         */
+        if( vout_CountPictureAvailable( p_owner->p_vout ) >= 2 )
         {
-            if( p_pic->i_status == READY_PICTURE )
-            {
-                if( i_ready_pic++ > 0 ) break;
-                else continue;
-            }
+            p_picture = vout_CreatePicture( p_owner->p_vout, 0, 0, 0 );
+            if( p_picture )
+                return p_picture;
+        }
 
-            if( p_pic->i_status != DISPLAYED_PICTURE &&
-                p_pic->i_status != RESERVED_PICTURE &&
-                p_pic->i_status != READY_PICTURE ) break;
+        if( DecoderIsFlushing( p_dec ) )
+            return NULL;
 
-            if( !p_pic->i_refcount && p_pic->i_status != RESERVED_PICTURE )
-                break;
-        }
-        if( i_pic == p_dec->p_owner->p_vout->render.i_pictures )
-        {
-            msg_Err( p_dec, "decoder is leaking pictures, resetting the heap" );
+        /* */
+        DecoderSignalBuffering( p_dec, true );
 
-            /* Just free all the pictures */
-            for( i_pic = 0; i_pic < p_dec->p_owner->p_vout->render.i_pictures;
-                 i_pic++ )
-            {
-                if( p_pic->i_status == RESERVED_PICTURE )
-                    vout_DestroyPicture( p_dec->p_owner->p_vout, p_pic );
-                if( p_pic->i_refcount > 0 )
-                vout_UnlinkPicture( p_dec->p_owner->p_vout, p_pic );
-            }
-        }
-#undef p_pic
+        /* Check the decoder doesn't leak pictures */
+        vout_FixLeaks( p_owner->p_vout, false );
 
+        /* FIXME add a vout_WaitPictureAvailable (timedwait) */
         msleep( VOUT_OUTMEM_SLEEP );
     }
-
-    return p_pic;
 }
 
 static void vout_del_buffer( decoder_t *p_dec, picture_t *p_pic )
 {
-    VoutDisplayedPicture( p_dec->p_owner->p_vout, p_pic );
+    vout_DropPicture( p_dec->p_owner->p_vout, p_pic );
 }
 
 static void vout_link_picture( decoder_t *p_dec, picture_t *p_pic )
@@ -1254,17 +2132,19 @@ static void vout_unlink_picture( decoder_t *p_dec, picture_t *p_pic )
 
 static subpicture_t *spu_new_buffer( decoder_t *p_dec )
 {
-    decoder_owner_sys_t *p_sys = (decoder_owner_sys_t *)p_dec->p_owner;
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
     vout_thread_t *p_vout = NULL;
     subpicture_t *p_subpic;
     int i_attempts = 30;
 
     while( i_attempts-- )
     {
-        if( p_dec->b_die || p_dec->b_error ) break;
+        if( p_dec->b_die || p_dec->b_error )
+            break;
 
         p_vout = vlc_object_find( p_dec, VLC_OBJECT_VOUT, FIND_ANYWHERE );
-        if( p_vout ) break;
+        if( p_vout )
+            break;
 
         msleep( VOUT_DISPLAY_DELAY );
     }
@@ -1275,17 +2155,26 @@ static subpicture_t *spu_new_buffer( decoder_t *p_dec )
         return NULL;
     }
 
-    if( p_sys->p_spu_vout != p_vout )
+    if( p_owner->p_spu_vout != p_vout )
     {
+        vlc_mutex_lock( &p_owner->lock );
+
+        DecoderFlushBuffering( p_dec );
+
+        vlc_mutex_unlock( &p_owner->lock );
+
         spu_Control( p_vout->p_spu, SPU_CHANNEL_REGISTER,
-                     &p_sys->i_spu_channel );
-        p_sys->p_spu_vout = p_vout;
+                     &p_owner->i_spu_channel );
+        p_owner->i_spu_order = 0;
+        p_owner->p_spu_vout = p_vout;
     }
 
-    p_subpic = spu_CreateSubpicture( p_vout->p_spu );
+    p_subpic = subpicture_New();
     if( p_subpic )
     {
-        p_subpic->i_channel = p_sys->i_spu_channel;
+        p_subpic->i_channel = p_owner->i_spu_channel;
+        p_subpic->i_order = p_owner->i_spu_order++;
+        p_subpic->b_subtitle = true;
     }
 
     vlc_object_release( p_vout );
@@ -1295,11 +2184,11 @@ static subpicture_t *spu_new_buffer( decoder_t *p_dec )
 
 static void spu_del_buffer( decoder_t *p_dec, subpicture_t *p_subpic )
 {
-    decoder_owner_sys_t *p_sys = (decoder_owner_sys_t *)p_dec->p_owner;
+    decoder_owner_sys_t *p_owner = p_dec->p_owner;
     vout_thread_t *p_vout = NULL;
 
     p_vout = vlc_object_find( p_dec, VLC_OBJECT_VOUT, FIND_ANYWHERE );
-    if( !p_vout || p_sys->p_spu_vout != p_vout )
+    if( !p_vout || p_owner->p_spu_vout != p_vout )
     {
         if( p_vout )
             vlc_object_release( p_vout );
@@ -1307,7 +2196,7 @@ static void spu_del_buffer( decoder_t *p_dec, subpicture_t *p_subpic )
         return;
     }
 
-    spu_DestroySubpicture( p_vout->p_spu, p_subpic );
+    subpicture_Delete( p_subpic );
 
     vlc_object_release( p_vout );
 }