]> git.sesse.net Git - vlc/blobdiff - src/video_output/video_output.c
Join and unload vout safely
[vlc] / src / video_output / video_output.c
index 2d46908284fe053c8fb390ce2bad59d080e957c9..0fe1be372ff74940b235d13784281b7c2e0ff49a 100644 (file)
@@ -44,7 +44,6 @@
 #endif
 
 #include <vlc_vout.h>
-#include <vlc_playlist.h>
 
 #include <vlc_filter.h>
 #include <vlc_osd.h>
@@ -58,6 +57,7 @@
 #include "input/input_internal.h"
 
 #include "modules/modules.h"
+#include <assert.h>
 
 /*****************************************************************************
  * Local prototypes
@@ -65,6 +65,7 @@
 static int      InitThread        ( vout_thread_t * );
 static void     RunThread         ( vout_thread_t * );
 static void     ErrorThread       ( vout_thread_t * );
+static void     CleanThread       ( vout_thread_t * );
 static void     EndThread         ( vout_thread_t * );
 
 static void     AspectRatio       ( int, int *, int * );
@@ -84,25 +85,16 @@ static int VideoFilter2Callback( vlc_object_t *, char const *,
 /* From vout_intf.c */
 int vout_Snapshot( vout_thread_t *, picture_t * );
 
-/* Video filter2 parsing */
-static int ParseVideoFilter2Chain( vout_thread_t *, char * );
-static void RemoveVideoFilters2( vout_thread_t *p_vout );
-
 /* Display media title in OSD */
 static void DisplayTitleOnOSD( vout_thread_t *p_vout );
 
 /*****************************************************************************
  * Video Filter2 functions
  *****************************************************************************/
-struct filter_owner_sys_t
-{
-    vout_thread_t *p_vout;
-};
-
 static picture_t *video_new_buffer_filter( filter_t *p_filter )
 {
     picture_t *p_picture;
-    vout_thread_t *p_vout = p_filter->p_owner->p_vout;
+    vout_thread_t *p_vout = (vout_thread_t*)p_filter->p_owner;
 
     p_picture = vout_CreatePicture( p_vout, 0, 0, 0 );
 
@@ -111,7 +103,15 @@ static picture_t *video_new_buffer_filter( filter_t *p_filter )
 
 static void video_del_buffer_filter( filter_t *p_filter, picture_t *p_pic )
 {
-    vout_DestroyPicture( p_filter->p_owner->p_vout, p_pic );
+    vout_DestroyPicture( (vout_thread_t*)p_filter->p_owner, p_pic );
+}
+
+static int video_filter_buffer_allocation_init( filter_t *p_filter, void *p_data )
+{
+    p_filter->pf_vout_buffer_new = video_new_buffer_filter;
+    p_filter->pf_vout_buffer_del = video_del_buffer_filter;
+    p_filter->p_owner = p_data; /* p_vout */
+    return VLC_SUCCESS;
 }
 
 /*****************************************************************************
@@ -125,12 +125,16 @@ vout_thread_t *__vout_Request( vlc_object_t *p_this, vout_thread_t *p_vout,
 {
     if( !p_fmt )
     {
-        /* Reattach video output to playlist before bailing out */
+        /* Video output is no longer used.
+         * TODO: support for reusing video outputs with proper _thread-safe_
+         * reference handling. */
         if( p_vout )
         {
             spu_Attach( p_vout->p_spu, p_this, false );
-            vlc_object_detach( p_vout );
-            vlc_object_attach( p_vout, p_this->p_libvlc );
+            vlc_object_kill( p_vout );
+            vlc_thread_join( p_vout );
+            module_Unneed( p_vout, p_vout->p_module );
+            vlc_object_release( p_vout );
         }
         return NULL;
     }
@@ -140,24 +144,8 @@ vout_thread_t *__vout_Request( vlc_object_t *p_this, vout_thread_t *p_vout,
     {
         vlc_object_yield( p_vout );
     }
-    else
-    {
-        p_vout = vlc_object_find( p_this, VLC_OBJECT_VOUT, FIND_CHILD );
 
-        if( !p_vout )
-        {
-            p_vout = vlc_object_find( p_this->p_libvlc,
-                                      VLC_OBJECT_VOUT, FIND_CHILD );
-            /* only first children of p_input for unused vout */
-            if( p_vout && p_vout->p_parent != VLC_OBJECT(p_this->p_libvlc) )
-            {
-                vlc_object_release( p_vout );
-                p_vout = NULL;
-            }
-            if( p_vout )
-                vlc_object_detach( p_vout );    /* Remove it from the GC */
-        }
-    }
+    /* TODO: find a suitable unused video output */
 
     /* If we now have a video output, check it has the right properties */
     if( p_vout )
@@ -245,12 +233,11 @@ vout_thread_t * __vout_Create( vlc_object_t *p_parent, video_format_t *p_fmt )
     char *psz_name;
 
     /* Allocate descriptor */
-    p_vout = vlc_object_create( p_parent, VLC_OBJECT_VOUT );
+    static const char typename[] = "video output";
+    p_vout = vlc_custom_create( p_parent, sizeof( *p_vout ), VLC_OBJECT_VOUT,
+                                typename );
     if( p_vout == NULL )
-    {
-        msg_Err( p_parent, "out of memory" );
         return NULL;
-    }
 
     /* Initialize pictures - translation tables and functions
      * will be initialized later in InitThread */
@@ -279,9 +266,9 @@ vout_thread_t * __vout_Create( vlc_object_t *p_parent, video_format_t *p_fmt )
     p_vout->render.i_chroma   = i_chroma;
     p_vout->render.i_aspect   = i_aspect;
 
-    p_vout->render.i_rmask    = 0;
-    p_vout->render.i_gmask    = 0;
-    p_vout->render.i_bmask    = 0;
+    p_vout->render.i_rmask    = p_fmt->i_rmask;
+    p_vout->render.i_gmask    = p_fmt->i_gmask;
+    p_vout->render.i_bmask    = p_fmt->i_bmask;
 
     p_vout->render.i_last_used_pic = -1;
     p_vout->render.b_allow_modify_pics = 1;
@@ -309,8 +296,8 @@ vout_thread_t * __vout_Create( vlc_object_t *p_parent, video_format_t *p_fmt )
     p_vout->render_time  = 10;
     p_vout->c_fps_samples = 0;
     p_vout->b_filter_change = 0;
-    p_vout->pf_control = 0;
-    p_vout->p_parent_intf = 0;
+    p_vout->pf_control = NULL;
+    p_vout->p_window = NULL;
     p_vout->i_par_num = p_vout->i_par_den = 1;
 
     /* Initialize locks */
@@ -342,16 +329,12 @@ vout_thread_t * __vout_Create( vlc_object_t *p_parent, video_format_t *p_fmt )
     if( p_parent->i_object_type != VLC_OBJECT_VOUT )
     {
         /* Look for the default filter configuration */
-        var_Create( p_vout, "vout-filter", VLC_VAR_STRING | VLC_VAR_DOINHERIT );
-        var_Get( p_vout, "vout-filter", &val );
-        p_vout->psz_filter_chain = val.psz_string;
+        p_vout->psz_filter_chain =
+            var_CreateGetStringCommand( p_vout, "vout-filter" );
 
         /* Apply video filter2 objects on the first vout */
-        var_Create( p_vout, "video-filter",
-                    VLC_VAR_STRING | VLC_VAR_DOINHERIT );
-        var_Get( p_vout, "video-filter", &val );
-        ParseVideoFilter2Chain( p_vout, val.psz_string );
-        free( val.psz_string );
+        p_vout->psz_vf2 =
+            var_CreateGetStringCommand( p_vout, "video-filter" );
     }
     else
     {
@@ -367,13 +350,14 @@ vout_thread_t * __vout_Create( vlc_object_t *p_parent, video_format_t *p_fmt )
         free( psz_tmp );
 
         /* Create a video filter2 var ... but don't inherit values */
-        var_Create( p_vout, "video-filter", VLC_VAR_STRING );
-        ParseVideoFilter2Chain( p_vout, NULL );
+        var_Create( p_vout, "video-filter",
+                    VLC_VAR_STRING | VLC_VAR_ISCOMMAND );
+        p_vout->psz_vf2 = var_GetString( p_vout, "video-filter" );
     }
 
     var_AddCallback( p_vout, "video-filter", VideoFilter2Callback, NULL );
-    p_vout->b_vfilter_change = true;
-    p_vout->i_vfilters = 0;
+    p_vout->p_vf2_chain = filter_chain_New( p_vout, "video filter2",
+        false, video_filter_buffer_allocation_init, NULL, p_vout );
 
     /* Choose the video output module */
     if( !p_vout->psz_filter_chain || !*p_vout->psz_filter_chain )
@@ -399,6 +383,8 @@ vout_thread_t * __vout_Create( vlc_object_t *p_parent, video_format_t *p_fmt )
     if( p_vout->p_module == NULL )
     {
         msg_Err( p_vout, "no suitable vout module" );
+        // FIXME it's ugly but that's exactly the function that need to be called.
+        EndThread( p_vout );
         vlc_object_detach( p_vout );
         vlc_object_release( p_vout );
         return NULL;
@@ -451,23 +437,25 @@ vout_thread_t * __vout_Create( vlc_object_t *p_parent, video_format_t *p_fmt )
     if( vlc_thread_create( p_vout, "video output", RunThread,
                            VLC_THREAD_PRIORITY_OUTPUT, true ) )
     {
-        msg_Err( p_vout, "out of memory" );
         module_Unneed( p_vout, p_vout->p_module );
         vlc_object_release( p_vout );
         return NULL;
     }
 
+    vlc_object_set_destructor( p_vout, vout_Destructor );
+
     if( p_vout->b_error )
     {
         msg_Err( p_vout, "video output creation failed" );
 
-        /* Make sure the thread is destroyed */
+        /* Make sure the thread is destroyed and data released */
+        vlc_object_kill( p_vout );
+        vlc_thread_join( p_vout );
+        module_Unneed( p_vout, p_vout->p_module );
         vlc_object_release( p_vout );
         return NULL;
     }
 
-    vlc_object_set_destructor( p_vout, vout_Destructor );
-
     return p_vout;
 }
 
@@ -480,12 +468,6 @@ static void vout_Destructor( vlc_object_t * p_this )
     vlc_mutex_destroy( &p_vout->change_lock );
     vlc_mutex_destroy( &p_vout->vfilter_lock );
 
-    /* Release the module */
-    if( p_vout->p_module )
-    {
-        module_Unneed( p_vout, p_vout->p_module );
-    }
-
     free( p_vout->psz_filter_chain );
 
     config_ChainDestroy( p_vout->p_cfg );
@@ -513,37 +495,28 @@ static void vout_Destructor( vlc_object_t * p_this )
  * This function is called from RunThread and performs the second step of the
  * initialization. It returns 0 on success. Note that the thread's flag are not
  * modified inside this function.
+ * XXX You have to enter it with change_lock taken.
  *****************************************************************************/
-static picture_t *get_pic( filter_t *p_filter )
-{
-    picture_t *p_pic = (picture_t *)p_filter->p_owner;
-    p_filter->p_owner = NULL;
-    return p_pic;
-}
+static int ChromaCreate( vout_thread_t *p_vout );
+static void ChromaDestroy( vout_thread_t *p_vout );
 
 static int InitThread( vout_thread_t *p_vout )
 {
     int i, i_aspect_x, i_aspect_y;
 
-    vlc_mutex_lock( &p_vout->change_lock );
-
 #ifdef STATS
     p_vout->c_loops = 0;
 #endif
 
     /* Initialize output method, it allocates direct buffers for us */
     if( p_vout->pf_init( p_vout ) )
-    {
-        vlc_mutex_unlock( &p_vout->change_lock );
         return VLC_EGENERIC;
-    }
 
     if( !I_OUTPUTPICTURES )
     {
         msg_Err( p_vout, "plugin was unable to allocate at least "
                          "one direct buffer" );
         p_vout->pf_end( p_vout );
-        vlc_mutex_unlock( &p_vout->change_lock );
         return VLC_EGENERIC;
     }
 
@@ -552,7 +525,6 @@ static int InitThread( vout_thread_t *p_vout )
         msg_Err( p_vout, "plugin allocated too many direct buffers, "
                          "our internal buffers must have overflown." );
         p_vout->pf_end( p_vout );
-        vlc_mutex_unlock( &p_vout->change_lock );
         return VLC_EGENERIC;
     }
 
@@ -617,6 +589,13 @@ static int InitThread( vout_thread_t *p_vout )
              p_vout->fmt_out.i_sar_num, p_vout->fmt_out.i_sar_den );
 
     /* Calculate shifts from system-updated masks */
+    MaskToShift( &p_vout->render.i_lrshift, &p_vout->output.i_rrshift,
+                 p_vout->render.i_rmask );
+    MaskToShift( &p_vout->render.i_lgshift, &p_vout->output.i_rgshift,
+                 p_vout->render.i_gmask );
+    MaskToShift( &p_vout->render.i_lbshift, &p_vout->output.i_rbshift,
+                 p_vout->render.i_bmask );
+
     MaskToShift( &p_vout->output.i_lrshift, &p_vout->output.i_rrshift,
                  p_vout->output.i_rmask );
     MaskToShift( &p_vout->output.i_lgshift, &p_vout->output.i_rgshift,
@@ -661,40 +640,11 @@ static int InitThread( vout_thread_t *p_vout )
          * find a chroma plugin to do the conversion */
         p_vout->b_direct = 0;
 
-        /* Choose the best module */
-        p_vout->p_chroma = vlc_object_create( p_vout, VLC_OBJECT_FILTER );
-        filter_t *p_chroma = p_vout->p_chroma;
-        vlc_object_attach( p_chroma, p_vout );
-        /* TODO: Set the fmt_in and fmt_out stuff here */
-        p_chroma->fmt_in.video = p_vout->fmt_render;
-        p_chroma->fmt_out.video = p_vout->fmt_out;
-
-        /* TODO: put in a function */
-        p_chroma->fmt_out.video.i_rmask = p_vout->output.i_rmask;
-        p_chroma->fmt_out.video.i_gmask = p_vout->output.i_gmask;
-        p_chroma->fmt_out.video.i_bmask = p_vout->output.i_bmask;
-        p_chroma->fmt_out.video.i_rrshift = p_vout->output.i_rrshift;
-        p_chroma->fmt_out.video.i_lrshift = p_vout->output.i_lrshift;
-        p_chroma->fmt_out.video.i_rgshift = p_vout->output.i_rgshift;
-        p_chroma->fmt_out.video.i_lgshift = p_vout->output.i_lgshift;
-        p_chroma->fmt_out.video.i_rbshift = p_vout->output.i_rbshift;
-        p_chroma->fmt_out.video.i_lbshift = p_vout->output.i_lbshift;
-        msg_Err( p_vout, "HOLA! %4.4s", (char*)&p_chroma->fmt_in.video.i_chroma );
-        msg_Err( p_vout, "HOLA! %4.4s", (char*)&p_chroma->fmt_out.video.i_chroma );
-        p_chroma->p_module = module_Need( p_chroma, "video filter2", NULL, 0 );
-
-        if( p_chroma->p_module == NULL )
+        if( ChromaCreate( p_vout ) )
         {
-            msg_Err( p_vout, "no chroma module for %4.4s to %4.4s",
-                     (char*)&p_vout->render.i_chroma,
-                     (char*)&p_vout->output.i_chroma );
-            vlc_object_detach( p_vout->p_chroma );
-            p_vout->p_chroma = NULL;
             p_vout->pf_end( p_vout );
-            vlc_mutex_unlock( &p_vout->change_lock );
             return VLC_EGENERIC;
         }
-        p_chroma->pf_vout_buffer_new = get_pic;
 
         msg_Dbg( p_vout, "indirect render, mapping "
                  "render pictures 0-%i to system pictures %i-%i",
@@ -724,7 +674,6 @@ static int InitThread( vout_thread_t *p_vout )
         PP_OUTPUTPICTURE[ i ]->p_heap = &p_vout->output;
     }
 
-/* XXX XXX mark thread ready */
     return VLC_SUCCESS;
 }
 
@@ -758,6 +707,7 @@ static void RunThread( vout_thread_t *p_vout)
     /*
      * Initialize thread
      */
+    vlc_mutex_lock( &p_vout->change_lock );
     p_vout->b_error = InitThread( p_vout );
 
     var_Create( p_vout, "drop-late-frames", VLC_VAR_BOOL | VLC_VAR_DOINHERIT );
@@ -768,7 +718,11 @@ static void RunThread( vout_thread_t *p_vout)
     vlc_thread_ready( p_vout );
 
     if( p_vout->b_error )
+    {
+        EndThread( p_vout );
+        vlc_mutex_unlock( &p_vout->change_lock );
         return;
+    }
 
     vlc_object_lock( p_vout );
 
@@ -955,91 +909,10 @@ static void RunThread( vout_thread_t *p_vout)
             i_idle_loops++;
         }
 
-        /* Video Filter2 stuff */
-        if( p_vout->b_vfilter_change == true )
-        {
-            int i;
-            vlc_mutex_lock( &p_vout->vfilter_lock );
-            RemoveVideoFilters2( p_vout );
-            for( i = 0; i < p_vout->i_vfilters_cfg; i++ )
-            {
-                filter_t *p_vfilter =
-                    p_vout->pp_vfilters[p_vout->i_vfilters] =
-                        vlc_object_create( p_vout, VLC_OBJECT_FILTER );
-
-                vlc_object_attach( p_vfilter, p_vout );
-
-                p_vfilter->pf_vout_buffer_new = video_new_buffer_filter;
-                p_vfilter->pf_vout_buffer_del = video_del_buffer_filter;
-
-                if( !p_vout->i_vfilters )
-                {
-                    p_vfilter->fmt_in.video = p_vout->fmt_render;
-                }
-                else
-                {
-                    p_vfilter->fmt_in.video = (p_vfilter-1)->fmt_out.video;
-                }
-                /* TODO: one day filters in the middle of the chain might
-                 * have a different fmt_out.video than fmt_render ... */
-                p_vfilter->fmt_out.video = p_vout->fmt_render;
-
-                p_vfilter->p_cfg = p_vout->p_vfilters_cfg[i];
-                p_vfilter->p_module = module_Need( p_vfilter, "video filter2",
-                                                   p_vout->psz_vfilters[i],
-                                                   true );
-
-                if( p_vfilter->p_module )
-                {
-                    p_vfilter->p_owner =
-                        malloc( sizeof( filter_owner_sys_t ) );
-                    p_vfilter->p_owner->p_vout = p_vout;
-                    p_vout->i_vfilters++;
-                    msg_Dbg( p_vout, "video filter found (%s)",
-                             p_vout->psz_vfilters[i] );
-                }
-                else
-                {
-                    msg_Err( p_vout, "no video filter found (%s)",
-                             p_vout->psz_vfilters[i] );
-                    vlc_object_detach( p_vfilter );
-                    vlc_object_release( p_vfilter );
-                }
-            }
-            p_vout->b_vfilter_change = false;
-            vlc_mutex_unlock( &p_vout->vfilter_lock );
-        }
-
         if( p_picture )
         {
-            int i;
-            for( i = 0; i < p_vout->i_vfilters; i++ )
-            {
-                picture_t *p_old = p_picture;
-                p_picture  = p_vout->pp_vfilters[i]->pf_video_filter(
-                                 p_vout->pp_vfilters[i], p_picture );
-                if( !p_picture )
-                {
-                    break;
-                }
-                /* FIXME: this is kind of wrong
-                 * if you have 2 or more vfilters and the 2nd breaks,
-                 * on the next loop the 1st one will be applied again */
-
-                /* if p_old and p_picture are the same (ie the filter
-                 * worked on the old picture), then following code is
-                 * still alright since i_status gets changed back to
-                 * the right value */
-                if( p_old->i_refcount )
-                {
-                    p_old->i_status = DISPLAYED_PICTURE;
-                }
-                else
-                {
-                    p_old->i_status = DESTROYED_PICTURE;
-                }
-                p_picture->i_status = READY_PICTURE;
-            }
+            p_picture = filter_chain_VideoFilter( p_vout->p_vf2_chain,
+                                                  p_picture );
         }
 
         if( p_picture && p_vout->b_snapshot )
@@ -1156,7 +1029,9 @@ static void RunThread( vout_thread_t *p_vout)
             /* A fatal error occurred, and the thread must terminate
              * immediately, without displaying anything - setting b_error to 1
              * causes the immediate end of the main while() loop. */
+            // FIXME pf_end
             p_vout->b_error = 1;
+            break;
         }
 
         if( p_vout->i_changes & VOUT_SIZE_CHANGE )
@@ -1169,11 +1044,19 @@ static void RunThread( vout_thread_t *p_vout)
 
             p_vout->i_changes &= ~VOUT_SIZE_CHANGE;
 
+            assert( !p_vout->b_direct );
+
+            ChromaDestroy( p_vout );
+
+            vlc_mutex_lock( &p_vout->picture_lock );
+
             p_vout->pf_end( p_vout );
+
             for( i = 0; i < I_OUTPUTPICTURES; i++ )
                  p_vout->p_picture[ i ].i_status = FREE_PICTURE;
 
             I_OUTPUTPICTURES = 0;
+
             if( p_vout->pf_init( p_vout ) )
             {
                 msg_Err( p_vout, "cannot resize display" );
@@ -1181,13 +1064,19 @@ static void RunThread( vout_thread_t *p_vout)
                 p_vout->b_error = 1;
             }
 
-            /* Need to reinitialise the chroma plugin */
-            if( p_vout->p_chroma->p_module )
+            vlc_mutex_unlock( &p_vout->picture_lock );
+
+            /* Need to reinitialise the chroma plugin. Since we might need
+             * resizing too and it's not sure that we already had it,
+             * recreate the chroma plugin chain from scratch. */
+            /* dionoea */
+            if( ChromaCreate( p_vout ) )
             {
-                if( p_vout->p_chroma->p_module->pf_deactivate )
-                    p_vout->p_chroma->p_module->pf_deactivate( VLC_OBJECT(p_vout->p_chroma) );
-                p_vout->p_chroma->p_module->pf_activate( VLC_OBJECT(p_vout->p_chroma) );
+                msg_Err( p_vout, "WOW THIS SUCKS BIG TIME!!!!!" );
+                p_vout->b_error = 1;
             }
+            if( p_vout->b_error )
+                break;
         }
 
         if( p_vout->i_changes & VOUT_PICTURE_BUFFERS_CHANGE )
@@ -1200,10 +1089,7 @@ static void RunThread( vout_thread_t *p_vout)
             p_vout->i_changes &= ~VOUT_PICTURE_BUFFERS_CHANGE;
 
             if( !p_vout->b_direct )
-            {
-                module_Unneed( p_vout->p_chroma, p_vout->p_chroma->p_module );
-                p_vout->p_chroma = NULL;
-            }
+                ChromaDestroy( p_vout );
 
             vlc_mutex_lock( &p_vout->picture_lock );
 
@@ -1212,9 +1098,33 @@ static void RunThread( vout_thread_t *p_vout)
             I_OUTPUTPICTURES = I_RENDERPICTURES = 0;
 
             p_vout->b_error = InitThread( p_vout );
+            if( p_vout->b_error )
+                msg_Err( p_vout, "InitThread after VOUT_PICTURE_BUFFERS_CHANGE failed\n" );
 
             vlc_mutex_unlock( &p_vout->picture_lock );
+
+            if( p_vout->b_error )
+                break;
+        }
+
+        /* Check for "video filter2" changes */
+        vlc_mutex_lock( &p_vout->vfilter_lock );
+        if( p_vout->psz_vf2 )
+        {
+            es_format_t fmt;
+
+            es_format_Init( &fmt, VIDEO_ES, p_vout->fmt_render.i_chroma );
+            fmt.video = p_vout->fmt_render;
+            filter_chain_Reset( p_vout->p_vf2_chain, &fmt, &fmt );
+
+            if( filter_chain_AppendFromString( p_vout->p_vf2_chain,
+                                               p_vout->psz_vf2 ) < 0 )
+                msg_Err( p_vout, "Video filter chain creation failed" );
+
+            free( p_vout->psz_vf2 );
+            p_vout->psz_vf2 = NULL;
         }
+        vlc_mutex_unlock( &p_vout->vfilter_lock );
     }
 
 
@@ -1227,12 +1137,13 @@ static void RunThread( vout_thread_t *p_vout)
      * Error loop - wait until the thread destruction is requested
      */
     if( p_vout->b_error )
-    {
         ErrorThread( p_vout );
-    }
 
     /* End of thread */
+    CleanThread( p_vout );
     EndThread( p_vout );
+    vlc_mutex_unlock( &p_vout->change_lock );
+
     vlc_object_unlock( p_vout );
 }
 
@@ -1246,23 +1157,47 @@ static void RunThread( vout_thread_t *p_vout)
 static void ErrorThread( vout_thread_t *p_vout )
 {
     /* Wait until a `die' order */
-    while( !p_vout->b_die )
+    while( vlc_object_alive( p_vout ) )
+        vlc_object_wait( p_vout );
+}
+
+/*****************************************************************************
+ * CleanThread: clean up after InitThread
+ *****************************************************************************
+ * This function is called after a sucessful
+ * initialization. It frees all resources allocated by InitThread.
+ * XXX You have to enter it with change_lock taken.
+ *****************************************************************************/
+static void CleanThread( vout_thread_t *p_vout )
+{
+    int     i_index;                                        /* index in heap */
+
+    if( !p_vout->b_direct )
+        ChromaDestroy( p_vout );
+
+    /* Destroy all remaining pictures */
+    for( i_index = 0; i_index < 2 * VOUT_MAX_PICTURES + 1; i_index++ )
     {
-        /* Sleep a while */
-        msleep( VOUT_IDLE_SLEEP );
+        if ( p_vout->p_picture[i_index].i_type == MEMORY_PICTURE )
+        {
+            free( p_vout->p_picture[i_index].p_data_orig );
+        }
     }
+
+    /* Destroy translation tables */
+    if( !p_vout->b_error )
+        p_vout->pf_end( p_vout );
 }
 
 /*****************************************************************************
  * EndThread: thread destruction
  *****************************************************************************
- * This function is called when the thread ends after a sucessful
- * initialization. It frees all resources allocated by InitThread.
+ * This function is called when the thread ends.
+ * It frees all resources not allocated by InitThread.
+ * XXX You have to enter it with change_lock taken.
  *****************************************************************************/
 static void EndThread( vout_thread_t *p_vout )
 {
-    int     i_index;                                        /* index in heap */
-
 #ifdef STATS
     {
         struct tms cpu_usage;
@@ -1273,35 +1208,85 @@ static void EndThread( vout_thread_t *p_vout )
     }
 #endif
 
-    if( !p_vout->b_direct )
-    {
-        module_Unneed( p_vout->p_chroma, p_vout->p_chroma->p_module );
-        p_vout->p_chroma->p_module = NULL;
-    }
-
-    /* Destroy all remaining pictures */
-    for( i_index = 0; i_index < 2 * VOUT_MAX_PICTURES + 1; i_index++ )
-    {
-        if ( p_vout->p_picture[i_index].i_type == MEMORY_PICTURE )
-        {
-            free( p_vout->p_picture[i_index].p_data_orig );
-        }
-    }
+    /* FIXME does that function *really* need to be called inside the thread ? */
 
     /* Destroy subpicture unit */
     spu_Attach( p_vout->p_spu, VLC_OBJECT(p_vout), false );
     spu_Destroy( p_vout->p_spu );
 
     /* Destroy the video filters2 */
-    RemoveVideoFilters2( p_vout );
+    filter_chain_Delete( p_vout->p_vf2_chain );
+}
 
-    /* Destroy translation tables */
-    p_vout->pf_end( p_vout );
+/* Thread helpers */
+static picture_t *ChromaGetPicture( filter_t *p_filter )
+{
+    picture_t *p_pic = (picture_t *)p_filter->p_owner;
+    p_filter->p_owner = NULL;
+    return p_pic;
+}
 
-    /* Release the change lock */
-    vlc_mutex_unlock( &p_vout->change_lock );
+static void ChromaCopyRgbInfo( es_format_t *p_fmt, picture_heap_t *p_heap )
+{
+    p_fmt->video.i_rmask = p_heap->i_rmask;
+    p_fmt->video.i_gmask = p_heap->i_gmask;
+    p_fmt->video.i_bmask = p_heap->i_bmask;
+    p_fmt->video.i_rrshift = p_heap->i_rrshift;
+    p_fmt->video.i_lrshift = p_heap->i_lrshift;
+    p_fmt->video.i_rgshift = p_heap->i_rgshift;
+    p_fmt->video.i_lgshift = p_heap->i_lgshift;
+    p_fmt->video.i_rbshift = p_heap->i_rbshift;
+    p_fmt->video.i_lbshift = p_heap->i_lbshift;
 }
 
+static int ChromaCreate( vout_thread_t *p_vout )
+{
+    filter_t *p_chroma;
+
+    /* Choose the best module */
+    p_chroma = p_vout->p_chroma = vlc_object_create( p_vout, sizeof(filter_t) );
+
+    vlc_object_attach( p_chroma, p_vout );
+
+    /* TODO: Set the fmt_in and fmt_out stuff here */
+    p_chroma->fmt_in.video = p_vout->fmt_render;
+    p_chroma->fmt_out.video = p_vout->fmt_out;
+    ChromaCopyRgbInfo( &p_chroma->fmt_in, &p_vout->render );
+    ChromaCopyRgbInfo( &p_chroma->fmt_out, &p_vout->output );
+
+    p_chroma->p_module = module_Need( p_chroma, "video filter2", NULL, 0 );
+
+    if( p_chroma->p_module == NULL )
+    {
+        msg_Err( p_vout, "no chroma module for %4.4s to %4.4s i=%dx%d o=%dx%d",
+                 (char*)&p_vout->render.i_chroma,
+                 (char*)&p_vout->output.i_chroma,
+                 p_chroma->fmt_in.video.i_width, p_chroma->fmt_in.video.i_height,
+                 p_chroma->fmt_out.video.i_width, p_chroma->fmt_out.video.i_height
+                 );
+
+        vlc_object_release( p_vout->p_chroma );
+        p_vout->p_chroma = NULL;
+        return VLC_EGENERIC;
+    }
+    p_chroma->pf_vout_buffer_new = ChromaGetPicture;
+    return VLC_SUCCESS;
+}
+
+static void ChromaDestroy( vout_thread_t *p_vout )
+{
+    assert( !p_vout->b_direct );
+
+    if( !p_vout->p_chroma )
+        return;
+
+    module_Unneed( p_vout->p_chroma, p_vout->p_chroma->p_module );
+    vlc_object_release( p_vout->p_chroma );
+    p_vout->p_chroma = NULL;
+}
+
+
+
 /* following functions are local */
 
 static int ReduceHeight( int i_ratio )
@@ -1426,18 +1411,12 @@ typedef struct suxor_thread_t
 
 static void SuxorRestartVideoES( suxor_thread_t *p_this )
 {
-    vlc_value_t val;
-
-    vlc_thread_ready( p_this );
-
     /* Now restart current video stream */
-    var_Get( p_this->p_input, "video-es", &val );
-    if( val.i_int >= 0 )
+    int val = var_GetInteger( p_this->p_input, "video-es" );
+    if( val >= 0 )
     {
-        vlc_value_t val_es;
-        val_es.i_int = -VIDEO_ES;
-        var_Set( p_this->p_input, "video-es", val_es );
-        var_Set( p_this->p_input, "video-es", val );
+        var_SetInteger( p_this->p_input, "video-es", -VIDEO_ES );
+        var_SetInteger( p_this->p_input, "video-es", val );
     }
 
     vlc_object_release( p_this->p_input );
@@ -1550,46 +1529,6 @@ static int FilterCallback( vlc_object_t *p_this, char const *psz_cmd,
 /*****************************************************************************
  * Video Filter2 stuff
  *****************************************************************************/
-static int ParseVideoFilter2Chain( vout_thread_t *p_vout, char *psz_vfilters )
-{
-    int i;
-    for( i = 0; i < p_vout->i_vfilters_cfg; i++ )
-    {
-        struct config_chain_t *p_cfg =
-            p_vout->p_vfilters_cfg[p_vout->i_vfilters_cfg];
-        config_ChainDestroy( p_cfg );
-        free( p_vout->psz_vfilters[p_vout->i_vfilters_cfg] );
-        p_vout->psz_vfilters[p_vout->i_vfilters_cfg] = NULL;
-    }
-    p_vout->i_vfilters_cfg = 0;
-    if( psz_vfilters && *psz_vfilters )
-    {
-        char *psz_parser = psz_vfilters;
-
-        while( psz_parser && *psz_parser )
-        {
-            psz_parser = config_ChainCreate(
-                            &p_vout->psz_vfilters[p_vout->i_vfilters_cfg],
-                            &p_vout->p_vfilters_cfg[p_vout->i_vfilters_cfg],
-                            psz_parser );
-            msg_Dbg( p_vout, "adding vfilter: %s",
-                     p_vout->psz_vfilters[p_vout->i_vfilters_cfg] );
-            p_vout->i_vfilters_cfg++;
-            if( psz_parser && *psz_parser )
-            {
-                if( p_vout->i_vfilters_cfg == MAX_VFILTERS )
-                {
-                    msg_Warn( p_vout,
-                  "maximum number of video filters reached. \"%s\" discarded",
-                              psz_parser );
-                    break;
-                }
-            }
-        }
-    }
-    return VLC_SUCCESS;
-}
-
 static int VideoFilter2Callback( vlc_object_t *p_this, char const *psz_cmd,
                        vlc_value_t oldval, vlc_value_t newval, void *p_data )
 {
@@ -1597,31 +1536,12 @@ static int VideoFilter2Callback( vlc_object_t *p_this, char const *psz_cmd,
     (void)psz_cmd; (void)oldval; (void)p_data;
 
     vlc_mutex_lock( &p_vout->vfilter_lock );
-    ParseVideoFilter2Chain( p_vout, newval.psz_string );
-    p_vout->b_vfilter_change = true;
+    p_vout->psz_vf2 = strdup( newval.psz_string );
     vlc_mutex_unlock( &p_vout->vfilter_lock );
 
     return VLC_SUCCESS;
 }
 
-static void RemoveVideoFilters2( vout_thread_t *p_vout )
-{
-    int i;
-    for( i = 0; i < p_vout->i_vfilters; i++ )
-    {
-        vlc_object_detach( p_vout->pp_vfilters[i] );
-        if( p_vout->pp_vfilters[i]->p_module )
-        {
-            module_Unneed( p_vout->pp_vfilters[i],
-                           p_vout->pp_vfilters[i]->p_module );
-        }
-
-        free( p_vout->pp_vfilters[i]->p_owner );
-        vlc_object_release( p_vout->pp_vfilters[i] );
-    }
-    p_vout->i_vfilters = 0;
-}
-
 static void DisplayTitleOnOSD( vout_thread_t *p_vout )
 {
     input_thread_t *p_input;