/* 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
*
* Boston, MA 02111-1307, USA.
*/
-#include <unistd.h>
-
-/* #define GST_DEBUG_ENABLED */
-#include "gst_private.h"
-
-#include "gst.h"
#include "gstthread.h"
#include "gstscheduler.h"
-#include "gstqueue.h"
+#include "gstinfo.h"
+#include "gstlog.h"
+
+#define STACK_SIZE 0x200000
GstElementDetails gst_thread_details = {
"Threaded container",
"LGPL",
"Container that creates/manages a thread",
VERSION,
- "Erik Walthinsen <omega@cse.ogi.edu>",
- "(C) 1999, 2000",
+ "Erik Walthinsen <omega@cse.ogi.edu>,"
+ "Benjamin Otte <in7y118@informatik.uni-hamburg.de",
+ "(C) 1999-2003",
};
enum {
ARG_0,
- ARG_SCHEDPOLICY,
ARG_PRIORITY,
};
-static void gst_thread_class_init (GstThreadClass *klass);
-static void gst_thread_init (GstThread *thread);
+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_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 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);
#ifndef GST_DISABLE_LOADSAVE
-static xmlNodePtr gst_thread_save_thyself (GstObject *object, xmlNodePtr parent);
-static void gst_thread_restore_thyself (GstObject *object, xmlNodePtr self);
+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_main_loop (void *arg);
+static void * gst_thread_main_loop (void *arg);
-#define GST_TYPE_THREAD_SCHEDPOLICY (gst_thread_schedpolicy_get_type())
+#define GST_TYPE_THREAD_PRIORITY (gst_thread_priority_get_type())
static GType
-gst_thread_schedpolicy_get_type(void) {
- static GType thread_schedpolicy_type = 0;
- static GEnumValue thread_schedpolicy[] = {
- {SCHED_OTHER, "SCHED_OTHER", "Normal Scheduling"},
- {SCHED_FIFO, "SCHED_FIFO", "FIFO Scheduling (requires root)"},
- {SCHED_RR, "SCHED_RR", "Round-Robin Scheduling (requires root)"},
- {0, NULL, NULL},
+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_schedpolicy_type) {
- thread_schedpolicy_type = g_enum_register_static("GstThreadSchedPolicy", thread_schedpolicy);
+ if (!thread_priority_type) {
+ thread_priority_type = g_enum_register_static("GstThreadPriority", thread_priority);
}
- return thread_schedpolicy_type;
+ 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;
GType
gst_thread_get_type(void) {
if (!thread_type) {
static const GTypeInfo thread_info = {
- sizeof(GstThreadClass),
- NULL,
- NULL,
- (GClassInitFunc)gst_thread_class_init,
- NULL,
- NULL,
- sizeof(GstThread),
+ sizeof (GstThreadClass), NULL, NULL,
+ (GClassInitFunc) gst_thread_class_init, NULL, NULL,
+ sizeof (GstThread),
4,
- (GInstanceInitFunc)gst_thread_init,
+ (GInstanceInitFunc) gst_thread_init,
NULL
};
- thread_type = g_type_register_static(GST_TYPE_BIN, "GstThread", &thread_info, 0);
+ 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)
{
GstElementClass *gstelement_class;
GstBinClass *gstbin_class;
+ /* setup gst_thread_current */
+ gst_thread_current = g_private_new (do_nothing);
+
gobject_class = (GObjectClass*)klass;
gstobject_class = (GstObjectClass*)klass;
gstelement_class = (GstElementClass*)klass;
parent_class = g_type_class_ref (GST_TYPE_BIN);
- g_object_class_install_property(G_OBJECT_CLASS (klass), ARG_SCHEDPOLICY,
- g_param_spec_enum("schedpolicy", "Scheduling Policy", "The scheduling policy of the thread",
- GST_TYPE_THREAD_SCHEDPOLICY, SCHED_OTHER, G_PARAM_READWRITE));
- g_object_class_install_property(G_OBJECT_CLASS (klass), ARG_PRIORITY,
- g_param_spec_int("priority", "Scheduling Priority", "The scheduling priority of the thread",
- 0, 99, 0, G_PARAM_READWRITE));
+ 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));
gst_thread_signals[SHUTDOWN] =
g_signal_new ("shutdown", G_TYPE_FROM_CLASS (klass), G_SIGNAL_RUN_LAST,
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_FLAG_SET (thread, GST_BIN_SELF_SCHEDULABLE);
scheduler = gst_scheduler_factory_make (NULL, GST_ELEMENT (thread));
+ g_assert (scheduler);
thread->lock = g_mutex_new ();
thread->cond = g_cond_new ();
- thread->ppid = getpid ();
- thread->thread_id = (pthread_t) -1;
- thread->sched_policy = SCHED_OTHER;
- thread->priority = 0;
- thread->stack = NULL;
+ thread->thread_id = (GThread *) NULL; /* set in NULL -> READY */
+ thread->priority = G_THREAD_PRIORITY_NORMAL;
}
static void
{
GstThread *thread = GST_THREAD (object);
- GST_DEBUG (GST_CAT_REFCOUNTING, "dispose");
+ GST_DEBUG (GST_CAT_REFCOUNTING, "GstThread: dispose");
+
+ 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);
- G_OBJECT_CLASS (parent_class)->dispose (object);
+ gst_object_replace ((GstObject **)&GST_ELEMENT_SCHED (thread), NULL);
+}
- if (GST_ELEMENT_SCHED (thread)) {
- gst_object_unref (GST_OBJECT (GST_ELEMENT_SCHED (thread)));
- }
+/**
+ * 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)
+{
+ g_return_if_fail (GST_IS_THREAD (thread));
+
+ thread->priority = priority;
}
static void
{
GstThread *thread;
- /* it's not null if we got it, but it might not be ours */
- g_return_if_fail (GST_IS_THREAD (object));
-
thread = GST_THREAD (object);
switch (prop_id) {
- case ARG_SCHEDPOLICY:
- thread->sched_policy = g_value_get_enum (value);
- break;
case ARG_PRIORITY:
- thread->priority = g_value_get_int (value);
+ thread->priority = g_value_get_enum (value);
break;
default:
G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
{
GstThread *thread;
- /* it's not null if we got it, but it might not be ours */
- g_return_if_fail (GST_IS_THREAD (object));
-
thread = GST_THREAD (object);
switch (prop_id) {
- case ARG_SCHEDPOLICY:
- g_value_set_enum (value, thread->sched_policy);
- break;
case ARG_PRIORITY:
- g_value_set_int (value, thread->priority);
+ g_value_set_enum (value, thread->priority);
break;
default:
G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
return gst_element_factory_make ("thread", name);
}
-/* these two macros are used for debug/info from the state_change function */
-
-#define THR_INFO(format,args...) \
- GST_INFO_ELEMENT(GST_CAT_THREAD, thread, "sync(" GST_DEBUG_THREAD_FORMAT "): " format , \
- GST_DEBUG_THREAD_ARGS(thread->pid) , ## args )
-
-#define THR_DEBUG(format,args...) \
- GST_DEBUG_ELEMENT(GST_CAT_THREAD, thread, "sync(" GST_DEBUG_THREAD_FORMAT "): " format , \
- GST_DEBUG_THREAD_ARGS(thread->pid) , ## args )
-
-/* these two macros are used for debug/info from the gst_thread_main_loop
- * function
+/**
+ * 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);
+}
-#define THR_INFO_MAIN(format,args...) \
- GST_INFO_ELEMENT(GST_CAT_THREAD, thread, "sync-main(" GST_DEBUG_THREAD_FORMAT "): " format , \
- GST_DEBUG_THREAD_ARGS(thread->ppid) , ## args )
+static inline void
+gst_thread_release_children_locks (GstThread *thread)
+{
+ GstRealPad *peer = NULL;
+ GstElement *peerelement;
+ GList *elements = (GList *) gst_bin_get_list (GST_BIN (thread));
+
+ while (elements) {
+ GstElement *element = GST_ELEMENT (elements->data);
+ GList *pads;
+
+ g_assert (element);
+ GST_DEBUG (GST_CAT_THREAD, "waking element \"%s\"", GST_ELEMENT_NAME (element));
+ elements = g_list_next (elements);
+
+ if (!gst_element_release_locks (element))
+ g_warning ("element %s could not release locks", GST_ELEMENT_NAME (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;
+ }
-#define THR_DEBUG_MAIN(format,args...) \
- GST_DEBUG_ELEMENT(GST_CAT_THREAD, thread, "sync-main(" GST_DEBUG_THREAD_FORMAT "): " format , \
- GST_DEBUG_THREAD_ARGS(thread->ppid) , ## args )
+ if (!peer)
+ continue;
-static GstElementStateReturn
-gst_thread_update_state (GstThread *thread)
+ 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)
{
- GST_DEBUG_ELEMENT (GST_CAT_THREAD, thread, "updating state of thread");
- /* check for state change */
- if (GST_STATE_PENDING (thread) != GST_STATE_VOID_PENDING) {
- /* punt and change state on all the children */
- if (GST_ELEMENT_CLASS (parent_class)->change_state)
- return GST_ELEMENT_CLASS (parent_class)->change_state (GST_ELEMENT (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));
}
-
- /* FIXME: in the case of no change_state function in the parent's class,
- * shouldn't we actually change the thread's state ? */
- g_warning ("thread's parent doesn't have change_state, returning success");
- return GST_STATE_SUCCESS;
+ 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)
+gst_thread_change_state (GstElement *element)
{
GstThread *thread;
- gboolean stateset = GST_STATE_SUCCESS;
+ GstElementStateReturn ret;
gint transition;
- pthread_t self = pthread_self ();
- glong stacksize;
g_return_val_if_fail (GST_IS_THREAD (element), GST_STATE_FAILURE);
- g_return_val_if_fail (gst_has_threads (), GST_STATE_FAILURE);
+ transition = GST_STATE_TRANSITION (element);
thread = GST_THREAD (element);
- transition = GST_STATE_TRANSITION (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)));
- THR_INFO ("changing state from %s to %s",
- gst_element_state_get_name (GST_STATE (element)),
- gst_element_state_get_name (GST_STATE_PENDING (element)));
+ gst_thread_catch (thread);
- if (pthread_equal (self, thread->thread_id)) {
- GST_DEBUG (GST_CAT_THREAD,
- "no sync(" GST_DEBUG_THREAD_FORMAT "): setting own thread's state to spinning",
- GST_DEBUG_THREAD_ARGS (thread->pid));
- return gst_thread_update_state (thread);
+ /* 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:
- /* set the state to idle */
- GST_FLAG_UNSET (thread, GST_THREAD_STATE_SPINNING);
-
- THR_DEBUG ("creating thread \"%s\"", GST_ELEMENT_NAME (element));
-
- /* this bit of code handles creation of pthreads
- * this is therefor tricky code
- * compare it with the block of code that handles the destruction
- * in GST_STATE_READY_TO_NULL below
- */
- g_mutex_lock (thread->lock);
-
- /* create attribute struct for pthread
- * and assign stack pointer and size to it
- *
- * the default state of a pthread is PTHREAD_CREATE_JOINABLE
- * (see man pthread_attr_init)
- * - other thread can sync on termination
- * - thread resources are kept allocated until other thread performs
- * pthread_join
- */
-
- if (pthread_attr_init (&thread->attr) != 0)
- g_warning ("pthread_attr_init returned an error !");
-
- /* this function should return a newly allocated stack
- * (using whatever method)
- * which we can initiate the pthreads with
- * the stack should be freed in
- */
- if (gst_scheduler_get_preferred_stack (GST_ELEMENT_SCHED (element),
- &thread->stack, &stacksize)) {
-#ifdef HAVE_PTHREAD_ATTR_SETSTACK
- if (pthread_attr_setstack (&thread->attr,
- thread->stack, stacksize) != 0) {
- g_warning ("pthread_attr_setstack failed\n");
- return GST_STATE_FAILURE;
- }
-#else
- if (pthread_attr_setstackaddr (&thread->attr, thread->stack) != 0) {
- g_warning ("pthread_attr_setstackaddr failed\n");
- return GST_STATE_FAILURE;
- }
- if (pthread_attr_setstacksize (&thread->attr, stacksize) != 0) {
- g_warning ("pthread_attr_setstacksize failed\n");
- return GST_STATE_FAILURE;
- }
-#endif
- GST_DEBUG (GST_CAT_THREAD, "pthread attr set stack at %p of size %ld",
- thread->stack, stacksize);
- }
- else {
- g_warning ("scheduler did not return a preferred stack");
- }
-
- /* create a new pthread
- * use the specified attributes
- * make it execute gst_thread_main_loop (thread)
- */
- GST_DEBUG (GST_CAT_THREAD, "going to pthread_create...");
- if (pthread_create (&thread->thread_id, &thread->attr,
- gst_thread_main_loop, thread) != 0) {
- GST_DEBUG (GST_CAT_THREAD, "pthread_create failed");
- g_mutex_unlock (thread->lock);
- GST_DEBUG (GST_CAT_THREAD, "could not create thread \"%s\"",
- GST_ELEMENT_NAME (element));
- return GST_STATE_FAILURE;
+ /* 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;
}
- GST_DEBUG (GST_CAT_THREAD, "pthread created");
+ GST_DEBUG (GST_CAT_THREAD, "GThread created");
/* wait for it to 'spin up' */
- THR_DEBUG ("waiting for child thread spinup");
g_cond_wait (thread->cond, thread->lock);
- THR_DEBUG ("thread claims to be up");
- g_mutex_unlock (thread->lock);
break;
case GST_STATE_READY_TO_PAUSED:
- THR_INFO ("readying thread");
- g_mutex_lock (thread->lock);
- THR_DEBUG ("signaling");
- g_cond_signal (thread->cond);
- THR_DEBUG ("waiting for ack");
- g_cond_wait (thread->cond, thread->lock);
- THR_DEBUG ("got ack");
- g_mutex_unlock (thread->lock);
break;
case GST_STATE_PAUSED_TO_PLAYING:
{
- /* fixme: recurse into sub-bins */
- const GList *elements = gst_bin_get_list (GST_BIN (thread));
+ /* 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);
}
-
- THR_DEBUG ("telling thread to start spinning");
- g_mutex_lock (thread->lock);
- THR_DEBUG ("signaling");
- g_cond_signal (thread->cond);
- THR_DEBUG ("waiting for ack");
- g_cond_wait (thread->cond, thread->lock);
- THR_DEBUG ("got ack");
- g_mutex_unlock (thread->lock);
break;
}
case GST_STATE_PLAYING_TO_PAUSED:
{
- const GList *elements = (GList *) gst_bin_get_list (GST_BIN (thread));
-
- THR_INFO ("pausing thread");
-
- /* the following code ensures that the bottom half of thread will run
- * to perform each elements' change_state() (by calling gstbin.c::
- * change_state()).
- * + the pending state was already set by gstelement.c::set_state()
- * + unlock all elements so the bottom half can start the state change.
- */
- g_mutex_lock (thread->lock);
-
- GST_FLAG_UNSET (thread, GST_THREAD_STATE_SPINNING);
-
- while (elements) {
- GstElement *element = GST_ELEMENT (elements->data);
- GList *pads;
-
-
- g_assert (element);
- THR_DEBUG (" waking element \"%s\"", GST_ELEMENT_NAME (element));
- elements = g_list_next (elements);
-
- if (!gst_element_release_locks (element)) {
- g_warning ("element %s could not release locks", GST_ELEMENT_NAME (element));
- }
-
- pads = GST_ELEMENT_PADS (element);
-
- while (pads) {
- GstRealPad *peer = NULL;
- GstElement *peerelement;
-
- if (GST_PAD_PEER (pads->data))
- peer = GST_REAL_PAD (GST_PAD_PEER (pads->data));
-
- pads = g_list_next (pads);
-
- if (!peer)
- continue;
-
- peerelement = GST_PAD_PARENT (peer);
- if (!peerelement)
- continue; /* deal with case where there's no peer */
-
- if (!GST_FLAG_IS_SET (peerelement, GST_ELEMENT_DECOUPLED)) {
- GST_DEBUG (GST_CAT_THREAD, "peer element isn't DECOUPLED");
- continue;
- }
-
- if (GST_ELEMENT_SCHED (peerelement) != GST_ELEMENT_SCHED (thread)) {
- THR_DEBUG (" element \"%s\" has pad cross sched boundary", GST_ELEMENT_NAME (element));
- THR_DEBUG (" 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));
- }
- }
- }
-
- }
- THR_DEBUG ("telling thread to pause, signaling");
- g_cond_signal (thread->cond);
- THR_DEBUG ("waiting for ack");
- g_cond_wait (thread->cond, thread->lock);
- THR_DEBUG ("got ack");
- g_mutex_unlock (thread->lock);
-
- elements = gst_bin_get_list (GST_BIN (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:
- THR_DEBUG ("telling thread to pause (null) - and joining");
- /* MattH FIXME revisit */
- g_mutex_lock (thread->lock);
- THR_DEBUG ("signaling");
- g_cond_signal (thread->cond);
- THR_DEBUG ("waiting for ack");
- g_cond_wait (thread->cond, thread->lock);
- THR_DEBUG ("got ack");
-
- /* this block of code is very tricky
- * basically, we try to clean up the whole thread and
- * everything related to it in the right order without
- * triggering segfaults
- * compare this block to the block
- */
-
-
- /* in glibc 2.2.5, pthread_attr_destroy does nothing more
- * than return 0 */
- if (pthread_attr_destroy (&thread->attr) != 0)
- g_warning ("pthread_attr_destroy has failed !");
-
- GST_DEBUG (GST_CAT_THREAD, "joining pthread %ld", thread->thread_id);
- if (pthread_join (thread->thread_id, NULL) != 0)
- g_warning ("pthread_join has failed !\n");
-
- thread->thread_id = -1;
-
- /* the stack was allocated when we created the thread
- * using scheduler->get_preferred_stack */
- if (thread->stack) {
- GST_DEBUG (GST_CAT_THREAD, "freeing allocated stack (%p)",
- thread->stack);
- free (thread->stack);
- thread->stack = NULL;
- }
-
- THR_DEBUG ("unlocking mutex");
- g_mutex_unlock (thread->lock);
+ /* 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);
+ 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_FLAG_UNSET (thread, GST_THREAD_STATE_REAPING);
- GST_FLAG_UNSET (thread, GST_THREAD_STATE_STARTED);
- GST_FLAG_UNSET (thread, GST_THREAD_STATE_SPINNING);
+ GST_INFO (GST_CAT_THREAD, "gstthread: thread \"%s\" is stopped",
+ GST_ELEMENT_NAME (thread));
- break;
- case GST_STATE_PAUSED_TO_READY:
- THR_DEBUG ("telling thread to stop spinning");
- g_mutex_lock (thread->lock);
- THR_DEBUG ("signaling");
+ 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);
- THR_DEBUG ("waiting for ack");
g_cond_wait (thread->cond, thread->lock);
- THR_DEBUG ("got ack");
- g_mutex_unlock (thread->lock);
-
+ /* it should be dead now */
break;
default:
- GST_DEBUG_ELEMENT (GST_CAT_THREAD, element, "UNHANDLED STATE CHANGE! %x", transition);
+ 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
gst_thread_main_loop (void *arg)
{
GstThread *thread = NULL;
- gint stateset;
+ gboolean status;
- GST_DEBUG (GST_CAT_THREAD, "gst_thread_main_loop started");
thread = GST_THREAD (arg);
g_mutex_lock (thread->lock);
+ GST_DEBUG (GST_CAT_THREAD, "Thread %s started main loop", GST_ELEMENT_NAME (thread));
- /* handle scheduler policy; do stuff if not the normal scheduler */
- if (thread->sched_policy != SCHED_OTHER) {
- struct sched_param sched_param;
-
- memset (&sched_param, 0, sizeof (sched_param));
- if (thread->priority == 0) {
- thread->priority = sched_get_priority_max (thread->sched_policy);
- }
- sched_param.sched_priority = thread->priority;
-
- if (sched_setscheduler (0, thread->sched_policy, &sched_param) != 0) {
- GST_DEBUG (GST_CAT_THREAD, "not running with real-time priority");
- }
- }
+ /* initialize gst_thread_current */
+ g_private_set (gst_thread_current, thread);
/* set up the element's scheduler */
gst_scheduler_setup (GST_ELEMENT_SCHED (thread));
GST_FLAG_UNSET (thread, GST_THREAD_STATE_REAPING);
- thread->pid = getpid();
- THR_INFO_MAIN ("thread is running");
-
- /* first we need to change the state of all the children */
- if (GST_ELEMENT_CLASS (parent_class)->change_state) {
- stateset = GST_ELEMENT_CLASS (parent_class)->change_state (GST_ELEMENT(thread));
-
- if (stateset != GST_STATE_SUCCESS) {
- THR_DEBUG_MAIN ("state change of children failed");
- }
- }
-
- THR_DEBUG_MAIN ("indicating spinup");
g_cond_signal (thread->cond);
- /* don't unlock the mutex because we hold it into the top of the while loop */
- THR_DEBUG_MAIN ("thread has indicated spinup to parent process");
-
- /***** THREAD IS NOW IN READY STATE *****/
-
- /* CR1: most of this code is handshaking */
- /* do this while the thread lives */
- while (!GST_FLAG_IS_SET (thread, GST_THREAD_STATE_REAPING)) {
- /* NOTE we hold the thread lock at this point */
- /* what we do depends on what state we're in */
- switch (GST_STATE (thread)) {
- /* NOTE: cannot be in NULL, we're not running in that state at all */
- case GST_STATE_READY:
- /* wait to be set to either the NULL or PAUSED states */
- THR_DEBUG_MAIN ("thread in %s state, waiting for either %s or %s",
- gst_element_state_get_name (GST_STATE_READY),
- gst_element_state_get_name (GST_STATE_NULL),
- gst_element_state_get_name (GST_STATE_PAUSED));
- g_cond_wait (thread->cond, thread->lock);
-
- /* this must have happened by a state change in the thread context */
- if (GST_STATE_PENDING (thread) != GST_STATE_NULL &&
- GST_STATE_PENDING (thread) != GST_STATE_PAUSED) {
- g_cond_signal (thread->cond);
- continue;
- }
-
- /* been signaled, we need to state transition now and signal back */
- gst_thread_update_state (thread);
- THR_DEBUG_MAIN ("done with state transition, "
- "signaling back to parent process");
- g_cond_signal (thread->cond);
- /* now we decide what to do next */
- if (GST_STATE (thread) == GST_STATE_NULL) {
- /* REAPING must be set, we can simply break this iteration */
- THR_DEBUG_MAIN ("set GST_THREAD_STATE_REAPING");
- GST_FLAG_SET (thread, GST_THREAD_STATE_REAPING);
+ 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);
}
- continue;
-
- case GST_STATE_PAUSED:
- /* wait to be set to either the READY or PLAYING states */
- THR_DEBUG_MAIN ("thread in %s state, waiting for either %s or %s",
- gst_element_state_get_name (GST_STATE_PAUSED),
- gst_element_state_get_name (GST_STATE_READY),
- gst_element_state_get_name (GST_STATE_PLAYING));
- g_cond_wait (thread->cond, thread->lock);
-
- /* this must have happened by a state change in the thread context */
- if (GST_STATE_PENDING (thread) != GST_STATE_READY &&
- GST_STATE_PENDING (thread) != GST_STATE_PLAYING) {
- g_cond_signal (thread->cond);
- continue;
- }
-
- /* been signaled, we need to state transition now and signal back */
- gst_thread_update_state (thread);
- /* now we decide what to do next */
- if (GST_STATE (thread) != GST_STATE_PLAYING) {
- /* either READY or the state change failed for some reason */
- g_cond_signal (thread->cond);
- continue;
- }
- else {
- GST_FLAG_SET (thread, GST_THREAD_STATE_SPINNING);
- /* PLAYING is coming up, so we can now start spinning */
- while (GST_FLAG_IS_SET (thread, GST_THREAD_STATE_SPINNING)) {
- gboolean status;
-
- g_cond_signal (thread->cond);
- g_mutex_unlock (thread->lock);
- status = gst_bin_iterate (GST_BIN (thread));
- g_mutex_lock (thread->lock);
- /* g_cond_signal(thread->cond); */
-
- if (!status || GST_STATE_PENDING (thread) != GST_STATE_VOID_PENDING)
- GST_FLAG_UNSET (thread, GST_THREAD_STATE_SPINNING);
- }
- /* looks like we were stopped because of a statechange */
- if (GST_STATE_PENDING (thread)) {
- gst_thread_update_state (thread);
- }
- /* once we're here, SPINNING has stopped, we should signal that we're done */
- THR_DEBUG_MAIN ("SPINNING stopped, signaling back to parent process");
- g_cond_signal (thread->cond);
- /* now we can wait for PAUSED */
- continue;
- }
- case GST_STATE_PLAYING:
- /* wait to be set to PAUSED */
- THR_DEBUG_MAIN ("thread in %s state, waiting for %s",
- gst_element_state_get_name (GST_STATE_PLAYING),
- gst_element_state_get_name (GST_STATE_PAUSED));
- g_cond_wait (thread->cond,thread->lock);
-
- /* been signaled, we need to state transition now and signal back */
- gst_thread_update_state (thread);
- g_cond_signal (thread->cond);
- /* now we decide what to do next */
- /* there's only PAUSED, we we just wait for it */
- continue;
- case GST_STATE_NULL:
- THR_DEBUG_MAIN ("thread in %s state, preparing to die",
- gst_element_state_get_name (GST_STATE_NULL));
- GST_FLAG_SET (thread, GST_THREAD_STATE_REAPING);
- break;
- default:
- g_assert_not_reached ();
- break;
+ }
+ 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);
}
- /* THREAD HAS STOPPED RUNNING */
-
/* 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));
- /* since we don't unlock at the end of the while loop, do it here */
+ /* 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);
+
+ /* unlock and signal - we are out */
+ g_cond_signal (thread->cond);
g_mutex_unlock (thread->lock);
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);
+ GST_ELEMENT_NAME (thread));
return NULL;
}