mirror of
https://gitlab.freedesktop.org/gstreamer/gstreamer.git
synced 2024-12-25 01:30:38 +00:00
update for the latest jack cvs and non-cothreaded gst scheduler
Original commit message from CVS: update for the latest jack cvs and non-cothreaded gst scheduler
This commit is contained in:
parent
42aadabc81
commit
3503def596
3 changed files with 66 additions and 82 deletions
|
@ -263,9 +263,9 @@ gst_jack_request_new_pad (GstElement *element, GstPadTemplate *templ, const gcha
|
|||
newname = g_strdup (name);
|
||||
} else {
|
||||
if (this->direction == GST_PAD_SINK)
|
||||
newname = g_strdup ("alsa_pcm:out_1");
|
||||
newname = g_strdup ("alsa_pcm:playback_1");
|
||||
else
|
||||
newname = g_strdup ("alsa_pcm:in_1");
|
||||
newname = g_strdup ("alsa_pcm:capture_1");
|
||||
}
|
||||
|
||||
pad = g_new0(GstJackPad, 1);
|
||||
|
@ -337,12 +337,12 @@ gst_jack_change_state (GstElement *element)
|
|||
|
||||
switch (GST_STATE_PENDING (element)) {
|
||||
case GST_STATE_NULL:
|
||||
g_message ("jack client %s: NULL", GST_OBJECT_NAME (GST_OBJECT (this)));
|
||||
JACK_DEBUG ("%s: NULL", GST_OBJECT_NAME (GST_OBJECT (this)));
|
||||
|
||||
break;
|
||||
|
||||
case GST_STATE_READY:
|
||||
g_message ("jack client %s: READY", GST_OBJECT_NAME (GST_OBJECT (this)));
|
||||
JACK_DEBUG ("%s: READY", GST_OBJECT_NAME (GST_OBJECT (this)));
|
||||
|
||||
if (!this->bin) {
|
||||
if (!(this->bin = (GstJackBin*)gst_element_get_managing_bin (element))
|
||||
|
@ -358,7 +358,7 @@ gst_jack_change_state (GstElement *element)
|
|||
pads = (this->direction == GST_PAD_SRC) ? &this->bin->src_pads : &this->bin->sink_pads;
|
||||
while (l) {
|
||||
pad = GST_JACK_PAD (l);
|
||||
g_message ("jack: appending pad %s:%s to list", pad->name, pad->peer_name);
|
||||
JACK_DEBUG ("%s: appending pad %s:%s to list", GST_OBJECT_NAME (this), pad->name, pad->peer_name);
|
||||
*pads = g_list_append (*pads, pad);
|
||||
l = g_list_next (l);
|
||||
}
|
||||
|
@ -366,7 +366,7 @@ gst_jack_change_state (GstElement *element)
|
|||
break;
|
||||
|
||||
case GST_STATE_PAUSED:
|
||||
g_message ("jack client %s: PAUSED", GST_OBJECT_NAME (GST_OBJECT (this)));
|
||||
JACK_DEBUG ("%s: PAUSED", GST_OBJECT_NAME (GST_OBJECT (this)));
|
||||
|
||||
if (GST_STATE (element) == GST_STATE_READY) {
|
||||
/* we're in READY->PAUSED */
|
||||
|
@ -383,11 +383,11 @@ gst_jack_change_state (GstElement *element)
|
|||
}
|
||||
break;
|
||||
case GST_STATE_PLAYING:
|
||||
g_message ("jack client %s: PLAYING", GST_OBJECT_NAME (GST_OBJECT (this)));
|
||||
JACK_DEBUG ("%s: PLAYING", GST_OBJECT_NAME (GST_OBJECT (this)));
|
||||
break;
|
||||
}
|
||||
|
||||
g_message ("jack: state change finished");
|
||||
JACK_DEBUG ("%s: state change finished", GST_OBJECT_NAME (this));
|
||||
|
||||
if (GST_ELEMENT_CLASS (parent_class)->change_state)
|
||||
return GST_ELEMENT_CLASS (parent_class)->change_state (element);
|
||||
|
@ -455,7 +455,7 @@ gst_jack_loop (GstElement *element)
|
|||
* bytes, but hey. */
|
||||
gst_element_set_eos (element);
|
||||
gst_event_unref (event);
|
||||
gst_element_yield (element); /* shouldn't return */
|
||||
// gst_element_yield (element); /* shouldn't return */
|
||||
return;
|
||||
}
|
||||
goto read;
|
||||
|
@ -481,7 +481,8 @@ gst_jack_loop (GstElement *element)
|
|||
pads = g_list_next (pads);
|
||||
}
|
||||
|
||||
gst_element_yield (element);
|
||||
return;
|
||||
// gst_element_yield (element);
|
||||
} while (TRUE);
|
||||
}
|
||||
|
||||
|
|
|
@ -24,6 +24,8 @@
|
|||
#include <gst/gst.h>
|
||||
#include <gst/bytestream/bytestream.h>
|
||||
|
||||
//#define JACK_DEBUG(str, a...) g_message (str, ##a)
|
||||
#define JACK_DEBUG(str, a...)
|
||||
|
||||
#define GST_JACK(obj) G_TYPE_CHECK_INSTANCE_CAST(obj, GST_TYPE_JACK, GstJack)
|
||||
#define GST_JACK_CLASS(klass) G_TYPE_CHECK_CLASS_CAST(klass, GST_TYPE_JACK, GstJackClass)
|
||||
|
@ -110,12 +112,6 @@ struct _GstJackBin {
|
|||
|
||||
guint rate;
|
||||
jack_nframes_t nframes;
|
||||
|
||||
/* the scheduler needs to be setup from within the jack client thread; this
|
||||
variable is to keep track of whether or not we have been set up yet */
|
||||
gboolean sched_setup;
|
||||
GCond *cond;
|
||||
GMutex *lock;
|
||||
};
|
||||
|
||||
struct _GstJackBinClass {
|
||||
|
|
|
@ -19,6 +19,7 @@
|
|||
|
||||
#include <stdlib.h>
|
||||
#include <string.h>
|
||||
#include <signal.h>
|
||||
#include "gstjack.h"
|
||||
|
||||
|
||||
|
@ -31,16 +32,20 @@ static GstElementStateReturn gst_jack_bin_change_state(GstElement *element);
|
|||
|
||||
/* jack callbacks */
|
||||
static int process (jack_nframes_t nframes, void *arg);
|
||||
static int buffer_size (jack_nframes_t nframes, void *arg);
|
||||
static int sample_rate (jack_nframes_t nframes, void *arg);
|
||||
static void shutdown (void *arg);
|
||||
|
||||
static void sighup_handler (int sig);
|
||||
static GstJackBin *_jackbin = NULL;
|
||||
static gboolean watchdog_please_set_the_jackbin_to_ready = FALSE;
|
||||
|
||||
/* fixme: we need a watchdog thread to see if we have received a SIGHUP, and if
|
||||
* so set the state of the bin to READY. */
|
||||
|
||||
GType
|
||||
gst_jack_bin_get_type (void)
|
||||
{
|
||||
static GType jack_bin_type = 0;
|
||||
|
||||
if (!jack_bin_type) {
|
||||
static const GTypeInfo jack_bin_info = {
|
||||
sizeof(GstJackBinClass),
|
||||
|
@ -83,11 +88,6 @@ gst_jack_bin_init(GstJackBin *this)
|
|||
|
||||
/* make a new scheduler and associate it with the bin */
|
||||
gst_scheduler_factory_make (NULL, GST_ELEMENT (this));
|
||||
|
||||
this->cond = g_cond_new ();
|
||||
this->lock = g_mutex_new ();
|
||||
g_mutex_lock (this->lock);
|
||||
this->sched_setup = FALSE;
|
||||
}
|
||||
|
||||
static GstElementStateReturn
|
||||
|
@ -102,27 +102,36 @@ gst_jack_bin_change_state (GstElement *element)
|
|||
|
||||
switch (GST_STATE_PENDING (element)) {
|
||||
case GST_STATE_NULL:
|
||||
g_message ("jackbin: NULL state");
|
||||
JACK_DEBUG ("jackbin: NULL state");
|
||||
if (this->client) {
|
||||
g_message ("jack: closing client");
|
||||
JACK_DEBUG ("jackbin: closing client");
|
||||
jack_client_close (this->client);
|
||||
this->client = NULL;
|
||||
}
|
||||
|
||||
if (_jackbin)
|
||||
signal (SIGHUP, SIG_DFL);
|
||||
_jackbin = NULL;
|
||||
|
||||
if (GST_ELEMENT_CLASS (parent_class)->change_state)
|
||||
return GST_ELEMENT_CLASS (parent_class)->change_state (element);
|
||||
break;
|
||||
|
||||
case GST_STATE_READY:
|
||||
g_message ("jackbin: READY");
|
||||
JACK_DEBUG ("jackbin: READY");
|
||||
|
||||
_jackbin = this;
|
||||
signal (SIGHUP, sighup_handler);
|
||||
|
||||
if (!this->client) {
|
||||
if (!(this->client = jack_client_new ("gst-jack"))) {
|
||||
g_warning ("jack server not running?");
|
||||
return GST_STATE_FAILURE;
|
||||
}
|
||||
|
||||
gst_scheduler_setup (GST_ELEMENT_SCHED (this));
|
||||
|
||||
jack_set_process_callback (this->client, process, this);
|
||||
jack_set_buffer_size_callback (this->client, buffer_size, this);
|
||||
jack_set_sample_rate_callback (this->client, sample_rate, this);
|
||||
jack_on_shutdown (this->client, shutdown, this);
|
||||
}
|
||||
|
@ -130,20 +139,20 @@ gst_jack_bin_change_state (GstElement *element)
|
|||
if (GST_FLAG_IS_SET (GST_OBJECT (this), GST_JACK_OPEN)) {
|
||||
l = this->src_pads;
|
||||
while (l) {
|
||||
g_message ("jack: unregistering pad %s:%s", GST_JACK_PAD (l)->name, GST_JACK_PAD (l)->peer_name);
|
||||
JACK_DEBUG ("jackbin: unregistering pad %s:%s", GST_JACK_PAD (l)->name, GST_JACK_PAD (l)->peer_name);
|
||||
jack_port_unregister (this->client, GST_JACK_PAD (l)->port);
|
||||
l = g_list_next (l);
|
||||
}
|
||||
l = this->sink_pads;
|
||||
while (l) {
|
||||
g_message ("jack: unregistering pad %s:%s", GST_JACK_PAD (l)->name, GST_JACK_PAD (l)->peer_name);
|
||||
JACK_DEBUG ("jackbin: unregistering pad %s:%s", GST_JACK_PAD (l)->name, GST_JACK_PAD (l)->peer_name);
|
||||
jack_port_unregister (this->client, GST_JACK_PAD (l)->port);
|
||||
l = g_list_next (l);
|
||||
}
|
||||
GST_FLAG_UNSET (GST_OBJECT (this), GST_JACK_OPEN);
|
||||
|
||||
if (GST_FLAG_IS_SET (GST_OBJECT (this), GST_JACK_ACTIVE)) {
|
||||
g_message ("jack: deactivating client");
|
||||
JACK_DEBUG ("jackbin: deactivating client");
|
||||
jack_deactivate (this->client);
|
||||
GST_FLAG_UNSET (GST_OBJECT (this), GST_JACK_ACTIVE);
|
||||
}
|
||||
|
@ -154,39 +163,37 @@ gst_jack_bin_change_state (GstElement *element)
|
|||
break;
|
||||
|
||||
case GST_STATE_PAUSED:
|
||||
g_message ("jackbin: PAUSED");
|
||||
JACK_DEBUG ("jackbin: PAUSED");
|
||||
|
||||
if (!GST_FLAG_IS_SET (GST_OBJECT (this), GST_JACK_OPEN)) {
|
||||
l = this->src_pads;
|
||||
while (l) {
|
||||
pad = GST_JACK_PAD (l);
|
||||
g_message ("jack: registering input port %s (peer %s)", pad->name, pad->peer_name);
|
||||
JACK_DEBUG ("jackbin: registering input port %s (peer %s)", pad->name, pad->peer_name);
|
||||
pad->port = jack_port_register (this->client, pad->name, JACK_DEFAULT_AUDIO_TYPE, JackPortIsInput|JackPortIsTerminal, 0);
|
||||
l = g_list_next (l);
|
||||
}
|
||||
l = this->sink_pads;
|
||||
while (l) {
|
||||
pad = GST_JACK_PAD (l);
|
||||
g_message ("jack: registering output port %s (peer %s)", pad->name, pad->peer_name);
|
||||
JACK_DEBUG ("jackbin: registering output port %s (peer %s)", pad->name, pad->peer_name);
|
||||
pad->port = jack_port_register (this->client, pad->name, JACK_DEFAULT_AUDIO_TYPE, JackPortIsOutput|JackPortIsTerminal, 0);
|
||||
l = g_list_next (l);
|
||||
}
|
||||
|
||||
/* must activate before connecting */
|
||||
if (!GST_FLAG_IS_SET (GST_OBJECT (this), GST_JACK_ACTIVE)) {
|
||||
g_message ("jack: activating client");
|
||||
JACK_DEBUG ("jackbin: activating client");
|
||||
jack_activate (this->client);
|
||||
GST_FLAG_SET (GST_OBJECT (this), GST_JACK_ACTIVE);
|
||||
}
|
||||
|
||||
g_cond_wait (this->cond, this->lock);
|
||||
|
||||
l = this->src_pads;
|
||||
while (l) {
|
||||
pad = GST_JACK_PAD (l);
|
||||
g_message ("connecting jack port %s to gst jack port %s", pad->peer_name, jack_port_name (pad->port));
|
||||
JACK_DEBUG ("connecting jack port %s to gst jack port %s", pad->peer_name, jack_port_name (pad->port));
|
||||
if (jack_connect (this->client, pad->peer_name, jack_port_name (pad->port))) {
|
||||
g_warning ("jack: could not connect %s and %s", pad->peer_name, jack_port_name (pad->port));
|
||||
g_warning ("jackbin: could not connect %s and %s", pad->peer_name, jack_port_name (pad->port));
|
||||
return GST_STATE_FAILURE;
|
||||
}
|
||||
l = g_list_next (l);
|
||||
|
@ -194,29 +201,34 @@ gst_jack_bin_change_state (GstElement *element)
|
|||
l = this->sink_pads;
|
||||
while (l) {
|
||||
pad = GST_JACK_PAD (l);
|
||||
g_message ("connecting gst jack port %s to jack port %s", jack_port_name (pad->port), pad->peer_name);
|
||||
JACK_DEBUG ("connecting gst jack port %s to jack port %s", jack_port_name (pad->port), pad->peer_name);
|
||||
if (jack_connect (this->client, jack_port_name (pad->port), pad->peer_name)) {
|
||||
g_warning ("jack: could not connect %s and %s", pad->peer_name, jack_port_name (pad->port));
|
||||
g_warning ("jackbin: could not connect %s and %s", pad->peer_name, jack_port_name (pad->port));
|
||||
return GST_STATE_FAILURE;
|
||||
}
|
||||
l = g_list_next (l);
|
||||
}
|
||||
|
||||
g_message ("jackbin: setting OPEN flag");
|
||||
JACK_DEBUG ("jackbin: setting OPEN flag");
|
||||
GST_FLAG_SET (GST_OBJECT (this), GST_JACK_OPEN);
|
||||
|
||||
if (GST_ELEMENT_CLASS (parent_class)->change_state)
|
||||
return GST_ELEMENT_CLASS (parent_class)->change_state (element);
|
||||
} else {
|
||||
g_cond_wait (this->cond, this->lock);
|
||||
if (GST_ELEMENT_CLASS (parent_class)->change_state)
|
||||
return GST_ELEMENT_CLASS (parent_class)->change_state (element);
|
||||
}
|
||||
|
||||
break;
|
||||
case GST_STATE_PLAYING:
|
||||
g_message ("jack: PLAYING");
|
||||
JACK_DEBUG ("jackbin: PLAYING");
|
||||
|
||||
g_cond_wait (this->cond, this->lock);
|
||||
if (GST_ELEMENT_CLASS (parent_class)->change_state)
|
||||
return GST_ELEMENT_CLASS (parent_class)->change_state (element);
|
||||
break;
|
||||
}
|
||||
|
||||
g_message ("jack: state change finished");
|
||||
JACK_DEBUG ("jackbin: state change finished");
|
||||
|
||||
return GST_STATE_SUCCESS;
|
||||
}
|
||||
|
@ -234,37 +246,13 @@ process (jack_nframes_t nframes, void *arg)
|
|||
|
||||
g_assert (bin);
|
||||
|
||||
g_message ("jack: process()");
|
||||
|
||||
if (GST_STATE_PENDING (bin) != GST_STATE_VOID_PENDING) {
|
||||
g_message ("jackbin: doing state change from %s to %s",
|
||||
gst_element_state_get_name (GST_STATE (bin)),
|
||||
gst_element_state_get_name (GST_STATE_PENDING (bin)));
|
||||
|
||||
/* FIXME: this breaks ultra-low latency... */
|
||||
g_mutex_lock (bin->lock);
|
||||
|
||||
switch (GST_STATE_TRANSITION (bin)) {
|
||||
case GST_STATE_READY_TO_PAUSED:
|
||||
if (!bin->sched_setup) {
|
||||
gst_scheduler_setup (GST_ELEMENT_SCHED (bin));
|
||||
bin->sched_setup = TRUE;
|
||||
}
|
||||
break;
|
||||
}
|
||||
|
||||
/* do the chaining up from within the jack thread, so that child
|
||||
elements are initialized from within the proper thread */
|
||||
if (GST_ELEMENT_CLASS (parent_class)->change_state)
|
||||
GST_ELEMENT_CLASS (parent_class)->change_state (GST_ELEMENT_CAST (bin));
|
||||
|
||||
g_cond_signal (bin->cond);
|
||||
g_mutex_unlock (bin->lock);
|
||||
}
|
||||
JACK_DEBUG ("jackbin: process()");
|
||||
|
||||
if (GST_STATE (bin) != GST_STATE_PLAYING) {
|
||||
g_message ("jackbin: bin is not PLAYING yet, returning");
|
||||
JACK_DEBUG ("jackbin: bin is not PLAYING yet, returning");
|
||||
return 0;
|
||||
} else {
|
||||
JACK_DEBUG ("jackbin: we are PLAYING, let's process()");
|
||||
}
|
||||
|
||||
l = bin->src_pads;
|
||||
|
@ -283,6 +271,7 @@ process (jack_nframes_t nframes, void *arg)
|
|||
|
||||
bin->nframes = nframes;
|
||||
|
||||
JACK_DEBUG ("jackbin: iterating to process %ld frames of audio...", nframes);
|
||||
if (!gst_bin_iterate (GST_BIN_CAST (bin))) {
|
||||
g_warning ("bin failed to iterate");
|
||||
return -1;
|
||||
|
@ -293,18 +282,11 @@ process (jack_nframes_t nframes, void *arg)
|
|||
return 0;
|
||||
}
|
||||
|
||||
static int
|
||||
buffer_size (jack_nframes_t nframes, void *arg)
|
||||
{
|
||||
printf ("the maximum buffer size is now %lu\n", nframes);
|
||||
return 0;
|
||||
}
|
||||
|
||||
static int
|
||||
sample_rate (jack_nframes_t nframes, void *arg)
|
||||
{
|
||||
GstJackBin *bin = (GstJackBin*) arg;
|
||||
printf ("the sample rate is now %lu/sec\n", nframes);
|
||||
JACK_DEBUG ("the sample rate is now %lu/sec\n", nframes);
|
||||
bin->rate = nframes;
|
||||
return 0;
|
||||
}
|
||||
|
@ -317,3 +299,8 @@ shutdown (void *arg)
|
|||
/* gst_element_set_state (GST_ELEMENT (client->manager), GST_STATE_READY); */
|
||||
}
|
||||
|
||||
static void sighup_handler (int sig)
|
||||
{
|
||||
g_message ("got sighup, setting state to READY");
|
||||
watchdog_please_set_the_jackbin_to_ready = TRUE;
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue