- remove some \n in GstThread debugging
[platform/upstream/gstreamer.git] / gst / gstthread.c
index 73561d3..74e4be6 100644 (file)
@@ -1,5 +1,9 @@
-/* Gnome-Streamer
- * Copyright (C) <1999> Erik Walthinsen <omega@cse.ogi.edu>
+/* GStreamer
+ * Copyright (C) 1999,2000 Erik Walthinsen <omega@cse.ogi.edu>
+ *                    2000 Wim Taymans <wtay@chello.be>
+ *                    2003 Benjamin Otte <in7y118@public.uni-hamburg.de>
+ *
+ * gstthread.c: Threaded container object
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * Boston, MA 02111-1307, USA.
  */
 
-#include <unistd.h>
-
 #include "gstthread.h"
-#include "gstdebug.h"
+#include "gstscheduler.h"
+#include "gstinfo.h"
+#include "gstlog.h"
+
+#define STACK_SIZE 0x200000
 
 GstElementDetails gst_thread_details = {
   "Threaded container",
-  "Bin",
+  "Generic/Bin",
+  "LGPL",
   "Container that creates/manages a thread",
   VERSION,
-  "Erik Walthinsen <omega@cse.ogi.edu>",
-  "(C) 1999",
+  "Erik Walthinsen <omega@cse.ogi.edu>,"
+  "Benjamin Otte <in7y118@informatik.uni-hamburg.de",
+  "(C) 1999-2003",
 };
 
 
 /* Thread signals and args */
 enum {
+  SHUTDOWN,
   /* FILL ME */
   LAST_SIGNAL
 };
 
 enum {
+  SPINUP=0,
+  STATECHANGE,
+  STARTUP
+};
+
+enum {
   ARG_0,
-  ARG_CREATE_THREAD,
+  ARG_PRIORITY,
 };
 
 
-static void                    gst_thread_class_init           (GstThreadClass *klass);
-static void                    gst_thread_init                 (GstThread *thread);
 
-static void                    gst_thread_set_arg              (GtkObject *object,GtkArg *arg,guint id);
-static void                    gst_thread_get_arg              (GtkObject *object,GtkArg *arg,guint id);
+static void            gst_thread_class_init           (GstThreadClass *klass);
+static void            gst_thread_init                 (GstThread *thread);
+
+static void            gst_thread_dispose              (GObject *object);
+
+static void            gst_thread_set_property         (GObject *object, guint prop_id, 
+                                                        const GValue *value, GParamSpec *pspec);
+static void            gst_thread_get_property         (GObject *object, guint prop_id,
+                                                        GValue *value, GParamSpec *pspec);
+static GstElementStateReturn gst_thread_change_state   (GstElement *element);
+static void            gst_thread_child_state_change   (GstBin *bin, GstElementState oldstate, 
+                                                        GstElementState newstate, GstElement *element);
 
-static GstElementStateReturn   gst_thread_change_state         (GstElement *element);
+static void            gst_thread_catch                (GstThread *thread);
+static void            gst_thread_release              (GstThread *thread);
 
-static xmlNodePtr              gst_thread_save_thyself         (GstElement *element,xmlNodePtr parent);
-static void                    gst_thread_restore_thyself      (GstElement *element,xmlNodePtr parent, 
-                                                                GHashTable *elements);
+#ifndef GST_DISABLE_LOADSAVE
+static xmlNodePtr      gst_thread_save_thyself         (GstObject *object,
+                                                        xmlNodePtr parent);
+static void            gst_thread_restore_thyself      (GstObject *object,
+                                                        xmlNodePtr self);
+#endif
 
-static void                    gst_thread_signal_thread        (GstThread *thread);
-static void                    gst_thread_create_plan_dummy    (GstBin *bin);
+static void *          gst_thread_main_loop            (void *arg);
 
-static void*                   gst_thread_main_loop            (void *arg);
+#define GST_TYPE_THREAD_PRIORITY (gst_thread_priority_get_type())
+static GType
+gst_thread_priority_get_type(void) 
+{
+  static GType thread_priority_type = 0;
+  static GEnumValue thread_priority[] = 
+  {
+    { G_THREAD_PRIORITY_LOW,    "LOW",     "Low Priority Scheduling" },
+    { G_THREAD_PRIORITY_NORMAL, "NORMAL",  "Normal Scheduling" },
+    { G_THREAD_PRIORITY_HIGH,   "HIGH",    "High Priority Scheduling" },
+    { G_THREAD_PRIORITY_URGENT, "URGENT",  "Urgent Scheduling" },
+    { 0, NULL, NULL },
+  };
+  if (!thread_priority_type) {
+    thread_priority_type = g_enum_register_static("GstThreadPriority", thread_priority);
+  }
+  return thread_priority_type;
+}
 
 static GstBinClass *parent_class = NULL;
-//static guint gst_thread_signals[LAST_SIGNAL] = { 0 };
+static guint gst_thread_signals[LAST_SIGNAL] = { 0 };
+GPrivate *gst_thread_current;
 
-GtkType
+GType
 gst_thread_get_type(void) {
-  static GtkType thread_type = 0;
+  static GType thread_type = 0;
 
   if (!thread_type) {
-    static const GtkTypeInfo thread_info = {
-      "GstThread",
-      sizeof(GstThread),
-      sizeof(GstThreadClass),
-      (GtkClassInitFunc)gst_thread_class_init,
-      (GtkObjectInitFunc)gst_thread_init,
-      (GtkArgSetFunc)NULL,
-      (GtkArgGetFunc)NULL,
-      (GtkClassInitFunc)NULL,
+    static const GTypeInfo thread_info = {
+      sizeof (GstThreadClass), NULL, NULL,
+      (GClassInitFunc) gst_thread_class_init, NULL, NULL,
+      sizeof (GstThread),
+      4,
+      (GInstanceInitFunc) gst_thread_init,
+      NULL
     };
-    thread_type = gtk_type_unique(GST_TYPE_BIN,&thread_info);
+    thread_type = g_type_register_static (GST_TYPE_BIN, "GstThread",
+                                         &thread_info, 0);
   }
   return thread_type;
 }
 
+static void do_nothing (gpointer hi) {}
 static void
-gst_thread_class_init (GstThreadClass *klass) 
+gst_thread_class_init (GstThreadClass *klass)
 {
-  GtkObjectClass *gtkobject_class;
+  GObjectClass *gobject_class;
   GstObjectClass *gstobject_class;
   GstElementClass *gstelement_class;
   GstBinClass *gstbin_class;
 
-  gtkobject_class =    (GtkObjectClass*)klass;
-  gstobject_class =    (GstObjectClass*)klass;
-  gstelement_class =   (GstElementClass*)klass;
-  gstbin_class =       (GstBinClass*)klass;
+  /* setup gst_thread_current */
+  gst_thread_current = g_private_new (do_nothing);
+
+  gobject_class =      (GObjectClass*)klass;
+  gstobject_class =    (GstObjectClass*)klass;
+  gstelement_class =   (GstElementClass*)klass;
+  gstbin_class =       (GstBinClass*)klass;
 
-  parent_class = gtk_type_class (GST_TYPE_BIN);
+  parent_class = g_type_class_ref (GST_TYPE_BIN);
 
-  gtk_object_add_arg_type ("GstThread::create_thread", GTK_TYPE_BOOL,
-                           GTK_ARG_READWRITE, ARG_CREATE_THREAD);
+  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_PRIORITY,
+    g_param_spec_enum ("priority", "Scheduling Policy", "The scheduling priority of the thread",
+                       GST_TYPE_THREAD_PRIORITY, G_THREAD_PRIORITY_NORMAL, G_PARAM_READWRITE));
 
-  gstelement_class->change_state =     gst_thread_change_state;
-  gstelement_class->save_thyself =     gst_thread_save_thyself;
-  gstelement_class->restore_thyself =  gst_thread_restore_thyself;
+  gst_thread_signals[SHUTDOWN] =
+    g_signal_new ("shutdown", G_TYPE_FROM_CLASS (klass), G_SIGNAL_RUN_LAST,
+                  G_STRUCT_OFFSET (GstThreadClass, shutdown), NULL, NULL,
+                  gst_marshal_VOID__VOID, G_TYPE_NONE, 0);
 
-  gstbin_class->create_plan = gst_thread_create_plan_dummy;
+  gobject_class->dispose =             gst_thread_dispose;
 
-  gtkobject_class->set_arg = gst_thread_set_arg;
-  gtkobject_class->get_arg = gst_thread_get_arg;
+#ifndef GST_DISABLE_LOADSAVE
+  gstobject_class->save_thyself =      GST_DEBUG_FUNCPTR (gst_thread_save_thyself);
+  gstobject_class->restore_thyself =   GST_DEBUG_FUNCPTR (gst_thread_restore_thyself);
+#endif
 
+  gstelement_class->change_state =     GST_DEBUG_FUNCPTR (gst_thread_change_state);
+
+  gobject_class->set_property = GST_DEBUG_FUNCPTR (gst_thread_set_property);
+  gobject_class->get_property = GST_DEBUG_FUNCPTR (gst_thread_get_property);
+
+  gstbin_class->child_state_change =   GST_DEBUG_FUNCPTR (gst_thread_child_state_change);
 }
 
-static void 
-gst_thread_init (GstThread *thread) 
+static void
+gst_thread_init (GstThread *thread)
 {
-  DEBUG("initializing thread '%s'\n",gst_element_get_name(GST_ELEMENT(thread)));
+  GstScheduler *scheduler;
+
+  GST_DEBUG (GST_CAT_THREAD, "initializing thread");
 
-  // we're a manager by default
+  /* threads are managing bins and iterate themselves */
+  /* CR1: the GstBin code checks these flags */
   GST_FLAG_SET (thread, GST_BIN_FLAG_MANAGER);
+  GST_FLAG_SET (thread, GST_BIN_SELF_SCHEDULABLE);
 
-  // default is to create a thread
-  GST_FLAG_SET (thread, GST_THREAD_CREATE);
-  GST_FLAG_UNSET (thread, GST_THREAD_STATE_REAPING);
+  scheduler = gst_scheduler_factory_make (NULL, GST_ELEMENT (thread));
+  g_assert (scheduler);
+
+  thread->lock = g_mutex_new ();
+  thread->cond = g_cond_new ();
 
-  thread->lock = g_mutex_new();
-  thread->cond = g_cond_new();
+  thread->thread_id = (GThread *) NULL; /* set in NULL -> READY */
+  thread->priority = G_THREAD_PRIORITY_NORMAL;
 }
 
-static void 
-gst_thread_create_plan_dummy (GstBin *bin) 
+static void
+gst_thread_dispose (GObject *object)
 {
-  g_return_if_fail (GST_IS_THREAD (bin));
+  GstThread *thread = GST_THREAD (object);
+
+  GST_DEBUG (GST_CAT_REFCOUNTING, "GstThread: dispose");
 
-  if (!GST_FLAG_IS_SET (GST_THREAD (bin), GST_THREAD_STATE_SPINNING)) 
-    gst_info("gstthread: create plan delayed until thread starts\n");
+  G_OBJECT_CLASS (parent_class)->dispose (object);
+
+  g_assert (GST_STATE (thread) == GST_STATE_NULL);
+
+  g_mutex_free (thread->lock);
+  g_cond_free (thread->cond);
+
+  gst_object_replace ((GstObject **)&GST_ELEMENT_SCHED (thread), NULL);
 }
 
-static void 
-gst_thread_set_arg (GtkObject *object,
-                   GtkArg *arg,
-                   guint id) 
+/**
+ * gst_thread_set_priority:
+ * @thread: the thread to change 
+ * @priority: the new priority for the thread
+ *
+ * change the thread's priority
+ */
+void
+gst_thread_set_priority (GstThread *thread, GThreadPriority priority)
 {
-  /* it's not null if we got it, but it might not be ours */
-  g_return_if_fail (GST_IS_THREAD (object));
-
-  switch(id) {
-    case ARG_CREATE_THREAD:
-      if (GTK_VALUE_BOOL (*arg)) {
-        gst_info("gstthread: turning ON the creation of the thread\n");
-        GST_FLAG_SET (object, GST_THREAD_CREATE);
-        gst_info("gstthread: flags are 0x%08x\n", GST_FLAGS (object));
-      } else {
-        gst_info("gstthread: turning OFF the creation of the thread\n");
-        GST_FLAG_UNSET (object, GST_THREAD_CREATE);
-        gst_info("gstthread: flags are 0x%08x\n", GST_FLAGS (object));
-      }
+  g_return_if_fail (GST_IS_THREAD (thread));
+
+  thread->priority = priority;
+}
+
+static void
+gst_thread_set_property (GObject *object, guint prop_id, const GValue *value, GParamSpec *pspec)
+{
+  GstThread *thread;
+
+  thread = GST_THREAD (object);
+
+  switch (prop_id) {
+    case ARG_PRIORITY:
+      thread->priority = g_value_get_enum (value);
       break;
     default:
+      G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
       break;
   }
 }
 
-static void 
-gst_thread_get_arg (GtkObject *object,
-                   GtkArg *arg,
-                   guint id) 
+static void
+gst_thread_get_property (GObject *object, guint prop_id, GValue *value, GParamSpec *pspec)
 {
-  /* it's not null if we got it, but it might not be ours */
-  g_return_if_fail (GST_IS_THREAD (object));
+  GstThread *thread;
 
-  switch (id) {
-    case ARG_CREATE_THREAD:
-      GTK_VALUE_BOOL (*arg) = GST_FLAG_IS_SET (object, GST_THREAD_CREATE);
+  thread = GST_THREAD (object);
+
+  switch (prop_id) {
+    case ARG_PRIORITY:
+      g_value_set_enum (value, thread->priority);
       break;
     default:
+      G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
       break;
   }
 }
@@ -185,161 +263,333 @@ gst_thread_get_arg (GtkObject *object,
  * gst_thread_new:
  * @name: the name of the thread
  *
- * Create a new thrad with the given name
+ * Create a new thread with the given name.
  *
- * Returns; The new thread
+ * Returns: The new thread
  */
 GstElement*
-gst_thread_new (guchar *name) 
+gst_thread_new (const gchar *name)
 {
-  return gst_elementfactory_make ("thread", name);
+  return gst_element_factory_make ("thread", name);
 }
 
+/**
+ * gst_thread_get_current:
+ *
+ * Create a new thread with the given name.
+ *
+ * Returns: The current GstThread or NULL if you are not running inside a 
+ *          #GstThread.
+ */
+GstThread *
+gst_thread_get_current (void)
+{
+  return (GstThread *) g_private_get (gst_thread_current);
+}
 
-
-static GstElementStateReturn 
-gst_thread_change_state (GstElement *element) 
+static inline void
+gst_thread_release_children_locks (GstThread *thread)
 {
-  GstThread *thread;
-  gboolean stateset = GST_STATE_SUCCESS;
-  gint pending, transition;
+  GstRealPad *peer = NULL;
+  GstElement *peerelement;
+  GList *elements = (GList *) gst_bin_get_list (GST_BIN (thread));
 
-  g_return_val_if_fail (GST_IS_THREAD(element), FALSE);
-  DEBUG_ENTER("(\"%s\")",gst_element_get_name(element));
+  while (elements) {
+    GstElement *element = GST_ELEMENT (elements->data);
+    GList *pads;
 
-  thread = GST_THREAD (element);
+    g_assert (element);
+    GST_DEBUG (GST_CAT_THREAD, "waking element \"%s\"", GST_ELEMENT_NAME (element));
+    elements = g_list_next (elements);
 
-  gst_info("gstthread: thread \"%s\" change state %d\n",
-               gst_element_get_name (GST_ELEMENT (element)), 
-              GST_STATE_PENDING (element));
+    if (!gst_element_release_locks (element))
+      g_warning ("element %s could not release locks", GST_ELEMENT_NAME (element));
 
-  pending = GST_STATE_PENDING (element);
+    pads = GST_ELEMENT_PADS (element);
+
+    while (pads) {
+      if (GST_PAD_PEER (pads->data)) {
+        peer = GST_REAL_PAD (GST_PAD_PEER (pads->data));
+       pads = g_list_next (pads);
+      } else {
+       pads = g_list_next (pads);
+       continue;
+      }
+
+      if (!peer)
+        continue;
+
+      peerelement = GST_PAD_PARENT (peer);
+      if (!peerelement)
+        continue; /* FIXME: deal with case where there's no peer */
+
+      if (GST_ELEMENT_SCHED (peerelement) != GST_ELEMENT_SCHED (thread)) {
+        GST_DEBUG (GST_CAT_THREAD, "element \"%s\" has pad cross sched boundary", GST_ELEMENT_NAME (element));
+        GST_DEBUG (GST_CAT_THREAD, "waking element \"%s\"", GST_ELEMENT_NAME (peerelement));
+        if (!gst_element_release_locks (peerelement))
+          g_warning ("element %s could not release locks", GST_ELEMENT_NAME (peerelement));
+      }
+    }
+  }
+}
+/* stops the main thread, if there is one and grabs the thread's mutex */
+static void
+gst_thread_catch (GstThread *thread)
+{
+  gboolean wait;
+  if (thread == gst_thread_get_current()) {
+    /* we're trying to catch ourself */
+    if (!GST_FLAG_IS_SET (thread, GST_THREAD_MUTEX_LOCKED)) {
+      g_mutex_lock (thread->lock);
+      GST_FLAG_SET (thread, GST_THREAD_MUTEX_LOCKED);
+    }
+    GST_DEBUG (GST_CAT_THREAD, "%s is catching itself", GST_ELEMENT_NAME (thread));
+    GST_FLAG_UNSET (thread, GST_THREAD_STATE_SPINNING);
+  } else {
+    /* another thread is trying to catch us */
+    g_mutex_lock (thread->lock);
+    wait = !GST_FLAG_IS_SET (thread, GST_THREAD_STATE_SPINNING);
+    while (!wait) {
+      GTimeVal tv;
+      GST_DEBUG (GST_CAT_THREAD, "catching %s...", GST_ELEMENT_NAME (thread));
+      GST_FLAG_UNSET (thread, GST_THREAD_STATE_SPINNING);
+      g_cond_signal (thread->cond);
+      gst_thread_release_children_locks (thread);
+      g_get_current_time (&tv);
+      g_time_val_add (&tv, 1000); /* wait a millisecond to catch the thread */
+      wait = g_cond_timed_wait (thread->cond, thread->lock, &tv);
+    }
+    GST_DEBUG (GST_CAT_THREAD, "caught %s", GST_ELEMENT_NAME (thread));
+  }
+  g_assert (!GST_FLAG_IS_SET (thread, GST_THREAD_STATE_SPINNING));
+}
+
+static void
+gst_thread_release (GstThread *thread)
+{
+  if (thread != gst_thread_get_current()) {
+    g_cond_signal (thread->cond);
+    g_mutex_unlock (thread->lock);
+  }
+}
+
+static GstElementStateReturn
+gst_thread_change_state (GstElement *element)
+{
+  GstThread *thread;
+  GstElementStateReturn ret;
+  gint transition;
+
+  g_return_val_if_fail (GST_IS_THREAD (element), GST_STATE_FAILURE);
   transition = GST_STATE_TRANSITION (element);
 
-  if (pending == GST_STATE (element)) return GST_STATE_SUCCESS;
+  thread = GST_THREAD (element);
+
+  GST_DEBUG (GST_CAT_STATES, "%s is changing state from %s to %s",
+            GST_ELEMENT_NAME (element), gst_element_state_get_name (GST_STATE (element)),
+            gst_element_state_get_name (GST_STATE_PENDING (element)));
 
-  GST_FLAG_UNSET (thread, GST_THREAD_STATE_SPINNING);
+  gst_thread_catch (thread);
 
-  if (GST_ELEMENT_CLASS (parent_class)->change_state)
-    stateset = GST_ELEMENT_CLASS (parent_class)->change_state (element);
-  
-  gst_info("gstthread: stateset %d %d %d %02x\n", GST_STATE (element), stateset, 
-                 GST_STATE_PENDING (element), GST_STATE_TRANSITION (element));
+  /* FIXME: (or GStreamers ideas about "threading"): the element variables are
+     commonly accessed by multiple threads at the same time (see bug #111146
+     for an example) */
+  if (transition != GST_STATE_TRANSITION (element)) {
+    g_warning ("inconsistent state information, fix threading please");
+  }
 
   switch (transition) {
     case GST_STATE_NULL_TO_READY:
-      if (!stateset) return FALSE;
-      // we want to prepare our internal state for doing the iterations
-      gst_info("gstthread: preparing thread \"%s\" for iterations:\n",
-               gst_element_get_name (GST_ELEMENT (element)));
-      
-      // set the state to idle
-      GST_FLAG_UNSET (thread, GST_THREAD_STATE_SPINNING);
-      // create the thread if that's what we're supposed to do
-      gst_info("gstthread: flags are 0x%08x\n", GST_FLAGS (thread));
-      
-      if (GST_FLAG_IS_SET (thread, GST_THREAD_CREATE)) {
-        gst_info("gstthread: starting thread \"%s\"\n",
-                 gst_element_get_name (GST_ELEMENT (element)));
-       
-        pthread_create (&thread->thread_id, NULL,
-                        gst_thread_main_loop, thread);
-      } else {
-        gst_info("gstthread: NOT starting thread \"%s\"\n",
-                gst_element_get_name (GST_ELEMENT (element)));
+      /* create the thread */
+      GST_FLAG_UNSET (thread, GST_THREAD_STATE_REAPING);
+      thread->thread_id = g_thread_create_full(gst_thread_main_loop,
+         thread, STACK_SIZE, FALSE, TRUE, thread->priority,
+         NULL);
+      if (!thread->thread_id){
+        GST_DEBUG (GST_CAT_THREAD, "g_thread_create_full for %sfailed", GST_ELEMENT_NAME (element));
+        goto error_out;
       }
-      return GST_STATE_ASYNC;
+      GST_DEBUG (GST_CAT_THREAD, "GThread created");
+
+      /* wait for it to 'spin up' */
+      g_cond_wait (thread->cond, thread->lock);
+      break;
+    case GST_STATE_READY_TO_PAUSED:
       break;
     case GST_STATE_PAUSED_TO_PLAYING:
-    case GST_STATE_READY_TO_PLAYING:
-      if (!stateset) return FALSE;
-      gst_info("gstthread: starting thread \"%s\"\n",
-              gst_element_get_name (GST_ELEMENT (element)));
-      
-      GST_FLAG_SET (thread, GST_THREAD_STATE_SPINNING);
-      gst_thread_signal_thread (thread);
-      break;  
+    {
+      /* FIXME: recurse into sub-bins */
+      GList *elements = (GList *) gst_bin_get_list (GST_BIN (thread));
+      while (elements) {
+        gst_element_enable_threadsafe_properties ((GstElement*)elements->data);
+        elements = g_list_next (elements);
+      }
+      break;
+    }
     case GST_STATE_PLAYING_TO_PAUSED:
-      gst_info("gstthread: pausing thread \"%s\"\n",
-              gst_element_get_name (GST_ELEMENT (element)));
-      
-      //GST_FLAG_UNSET(thread,GST_THREAD_STATE_SPINNING);
-      gst_thread_signal_thread (thread);
+    {
+      GList *elements = (GList *) gst_bin_get_list (GST_BIN (thread));
+      while (elements) {
+        gst_element_disable_threadsafe_properties ((GstElement*)elements->data);
+        elements = g_list_next (elements);
+      }
+      break;
+    }
+    case GST_STATE_PAUSED_TO_READY:
       break;
     case GST_STATE_READY_TO_NULL:
-      gst_info("gstthread: stopping thread \"%s\"\n",
-              gst_element_get_name (GST_ELEMENT (element)));
-      
+      /* we can't join the threads here, because this could have been triggered
+         by ourself (ouch) */
+      GST_DEBUG (GST_CAT_THREAD, "destroying GThread %p", thread->thread_id);
       GST_FLAG_SET (thread, GST_THREAD_STATE_REAPING);
-      gst_thread_signal_thread (thread);
+      thread->thread_id = NULL;
+      if (thread == gst_thread_get_current()) {
+        /* or should we continue? */
+        g_warning ("Thread %s is destroying itself. Function call will not return!", GST_ELEMENT_NAME (thread));
+        gst_scheduler_reset (GST_ELEMENT_SCHED (thread));
+       
+        /* unlock and signal - we are out */
+        gst_thread_release (thread);
+
+       GST_INFO (GST_CAT_THREAD, "gstthread: thread \"%s\" is stopped",
+                 GST_ELEMENT_NAME (thread));
+
+       g_signal_emit (G_OBJECT (thread), gst_thread_signals[SHUTDOWN], 0);
+
+        g_thread_exit (NULL);
+      }
+      /* now wait for the thread to destroy itself */
+      g_cond_signal (thread->cond);
+      g_cond_wait (thread->cond, thread->lock);
+      /* it should be dead now */
       break;
     default:
+      GST_DEBUG_ELEMENT (GST_CAT_THREAD, element, "UNHANDLED STATE CHANGE! %x", 
+                         GST_STATE_TRANSITION (element));
+      g_assert_not_reached ();
       break;
   }
 
-  return stateset;
+  if (GST_ELEMENT_CLASS (parent_class)->change_state) {
+    ret = GST_ELEMENT_CLASS (parent_class)->change_state (GST_ELEMENT (thread));
+  } else {
+    ret = GST_STATE_SUCCESS;
+  }
+
+  gst_thread_release (thread);
+  return ret;
+  
+error_out:
+  GST_DEBUG (GST_CAT_STATES, "changing state from %s to %s failed for %s",
+             gst_element_state_get_name (GST_STATE (element)),
+            gst_element_state_get_name (GST_STATE_PENDING (element)),
+             GST_ELEMENT_NAME (element));
+  gst_thread_release (thread);
+  return GST_STATE_FAILURE;
 }
 
+/* state changes work this way: We grab the lock and stop the thread from 
+   spinning (via gst_thread_catch) - then we change the state. After that the
+   thread may spin on. */
+static void
+gst_thread_child_state_change (GstBin *bin, GstElementState oldstate, 
+                              GstElementState newstate, GstElement *element)
+{
+  GST_DEBUG (GST_CAT_THREAD, "%s (from thread %s) child %s changed state from %s to %s",
+              GST_ELEMENT_NAME (bin), 
+              gst_thread_get_current() ? GST_ELEMENT_NAME (gst_thread_get_current()) : "(none)", 
+              GST_ELEMENT_NAME (element), gst_element_state_get_name (oldstate),
+              gst_element_state_get_name (newstate));
+  if (parent_class->child_state_change)
+    parent_class->child_state_change (bin, oldstate, newstate, element);
+  /* We'll wake up the main thread now. Note that we can't lock the thread here, 
+     because we might be called from inside gst_thread_change_state when holding
+     the lock. But this doesn't cause any problems. */
+  if (newstate == GST_STATE_PLAYING)
+    g_cond_signal (GST_THREAD (bin)->cond);
+}
 /**
  * gst_thread_main_loop:
  * @arg: the thread to start
  *
  * The main loop of the thread. The thread will iterate
- * while the state is GST_THREAD_STATE_SPINNING
+ * while the state is GST_THREAD_STATE_SPINNING.
  */
 static void *
-gst_thread_main_loop (void *arg) 
+gst_thread_main_loop (void *arg)
 {
-  GstThread *thread = GST_THREAD (arg);
+  GstThread *thread = NULL;
+  gboolean status;
 
-  gst_info("gstthread: thread \"%s\" is running with PID %d\n",
-                 gst_element_get_name (GST_ELEMENT (thread)), getpid ());
+  thread = GST_THREAD (arg);
+  g_mutex_lock (thread->lock);
+  GST_DEBUG (GST_CAT_THREAD, "Thread %s started main loop", GST_ELEMENT_NAME (thread));
 
-  if (GST_BIN_CLASS (parent_class)->create_plan)
-    GST_BIN_CLASS (parent_class)->create_plan (GST_BIN (thread));
+  /* initialize gst_thread_current */
+  g_private_set (gst_thread_current, thread);
 
-  while (!GST_FLAG_IS_SET (thread, GST_THREAD_STATE_REAPING)) {
-    if (GST_FLAG_IS_SET (thread, GST_THREAD_STATE_SPINNING))
-      gst_bin_iterate (GST_BIN (thread));
-    else {
-      g_mutex_lock (thread->lock);
-      g_cond_wait (thread->cond, thread->lock);
-      g_mutex_unlock (thread->lock);
+  /* set up the element's scheduler */
+  gst_scheduler_setup (GST_ELEMENT_SCHED (thread));
+  GST_FLAG_UNSET (thread, GST_THREAD_STATE_REAPING);
+
+  g_cond_signal (thread->cond);
+  while (!(GST_FLAG_IS_SET (thread, GST_THREAD_STATE_REAPING))) {
+    if (GST_STATE (thread) == GST_STATE_PLAYING) {
+      GST_FLAG_SET (thread, GST_THREAD_STATE_SPINNING);
+      status = TRUE;
+      GST_DEBUG (GST_CAT_THREAD, "%s starts iterating", GST_ELEMENT_NAME (thread));
+      while (status && GST_FLAG_IS_SET (thread, GST_THREAD_STATE_SPINNING)) {
+        g_mutex_unlock (thread->lock);
+        status = gst_bin_iterate (GST_BIN (thread));
+        if (GST_FLAG_IS_SET (thread, GST_THREAD_MUTEX_LOCKED)) {
+         GST_FLAG_UNSET (thread, GST_THREAD_MUTEX_LOCKED);
+       } else {
+          g_mutex_lock (thread->lock);
+       }
+      }
+      GST_FLAG_UNSET (thread, GST_THREAD_STATE_SPINNING);
     }
+    if (GST_FLAG_IS_SET (thread, GST_THREAD_STATE_REAPING))
+      break;
+    GST_DEBUG (GST_CAT_THREAD, "%s was caught", GST_ELEMENT_NAME (thread));
+    g_cond_signal (thread->cond);
+    g_cond_wait (thread->cond, thread->lock);
   }
 
-  GST_FLAG_UNSET (thread, GST_THREAD_STATE_REAPING);
-  pthread_join (thread->thread_id, 0);
+  /* we need to destroy the scheduler here because it has mapped it's
+   * stack into the threads stack space */
+  gst_scheduler_reset (GST_ELEMENT_SCHED (thread));
 
-  gst_info("gstthread: thread \"%s\" is stopped\n",
-                 gst_element_get_name (GST_ELEMENT (thread)));
-  return NULL;
-}
+  /* must do that before releasing the lock - we might get disposed before being done */
+  g_signal_emit (G_OBJECT (thread), gst_thread_signals[SHUTDOWN], 0);
 
-static void 
-gst_thread_signal_thread (GstThread *thread) 
-{
-  g_mutex_lock (thread->lock);
+  /* unlock and signal - we are out */
   g_cond_signal (thread->cond);
   g_mutex_unlock (thread->lock);
-}
 
-static void 
-gst_thread_restore_thyself (GstElement *element,
-                           xmlNodePtr parent, 
-                           GHashTable *elements) 
-{
-  g_print("gstthread: restore\n");
+  GST_INFO (GST_CAT_THREAD, "gstthread: thread \"%s\" is stopped",
+           GST_ELEMENT_NAME (thread));
 
-  if (GST_ELEMENT_CLASS (parent_class)->restore_thyself)
-    GST_ELEMENT_CLASS (parent_class)->restore_thyself (element,parent, elements);
+  return NULL;
 }
 
-static xmlNodePtr 
-gst_thread_save_thyself (GstElement *element,
-                        xmlNodePtr parent) 
+#ifndef GST_DISABLE_LOADSAVE
+static xmlNodePtr
+gst_thread_save_thyself (GstObject *object,
+                        xmlNodePtr self)
 {
-  if (GST_ELEMENT_CLASS (parent_class)->save_thyself)
-    GST_ELEMENT_CLASS (parent_class)->save_thyself (element,parent);
+  if (GST_OBJECT_CLASS (parent_class)->save_thyself)
+    GST_OBJECT_CLASS (parent_class)->save_thyself (object, self);
   return NULL;
 }
+
+static void
+gst_thread_restore_thyself (GstObject *object,
+                           xmlNodePtr self)
+{
+  GST_DEBUG (GST_CAT_THREAD,"gstthread: restore");
+
+  if (GST_OBJECT_CLASS (parent_class)->restore_thyself)
+    GST_OBJECT_CLASS (parent_class)->restore_thyself (object, self);
+}
+#endif /* GST_DISABLE_LOADSAVE */