]> git.sesse.net Git - vlc/blobdiff - src/video_parser/video_parser.c
The motion compensation routines are now modules as well ; choose your
[vlc] / src / video_parser / video_parser.c
index 4515a78e4397e0c996c31a20ea87561b978fe5f7..ccc0c28685a3e759409e44d1f7d5433fac99c795 100644 (file)
@@ -2,8 +2,10 @@
  * video_parser.c : video parser thread
  *****************************************************************************
  * Copyright (C) 1999, 2000 VideoLAN
+ * $Id: video_parser.c,v 1.69 2001/01/18 05:13:23 sam Exp $
  *
- * Authors:
+ * Authors: Christophe Massiot <massiot@via.ecp.fr>
+ *          Samuel Hocevar <sam@via.ecp.fr>
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -20,8 +22,6 @@
  * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111, USA.
  *****************************************************************************/
 
-/* FIXME: passer en terminate/destroy avec les signaux supplĂ©mentaires ?? */
-
 /*****************************************************************************
  * Preamble
  *****************************************************************************/
 
 #include <stdlib.h>                                      /* malloc(), free() */
 #include <unistd.h>                                              /* getpid() */
-#include <sys/types.h>                        /* on BSD, uio.h needs types.h */
-#include <sys/uio.h>                                            /* "input.h" */
+#include <errno.h>
+#include <string.h>
+
+#ifdef STATS
+#  include <sys/times.h>
+#endif
 
 #include "config.h"
 #include "common.h"
 #include "threads.h"
 #include "mtime.h"
 #include "plugins.h"
+#include "modules.h"
 
 #include "intf_msg.h"
-#include "debug.h"                 /* XXX?? temporaire, requis par netlist.h */
 
-#include "input.h"
-#include "input_netlist.h"
-#include "decoder_fifo.h"
+#include "stream_control.h"
+#include "input_ext-dec.h"
+
 #include "video.h"
 #include "video_output.h"
 
-#include "vdec_idct.h"
 #include "video_decoder.h"
 #include "vdec_motion.h"
+#include "../video_decoder/vdec_idct.h"
 
 #include "vpar_blocks.h"
-#include "vpar_headers.h"
-#include "vpar_synchro.h"
-#include "video_parser.h"
-#include "video_fifo.h"
+#include "../video_decoder/vpar_headers.h"
+#include "../video_decoder/vpar_synchro.h"
+#include "../video_decoder/video_parser.h"
+#include "../video_decoder/video_fifo.h"
+
+#include "main.h"
 
 /*
  * Local prototypes
  */
-//static int      CheckConfiguration  ( video_cfg_t *p_cfg );
 static int      InitThread          ( vpar_thread_t *p_vpar );
 static void     RunThread           ( vpar_thread_t *p_vpar );
 static void     ErrorThread         ( vpar_thread_t *p_vpar );
 static void     EndThread           ( vpar_thread_t *p_vpar );
-static int      SynchroType         ( );
+static void     BitstreamCallback   ( bit_stream_t *p_bit_stream,
+                                      boolean_t b_new_pes );
 
 /*****************************************************************************
  * vpar_CreateThread: create a generic parser thread
  *****************************************************************************
  * This function creates a new video parser thread, and returns a pointer
  * to its description. On error, it returns NULL.
- * Following configuration properties are used:
- * XXX??
  *****************************************************************************/
-#include "main.h"
-#include "interface.h"
-extern main_t* p_main;
-vpar_thread_t * vpar_CreateThread( /* video_cfg_t *p_cfg, */ input_thread_t *p_input /*,
-                                   vout_thread_t *p_vout, int *pi_status */ )
+vlc_thread_t vpar_CreateThread( vdec_config_t * p_config )
 {
     vpar_thread_t *     p_vpar;
 
-    intf_DbgMsg( "vpar debug: creating video parser thread\n" );
+    intf_DbgMsg( "vpar debug: creating video parser thread" );
 
     /* Allocate the memory needed to store the thread's structure */
     if ( (p_vpar = (vpar_thread_t *)malloc( sizeof(vpar_thread_t) )) == NULL )
     {
         intf_ErrMsg( "vpar error: not enough memory "
-                     "for vpar_CreateThread() to create the new thread\n");
-        return( NULL );
+                     "for vpar_CreateThread() to create the new thread");
+        return( 0 );
     }
 
     /*
      * Initialize the thread properties
      */
-    p_vpar->b_die = 0;
-    p_vpar->b_error = 0;
+    p_vpar->p_fifo = p_config->decoder_config.p_decoder_fifo;
+    p_vpar->p_config = p_config;
+
+    p_vpar->p_vout = p_config->p_vout;
 
     /*
-     * Initialize the input properties
+     * Choose the best motion compensation module
      */
-    /* Initialize the decoder fifo's data lock and conditional variable
-     * and set its buffer as empty */
-    vlc_mutex_init( &p_vpar->fifo.data_lock );
-    vlc_cond_init( &p_vpar->fifo.data_wait );
-    p_vpar->fifo.i_start = 0;
-    p_vpar->fifo.i_end = 0;
-    /* Initialize the bit stream structure */
-    p_vpar->bit_stream.p_input = p_input;
-    p_vpar->bit_stream.p_decoder_fifo = &p_vpar->fifo;
-    p_vpar->bit_stream.fifo.buffer = 0;
-    p_vpar->bit_stream.fifo.i_available = 0;
-
-    /* FIXME !!!!?? */
-    p_vpar->p_vout = p_main->p_intf->p_vout;
+    p_vpar->p_motion_module = module_Need( p_main->p_module_bank,
+                                           MODULE_CAPABILITY_MOTION, NULL );
+
+    if( p_vpar->p_motion_module == NULL )
+    {
+        intf_ErrMsg( "vpar error: no suitable motion compensation module" );
+        free( p_vpar );
+        return( 0 );
+    }
+
+#define m ( p_vpar->pppf_motion )
+#define s ( p_vpar->ppf_motion_skipped )
+#define f ( p_vpar->p_motion_module->p_functions->motion.functions.motion )
+    m[0][0][0] = m[0][0][1] = m[0][0][2] = m[0][0][3] = NULL;
+    m[0][1][0] = m[0][1][1] = m[0][1][2] = m[0][1][3] = NULL;
+    m[1][0][0] = NULL;
+    m[1][1][0] = NULL;
+    m[2][0][0] = NULL;
+    m[2][1][0] = NULL;
+    m[3][0][0] = NULL;
+    m[3][1][0] = NULL;
+
+    m[1][0][1] = f.pf_field_field_420;
+    m[1][1][1] = f.pf_frame_field_420;
+    m[2][0][1] = f.pf_field_field_422;
+    m[2][1][1] = f.pf_frame_field_422;
+    m[3][0][1] = f.pf_field_field_444;
+    m[3][1][1] = f.pf_frame_field_444;
+
+    m[1][0][2] = f.pf_field_16x8_420;
+    m[1][1][2] = f.pf_frame_frame_420;
+    m[2][0][2] = f.pf_field_16x8_422;
+    m[2][1][2] = f.pf_frame_frame_422;
+    m[3][0][2] = f.pf_field_16x8_444;
+    m[3][1][2] = f.pf_frame_frame_444;
+
+    m[1][0][3] = f.pf_field_dmv_420;
+    m[1][1][3] = f.pf_frame_dmv_420;
+    m[2][0][3] = f.pf_field_dmv_422;
+    m[2][1][3] = f.pf_frame_dmv_422;
+    m[3][0][3] = f.pf_field_dmv_444;
+    m[3][1][3] = f.pf_frame_dmv_444;
+
+    s[0][0] = s[0][1] = s[0][2] = s[0][3] = NULL;
+    s[1][0] = NULL;
+    s[2][0] = NULL;
+    s[3][0] = NULL;
+
+    s[1][1] = f.pf_field_field_420;
+    s[2][1] = f.pf_field_field_422;
+    s[3][1] = f.pf_field_field_444;
+
+    s[1][2] = f.pf_field_field_420;
+    s[2][2] = f.pf_field_field_422;
+    s[3][2] = f.pf_field_field_444;
+
+    s[1][3] = f.pf_frame_frame_420;
+    s[2][3] = f.pf_frame_frame_422;
+    s[3][3] = f.pf_frame_frame_444;
+#undef f
+#undef s
+#undef m
+
+     /*
+      * Choose the best IDCT module
+      */
+    p_vpar->p_idct_module = module_Need( p_main->p_module_bank,
+                                         MODULE_CAPABILITY_IDCT, NULL );
+
+    if( p_vpar->p_idct_module == NULL )
+    {
+        intf_ErrMsg( "vpar error: no suitable IDCT module" );
+        module_Unneed( p_main->p_module_bank, p_vpar->p_motion_module );
+        free( p_vpar );
+        return( 0 );
+    }
+
+#define f p_vpar->p_idct_module->p_functions->idct.functions.idct
+    p_vpar->pf_init         = f.pf_init;
+    p_vpar->pf_sparse_idct  = f.pf_sparse_idct;
+    p_vpar->pf_idct         = f.pf_idct;
+    p_vpar->pf_norm_scan    = f.pf_norm_scan;
+#undef f
 
     /* Spawn the video parser thread */
     if ( vlc_thread_create( &p_vpar->thread_id, "video parser",
                             (vlc_thread_func_t)RunThread, (void *)p_vpar ) )
     {
-        intf_ErrMsg("vpar error: can't spawn video parser thread\n");
+        intf_ErrMsg("vpar error: can't spawn video parser thread");
+        module_Unneed( p_main->p_module_bank, p_vpar->p_idct_module );
+        module_Unneed( p_main->p_module_bank, p_vpar->p_motion_module );
         free( p_vpar );
-        return( NULL );
+        return( 0 );
     }
 
-    intf_DbgMsg("vpar debug: video parser thread (%p) created\n", p_vpar);
-    return( p_vpar );
-}
-
-/*****************************************************************************
- * vpar_DestroyThread: destroy a generic parser thread
- *****************************************************************************
- * Destroy a terminated thread. This function will return 0 if the thread could
- * be destroyed, and non 0 else. The last case probably means that the thread
- * was still active, and another try may succeed.
- *****************************************************************************/
-void vpar_DestroyThread( vpar_thread_t *p_vpar /*, int *pi_status */ )
-{
-    intf_DbgMsg( "vpar debug: requesting termination of "
-                 "video parser thread %p\n", p_vpar);
-
-    /* Ask thread to kill itself */
-    p_vpar->b_die = 1;
-    /* Make sure the parser thread leaves the GetByte() function */
-    vlc_mutex_lock( &(p_vpar->fifo.data_lock) );
-    vlc_cond_signal( &(p_vpar->fifo.data_wait) );
-    vlc_mutex_unlock( &(p_vpar->fifo.data_lock) );
-
-    /* Waiting for the parser thread to exit */
-    /* Remove this as soon as the "status" flag is implemented */
-    vlc_thread_join( p_vpar->thread_id );
+    intf_DbgMsg("vpar debug: video parser thread (%p) created", p_vpar);
+    return( p_vpar->thread_id );
 }
 
 /* following functions are local */
 
-/*****************************************************************************
- * CheckConfiguration: check vpar_CreateThread() configuration
- *****************************************************************************
- * Set default parameters where required. In DEBUG mode, check if configuration
- * is valid.
- *****************************************************************************/
-#if 0
-static int CheckConfiguration( video_cfg_t *p_cfg )
-{
-    /* XXX?? */
-
-    return( 0 );
-}
-#endif
-
 /*****************************************************************************
  * InitThread: initialize vpar output thread
  *****************************************************************************
@@ -184,26 +218,12 @@ static int InitThread( vpar_thread_t *p_vpar )
     int i_dummy;
 #endif
 
-    intf_DbgMsg("vpar debug: initializing video parser thread %p\n", p_vpar);
+    intf_DbgMsg("vpar debug: initializing video parser thread %p", p_vpar);
 
-    /* Our first job is to initialize the bit stream structure with the
-     * beginning of the input stream */
-    vlc_mutex_lock( &p_vpar->fifo.data_lock );
-    while ( DECODER_FIFO_ISEMPTY(p_vpar->fifo) )
-    {
-        if ( p_vpar->b_die )
-        {
-            vlc_mutex_unlock( &p_vpar->fifo.data_lock );
-            return( 1 );
-        }
-        vlc_cond_wait( &p_vpar->fifo.data_wait, &p_vpar->fifo.data_lock );
-    }
-    p_vpar->bit_stream.p_ts = DECODER_FIFO_START( p_vpar->fifo )->p_first_ts;
-    p_vpar->bit_stream.p_byte = p_vpar->bit_stream.p_ts->buffer
-                                + p_vpar->bit_stream.p_ts->i_payload_start;
-    p_vpar->bit_stream.p_end = p_vpar->bit_stream.p_ts->buffer
-                               + p_vpar->bit_stream.p_ts->i_payload_end;
-    vlc_mutex_unlock( &p_vpar->fifo.data_lock );
+    p_vpar->p_config->decoder_config.pf_init_bit_stream( &p_vpar->bit_stream,
+        p_vpar->p_config->decoder_config.p_decoder_fifo );
+    p_vpar->bit_stream.pf_bitstream_callback = BitstreamCallback;
+    p_vpar->bit_stream.p_callback_arg = (void *)p_vpar;
 
     /* Initialize parsing data */
     p_vpar->sequence.p_forward = NULL;
@@ -212,6 +232,8 @@ static int InitThread( vpar_thread_t *p_vpar )
     p_vpar->sequence.nonintra_quant.b_allocated = 0;
     p_vpar->sequence.chroma_intra_quant.b_allocated = 0;
     p_vpar->sequence.chroma_nonintra_quant.b_allocated = 0;
+    p_vpar->sequence.i_matrix_coefficients = 1;
+    p_vpar->sequence.next_pts = p_vpar->sequence.next_dts = 0;
 
     /* Initialize copyright information */
     p_vpar->sequence.b_copyright_flag = 0;
@@ -225,15 +247,11 @@ static int InitThread( vpar_thread_t *p_vpar )
     /* Initialize other properties */
 #ifdef STATS
     p_vpar->c_loops = 0;
-    p_vpar->c_idle_loops = 0;
-    p_vpar->c_pictures = 0;
-    p_vpar->c_i_pictures = 0;
-    p_vpar->c_p_pictures = 0;
-    p_vpar->c_b_pictures = 0;
-    p_vpar->c_decoded_pictures = 0;
-    p_vpar->c_decoded_i_pictures = 0;
-    p_vpar->c_decoded_p_pictures = 0;
-    p_vpar->c_decoded_b_pictures = 0;
+    p_vpar->c_sequences = 0;
+    memset(p_vpar->pc_pictures, 0, sizeof(p_vpar->pc_pictures));
+    memset(p_vpar->pc_decoded_pictures, 0, sizeof(p_vpar->pc_decoded_pictures));
+    memset(p_vpar->pc_malformed_pictures, 0,
+           sizeof(p_vpar->pc_malformed_pictures));
 #endif
 
     /* Initialize video FIFO */
@@ -261,6 +279,15 @@ static int InitThread( vpar_thread_t *p_vpar )
     p_vpar->pp_vdec[0]->b_die = 0;
     p_vpar->pp_vdec[0]->b_error = 0;
     p_vpar->pp_vdec[0]->p_vpar = p_vpar;
+
+#   if VDEC_NICE
+    /* Re-nice ourself */
+    if( nice(VDEC_NICE) == -1 )
+    {
+        intf_WarnMsg( 2, "vpar warning : couldn't nice() (%s)",
+                      strerror(errno) );
+    }
+#   endif
 #endif
 
     /* Initialize lookup tables */
@@ -272,61 +299,15 @@ static int InitThread( vpar_thread_t *p_vpar )
     vpar_InitPMBType( p_vpar );
     vpar_InitBMBType( p_vpar );
     vpar_InitDCTTables( p_vpar );
+    vpar_InitScanTable( p_vpar );
 
     /*
      * Initialize the synchro properties
      */
-#ifdef SAM_SYNCHRO
-    /* Get an possible synchro algorithm */
-    p_vpar->synchro.i_type = SynchroType();
-
-    /* last seen PTS */
-    p_vpar->synchro.i_last_pts = 0;
-
-    /* for i frames */
-    p_vpar->synchro.i_last_seen_I_pts = 0;
-    p_vpar->synchro.i_last_kept_I_pts = 0;
-
-    /* the fifo */
-    p_vpar->synchro.i_start  = 0;
-    p_vpar->synchro.i_stop   = 0;
-
-    /* mean decoding time - at least 200 ms for a slow machine */
-    p_vpar->synchro.i_delay            = 200000;
-    p_vpar->synchro.i_theorical_delay  = 40000; /* 25 fps */
-    /* assume we can display all Is and 2 Ps */
-    p_vpar->synchro.b_all_I = 1 << 10;
-    p_vpar->synchro.b_all_P = 0;
-    p_vpar->synchro.displayable_p = 2 << 10;
-    p_vpar->synchro.b_all_B = 0;
-    p_vpar->synchro.displayable_b = 0;
-    p_vpar->synchro.b_dropped_last_B = 0;
-    /* assume there were about 3 P and 6 B images between I's */
-    p_vpar->synchro.i_P_seen = p_vpar->synchro.i_P_kept = 1 << 10;
-    p_vpar->synchro.i_B_seen = p_vpar->synchro.i_B_kept = 1 << 10;
-#endif
-
-#ifdef MEUUH_SYNCHRO
-    p_vpar->synchro.kludge_level = 5;
-    p_vpar->synchro.kludge_nbp = p_vpar->synchro.kludge_p = 5;
-    p_vpar->synchro.kludge_nbb = p_vpar->synchro.kludge_b = 6;
-    p_vpar->synchro.kludge_b = 0;
-    p_vpar->synchro.kludge_prevdate = 0;
-#endif
-
-#ifdef POLUX_SYNCHRO
-    p_vpar->synchro.i_current_frame_date = 0;
-    p_vpar->synchro.i_backward_frame_date = 0;
-
-    p_vpar->synchro.r_p_average = p_vpar->synchro.i_p_nb = 6;
-    p_vpar->synchro.r_b_average = p_vpar->synchro.i_b_nb = 6;
-    p_vpar->synchro.i_p_count = 0;
-    p_vpar->synchro.i_b_count = 0;
-    p_vpar->synchro.i_i_count = 0;
-#endif
+    vpar_SynchroInit( p_vpar );
 
     /* Mark thread as running and return */
-    intf_DbgMsg("vpar debug: InitThread(%p) succeeded\n", p_vpar);
+    intf_DbgMsg("vpar debug: InitThread(%p) succeeded", p_vpar);
     return( 0 );
 }
 
@@ -338,30 +319,27 @@ static int InitThread( vpar_thread_t *p_vpar )
  *****************************************************************************/
 static void RunThread( vpar_thread_t *p_vpar )
 {
-    intf_DbgMsg("vpar debug: running video parser thread (%p) (pid == %i)\n", p_vpar, getpid());
+    intf_DbgMsg("vpar debug: running video parser thread (%p) (pid == %i)", p_vpar, getpid());
 
     /*
      * Initialize thread
      */
-    p_vpar->b_error = InitThread( p_vpar );
-
-    p_vpar->b_run = 1;
+    p_vpar->p_fifo->b_error = InitThread( p_vpar );
 
     /*
      * Main loop - it is not executed if an error occured during
      * initialization
      */
-    while( (!p_vpar->b_die) && (!p_vpar->b_error) )
+    while( (!p_vpar->p_fifo->b_die) && (!p_vpar->p_fifo->b_error) )
     {
         /* Find the next sequence header in the stream */
-        p_vpar->b_error = vpar_NextSequenceHeader( p_vpar );
+        p_vpar->p_fifo->b_error = vpar_NextSequenceHeader( p_vpar );
 
+        while( (!p_vpar->p_fifo->b_die) && (!p_vpar->p_fifo->b_error) )
+        {
 #ifdef STATS
-        p_vpar->c_sequences++;
+            p_vpar->c_loops++;
 #endif
-
-        while( (!p_vpar->b_die) && (!p_vpar->b_error) )
-        {
             /* Parse the next sequence, group or picture header */
             if( vpar_ParseHeader( p_vpar ) )
             {
@@ -374,13 +352,11 @@ static void RunThread( vpar_thread_t *p_vpar )
     /*
      * Error loop
      */
-    if( p_vpar->b_error )
+    if( p_vpar->p_fifo->b_error )
     {
         ErrorThread( p_vpar );
     }
 
-    p_vpar->b_run = 0;
-
     /* End of thread */
     EndThread( p_vpar );
 }
@@ -396,25 +372,25 @@ static void ErrorThread( vpar_thread_t *p_vpar )
 {
     /* We take the lock, because we are going to read/write the start/end
      * indexes of the decoder fifo */
-    vlc_mutex_lock( &p_vpar->fifo.data_lock );
+    vlc_mutex_lock( &p_vpar->p_fifo->data_lock );
 
     /* Wait until a `die' order is sent */
-    while( !p_vpar->b_die )
+    while( !p_vpar->p_fifo->b_die )
     {
         /* Trash all received PES packets */
-        while( !DECODER_FIFO_ISEMPTY(p_vpar->fifo) )
+        while( !DECODER_FIFO_ISEMPTY(*p_vpar->p_fifo) )
         {
-            input_NetlistFreePES( p_vpar->bit_stream.p_input,
-                                  DECODER_FIFO_START(p_vpar->fifo) );
-            DECODER_FIFO_INCSTART( p_vpar->fifo );
+            p_vpar->p_fifo->pf_delete_pes( p_vpar->p_fifo->p_packets_mgt,
+                                  DECODER_FIFO_START(*p_vpar->p_fifo) );
+            DECODER_FIFO_INCSTART( *p_vpar->p_fifo );
         }
 
         /* Waiting for the input thread to put new PES packets in the fifo */
-        vlc_cond_wait( &p_vpar->fifo.data_wait, &p_vpar->fifo.data_lock );
+        vlc_cond_wait( &p_vpar->p_fifo->data_wait, &p_vpar->p_fifo->data_lock );
     }
 
     /* We can release the lock before leaving */
-    vlc_mutex_unlock( &p_vpar->fifo.data_lock );
+    vlc_mutex_unlock( &p_vpar->p_fifo->data_lock );
 }
 
 /*****************************************************************************
@@ -429,16 +405,62 @@ static void EndThread( vpar_thread_t *p_vpar )
     int i_dummy;
 #endif
 
-    intf_DbgMsg("vpar debug: destroying video parser thread %p\n", p_vpar);
+    intf_DbgMsg("vpar debug: destroying video parser thread %p", p_vpar);
 
-#ifdef DEBUG
-    /* Check for remaining PES packets */
-    /* XXX?? */
-#endif
+    /* Release used video buffers. */
+    if( p_vpar->sequence.p_forward != NULL )
+    {
+        vout_UnlinkPicture( p_vpar->p_vout, p_vpar->sequence.p_forward );
+    }
+    if( p_vpar->sequence.p_backward != NULL )
+    {
+        vout_DatePicture( p_vpar->p_vout, p_vpar->sequence.p_backward,
+                          vpar_SynchroDate( p_vpar ) );
+        vout_UnlinkPicture( p_vpar->p_vout, p_vpar->sequence.p_backward );
+    }
 
-    /* Destroy thread structures allocated by InitThread */
-//    vout_DestroyStream( p_vpar->p_vout, p_vpar->i_stream );
-    /* XXX?? */
+#ifdef STATS
+    intf_Msg("vpar stats: %d loops among %d sequence(s)",
+             p_vpar->c_loops, p_vpar->c_sequences);
+
+    {
+        struct tms cpu_usage;
+        times( &cpu_usage );
+
+        intf_Msg("vpar stats: cpu usage (user: %d, system: %d)",
+                 cpu_usage.tms_utime, cpu_usage.tms_stime);
+    }
+
+    intf_Msg("vpar stats: Read %d frames/fields (I %d/P %d/B %d)",
+             p_vpar->pc_pictures[I_CODING_TYPE]
+             + p_vpar->pc_pictures[P_CODING_TYPE]
+             + p_vpar->pc_pictures[B_CODING_TYPE],
+             p_vpar->pc_pictures[I_CODING_TYPE],
+             p_vpar->pc_pictures[P_CODING_TYPE],
+             p_vpar->pc_pictures[B_CODING_TYPE]);
+    intf_Msg("vpar stats: Decoded %d frames/fields (I %d/P %d/B %d)",
+             p_vpar->pc_decoded_pictures[I_CODING_TYPE]
+             + p_vpar->pc_decoded_pictures[P_CODING_TYPE]
+             + p_vpar->pc_decoded_pictures[B_CODING_TYPE],
+             p_vpar->pc_decoded_pictures[I_CODING_TYPE],
+             p_vpar->pc_decoded_pictures[P_CODING_TYPE],
+             p_vpar->pc_decoded_pictures[B_CODING_TYPE]);
+    intf_Msg("vpar stats: Read %d malformed frames/fields (I %d/P %d/B %d)",
+             p_vpar->pc_malformed_pictures[I_CODING_TYPE]
+             + p_vpar->pc_malformed_pictures[P_CODING_TYPE]
+             + p_vpar->pc_malformed_pictures[B_CODING_TYPE],
+             p_vpar->pc_malformed_pictures[I_CODING_TYPE],
+             p_vpar->pc_malformed_pictures[P_CODING_TYPE],
+             p_vpar->pc_malformed_pictures[B_CODING_TYPE]);
+#define S   p_vpar->sequence
+    intf_Msg("vpar info: %s stream (%dx%d), %d.%d pi/s",
+             S.b_mpeg2 ? "MPEG-2" : "MPEG-1",
+             S.i_width, S.i_height, S.i_frame_rate/1001, S.i_frame_rate % 1001);
+    intf_Msg("vpar info: %s, %s, matrix_coeff: %d",
+             S.b_progressive ? "Progressive" : "Non-progressive",
+             S.i_scalable_mode ? "scalable" : "non-scalable",
+             S.i_matrix_coefficients);
+#endif
 
     /* Dispose of matrices if they have been allocated. */
     if( p_vpar->sequence.intra_quant.b_allocated )
@@ -471,61 +493,40 @@ static void EndThread( vpar_thread_t *p_vpar )
     free( p_vpar->pp_vdec[0] );
 #endif
 
+    free( p_vpar->p_config );
+
+#ifdef VDEC_SMP
+    /* Destroy lock and cond */
+    vlc_mutex_destroy( &(p_vpar->vbuffer.lock) );
+    vlc_cond_destroy( &(p_vpar->vfifo.wait) );
+    vlc_mutex_destroy( &(p_vpar->vfifo.lock) );
+#endif
+    
+    vlc_mutex_destroy( &(p_vpar->synchro.fifo_lock) );
+    
+    module_Unneed( p_main->p_module_bank, p_vpar->p_idct_module );
+    module_Unneed( p_main->p_module_bank, p_vpar->p_motion_module );
+
     free( p_vpar );
 
-    intf_DbgMsg("vpar debug: EndThread(%p)\n", p_vpar);
+    intf_DbgMsg("vpar debug: EndThread(%p)", p_vpar);
 }
 
 /*****************************************************************************
- * SynchroType: Get the user's synchro type
+ * BitstreamCallback: Import parameters from the new data/PES packet
  *****************************************************************************
- * This function is called at initialization.
+ * This function is called by input's NextDataPacket.
  *****************************************************************************/
-static int SynchroType( )
+static void BitstreamCallback ( bit_stream_t * p_bit_stream,
+                                boolean_t b_new_pes )
 {
-    char * psz_synchro = main_GetPszVariable( VPAR_SYNCHRO_VAR, NULL );
+    vpar_thread_t * p_vpar = (vpar_thread_t *)p_bit_stream->p_callback_arg;
 
-    if( psz_synchro == NULL )
+    if( b_new_pes )
     {
-        return VPAR_SYNCHRO_DEFAULT;
-    }
-
-    switch( *psz_synchro++ )
-    {
-      case 'i':
-      case 'I':
-        switch( *psz_synchro++ )
-        {
-          case '\0':
-            return VPAR_SYNCHRO_I;
-
-          case 'p':
-          case 'P':
-            switch( *psz_synchro++ )
-            {
-              case '\0':
-                return VPAR_SYNCHRO_IP;
-
-              case '+':
-                if( *psz_synchro ) return 0;
-                return VPAR_SYNCHRO_IPplus;
-
-              case 'b':
-              case 'B':
-                if( *psz_synchro ) return 0;
-                return VPAR_SYNCHRO_IPB;
-
-              default:
-                return VPAR_SYNCHRO_DEFAULT;
-                
-            }
-
-          default:
-            return VPAR_SYNCHRO_DEFAULT;
-        }
+        p_vpar->sequence.next_pts =
+            DECODER_FIFO_START( *p_bit_stream->p_decoder_fifo )->i_pts;
+        p_vpar->sequence.next_dts =
+            DECODER_FIFO_START( *p_bit_stream->p_decoder_fifo )->i_dts;
     }
-
-    return VPAR_SYNCHRO_DEFAULT;
-
 }
-