Commit f0e85023 authored by Sebastian Dröge's avatar Sebastian Dröge 🍵

decklink{audio,video}src: Implement clock slaving if the pipeline clock is not the decklink clock

parent 408f0870
...@@ -476,12 +476,35 @@ public: ...@@ -476,12 +476,35 @@ public:
{ {
GstElement *videosrc = NULL, *audiosrc = NULL; GstElement *videosrc = NULL, *audiosrc = NULL;
void (*got_video_frame) (GstElement * videosrc, void (*got_video_frame) (GstElement * videosrc,
IDeckLinkVideoInputFrame * frame, GstDecklinkModeEnum mode) = NULL; IDeckLinkVideoInputFrame * frame, GstDecklinkModeEnum mode,
GstClockTime capture_time, GstClockTime capture_duration) = NULL;
void (*got_audio_packet) (GstElement * videosrc, void (*got_audio_packet) (GstElement * videosrc,
IDeckLinkAudioInputPacket * packet) = NULL; IDeckLinkAudioInputPacket * packet, GstClockTime capture_time) = NULL;
GstDecklinkModeEnum mode; GstDecklinkModeEnum mode;
BMDTimeValue capture_time, capture_duration;
HRESULT res;
res =
video_frame->GetHardwareReferenceTimestamp (GST_SECOND, &capture_time,
&capture_duration);
if (res != S_OK) {
GST_ERROR ("Failed to get capture time: 0x%08x", res);
capture_time = GST_CLOCK_TIME_NONE;
capture_duration = GST_CLOCK_TIME_NONE;
}
g_mutex_lock (&m_input->lock); g_mutex_lock (&m_input->lock);
if (capture_time > m_input->clock_start_time)
capture_time -= m_input->clock_start_time;
else
capture_time = 0;
if (capture_time > m_input->clock_offset)
capture_time -= m_input->clock_offset;
else
capture_time = 0;
if (m_input->videosrc) { if (m_input->videosrc) {
videosrc = GST_ELEMENT_CAST (gst_object_ref (m_input->videosrc)); videosrc = GST_ELEMENT_CAST (gst_object_ref (m_input->videosrc));
got_video_frame = m_input->got_video_frame; got_video_frame = m_input->got_video_frame;
...@@ -495,11 +518,12 @@ public: ...@@ -495,11 +518,12 @@ public:
g_mutex_unlock (&m_input->lock); g_mutex_unlock (&m_input->lock);
if (got_video_frame && videosrc) { if (got_video_frame && videosrc) {
got_video_frame (videosrc, video_frame, mode); got_video_frame (videosrc, video_frame, mode, capture_time,
capture_duration);
} }
if (got_audio_packet && audiosrc) { if (got_audio_packet && audiosrc) {
m_input->got_audio_packet (audiosrc, audio_packet); m_input->got_audio_packet (audiosrc, audio_packet, capture_time);
} }
gst_object_replace ((GstObject **) & videosrc, NULL); gst_object_replace ((GstObject **) & videosrc, NULL);
......
...@@ -169,12 +169,12 @@ struct _GstDecklinkInput { ...@@ -169,12 +169,12 @@ struct _GstDecklinkInput {
GMutex lock; GMutex lock;
/* Set by the video source */ /* Set by the video source */
void (*got_video_frame) (GstElement *videosrc, IDeckLinkVideoInputFrame * frame, GstDecklinkModeEnum mode); void (*got_video_frame) (GstElement *videosrc, IDeckLinkVideoInputFrame * frame, GstDecklinkModeEnum mode, GstClockTime capture_time, GstClockTime capture_duration);
/* Configured mode or NULL */ /* Configured mode or NULL */
const GstDecklinkMode *mode; const GstDecklinkMode *mode;
/* Set by the audio source */ /* Set by the audio source */
void (*got_audio_packet) (GstElement *videosrc, IDeckLinkAudioInputPacket * packet); void (*got_audio_packet) (GstElement *videosrc, IDeckLinkAudioInputPacket * packet, GstClockTime capture_time);
GstElement *audiosrc; GstElement *audiosrc;
gboolean audio_enabled; gboolean audio_enabled;
......
...@@ -23,6 +23,7 @@ ...@@ -23,6 +23,7 @@
#endif #endif
#include "gstdecklinkaudiosrc.h" #include "gstdecklinkaudiosrc.h"
#include "gstdecklinkvideosrc.h"
#include <string.h> #include <string.h>
GST_DEBUG_CATEGORY_STATIC (gst_decklink_audio_src_debug); GST_DEBUG_CATEGORY_STATIC (gst_decklink_audio_src_debug);
...@@ -408,29 +409,26 @@ gst_decklink_audio_src_get_caps (GstBaseSrc * bsrc, GstCaps * filter) ...@@ -408,29 +409,26 @@ gst_decklink_audio_src_get_caps (GstBaseSrc * bsrc, GstCaps * filter)
static void static void
gst_decklink_audio_src_got_packet (GstElement * element, gst_decklink_audio_src_got_packet (GstElement * element,
IDeckLinkAudioInputPacket * packet) IDeckLinkAudioInputPacket * packet, GstClockTime capture_time)
{ {
GstDecklinkAudioSrc *self = GST_DECKLINK_AUDIO_SRC_CAST (element); GstDecklinkAudioSrc *self = GST_DECKLINK_AUDIO_SRC_CAST (element);
GstClock *clock; GstDecklinkVideoSrc *videosrc = NULL;
GstClockTime capture_time;
clock = gst_element_get_clock (element);
if (clock) {
GstClockTime clock_time, base_time;
clock_time = gst_clock_get_time (clock);
g_return_if_fail (clock_time != GST_CLOCK_TIME_NONE);
base_time = gst_element_get_base_time (element);
g_return_if_fail (base_time != GST_CLOCK_TIME_NONE);
capture_time = clock_time - base_time;
gst_object_unref (clock);
} else {
capture_time = GST_CLOCK_TIME_NONE;
}
GST_LOG_OBJECT (self, "Got audio packet at %" GST_TIME_FORMAT, GST_LOG_OBJECT (self, "Got audio packet at %" GST_TIME_FORMAT,
GST_TIME_ARGS (capture_time)); GST_TIME_ARGS (capture_time));
g_mutex_lock (&self->input->lock);
if (self->input->videosrc)
videosrc =
GST_DECKLINK_VIDEO_SRC_CAST (gst_object_ref (self->input->videosrc));
g_mutex_unlock (&self->input->lock);
if (videosrc) {
gst_decklink_video_src_convert_to_external_clock (videosrc, &capture_time,
NULL);
gst_object_unref (videosrc);
}
g_mutex_lock (&self->lock); g_mutex_lock (&self->lock);
if (!self->flushing) { if (!self->flushing) {
CapturePacket *p; CapturePacket *p;
...@@ -782,4 +780,3 @@ out: ...@@ -782,4 +780,3 @@ out:
return ret; return ret;
} }
...@@ -45,7 +45,7 @@ enum ...@@ -45,7 +45,7 @@ enum
typedef struct typedef struct
{ {
IDeckLinkVideoInputFrame *frame; IDeckLinkVideoInputFrame *frame;
GstClockTime capture_time; GstClockTime capture_time, capture_duration;
GstDecklinkModeEnum mode; GstDecklinkModeEnum mode;
} CaptureFrame; } CaptureFrame;
...@@ -362,31 +362,90 @@ gst_decklink_video_src_get_caps (GstBaseSrc * bsrc, GstCaps * filter) ...@@ -362,31 +362,90 @@ gst_decklink_video_src_get_caps (GstBaseSrc * bsrc, GstCaps * filter)
return caps; return caps;
} }
static void void
gst_decklink_video_src_got_frame (GstElement * element, gst_decklink_video_src_convert_to_external_clock (GstDecklinkVideoSrc * self,
IDeckLinkVideoInputFrame * frame, GstDecklinkModeEnum mode) GstClockTime * timestamp, GstClockTime * duration)
{ {
GstDecklinkVideoSrc *self = GST_DECKLINK_VIDEO_SRC_CAST (element);
GstClock *clock; GstClock *clock;
GstClockTime capture_time;
g_assert (timestamp != NULL);
clock = gst_element_get_clock (element);
if (clock) { if (*timestamp == GST_CLOCK_TIME_NONE)
GstClockTime clock_time, base_time; return;
clock_time = gst_clock_get_time (clock); clock = gst_element_get_clock (GST_ELEMENT_CAST (self));
g_return_if_fail (clock_time != GST_CLOCK_TIME_NONE); if (clock && clock != self->input->clock) {
base_time = gst_element_get_base_time (element); GstClockTime internal, external, rate_n, rate_d;
g_return_if_fail (base_time != GST_CLOCK_TIME_NONE); gst_clock_get_calibration (self->input->clock, &internal, &external,
capture_time = clock_time - base_time; &rate_n, &rate_d);
gst_object_unref (clock);
if (rate_n != rate_d && self->internal_base_time != GST_CLOCK_TIME_NONE) {
GstClockTime internal_timestamp = *timestamp;
// Convert to the running time corresponding to both clock times
internal -= self->internal_base_time;
external -= self->external_base_time;
// Get the difference in the internal time, note
// that the capture time is internal time.
// Then scale this difference and offset it to
// our external time. Now we have the running time
// according to our external clock.
//
// For the duration we just scale
if (internal > internal_timestamp) {
guint64 diff = internal - internal_timestamp;
diff = gst_util_uint64_scale (diff, rate_d, rate_n);
*timestamp = external - diff;
} else {
guint64 diff = internal_timestamp - internal;
diff = gst_util_uint64_scale (diff, rate_d, rate_n);
*timestamp = external + diff;
}
GST_LOG_OBJECT (self,
"Converted %" GST_TIME_FORMAT " to %" GST_TIME_FORMAT " (external: %"
GST_TIME_FORMAT " internal %" GST_TIME_FORMAT " rate: %lf)",
GST_TIME_ARGS (internal_timestamp), GST_TIME_ARGS (*timestamp),
GST_TIME_ARGS (external), GST_TIME_ARGS (internal),
((gdouble) rate_n) / ((gdouble) rate_d));
if (duration) {
GstClockTime internal_duration = *duration;
*duration = gst_util_uint64_scale (internal_duration, rate_d, rate_n);
GST_LOG_OBJECT (self,
"Converted duration %" GST_TIME_FORMAT " to %" GST_TIME_FORMAT
" (external: %" GST_TIME_FORMAT " internal %" GST_TIME_FORMAT
" rate: %lf)", GST_TIME_ARGS (internal_duration),
GST_TIME_ARGS (*duration), GST_TIME_ARGS (external),
GST_TIME_ARGS (internal), ((gdouble) rate_n) / ((gdouble) rate_d));
}
} else {
GST_LOG_OBJECT (self, "No clock conversion needed, relative rate is 1.0");
}
} else { } else {
capture_time = GST_CLOCK_TIME_NONE; GST_LOG_OBJECT (self, "No clock conversion needed, same clocks");
} }
}
static void
gst_decklink_video_src_got_frame (GstElement * element,
IDeckLinkVideoInputFrame * frame, GstDecklinkModeEnum mode,
GstClockTime capture_time, GstClockTime capture_duration)
{
GstDecklinkVideoSrc *self = GST_DECKLINK_VIDEO_SRC_CAST (element);
GST_LOG_OBJECT (self, "Got video frame at %" GST_TIME_FORMAT, GST_LOG_OBJECT (self, "Got video frame at %" GST_TIME_FORMAT,
GST_TIME_ARGS (capture_time)); GST_TIME_ARGS (capture_time));
gst_decklink_video_src_convert_to_external_clock (self, &capture_time,
&capture_duration);
GST_LOG_OBJECT (self, "Actual timestamp %" GST_TIME_FORMAT,
GST_TIME_ARGS (capture_time));
g_mutex_lock (&self->lock); g_mutex_lock (&self->lock);
if (!self->flushing) { if (!self->flushing) {
CaptureFrame *f; CaptureFrame *f;
...@@ -401,6 +460,7 @@ gst_decklink_video_src_got_frame (GstElement * element, ...@@ -401,6 +460,7 @@ gst_decklink_video_src_got_frame (GstElement * element,
f = (CaptureFrame *) g_malloc0 (sizeof (CaptureFrame)); f = (CaptureFrame *) g_malloc0 (sizeof (CaptureFrame));
f->frame = frame; f->frame = frame;
f->capture_time = capture_time; f->capture_time = capture_time;
f->capture_duration = capture_duration;
f->mode = mode; f->mode = mode;
frame->AddRef (); frame->AddRef ();
g_queue_push_tail (&self->current_frames, f); g_queue_push_tail (&self->current_frames, f);
...@@ -418,7 +478,6 @@ gst_decklink_video_src_create (GstPushSrc * bsrc, GstBuffer ** buffer) ...@@ -418,7 +478,6 @@ gst_decklink_video_src_create (GstPushSrc * bsrc, GstBuffer ** buffer)
gsize data_size; gsize data_size;
VideoFrame *vf; VideoFrame *vf;
CaptureFrame *f; CaptureFrame *f;
GstClockTime timestamp, duration;
GstCaps *caps; GstCaps *caps;
g_mutex_lock (&self->lock); g_mutex_lock (&self->lock);
...@@ -467,18 +526,8 @@ gst_decklink_video_src_create (GstPushSrc * bsrc, GstBuffer ** buffer) ...@@ -467,18 +526,8 @@ gst_decklink_video_src_create (GstPushSrc * bsrc, GstBuffer ** buffer)
vf->input = self->input->input; vf->input = self->input->input;
vf->input->AddRef (); vf->input->AddRef ();
duration = GST_BUFFER_TIMESTAMP (*buffer) = f->capture_time;
gst_util_uint64_scale_int (GST_SECOND, self->info.fps_d, GST_BUFFER_DURATION (*buffer) = f->capture_duration;
self->info.fps_n);
// Our capture time is the end timestamp, subtract the
// duration to get the start timestamp
if (f->capture_time >= duration)
timestamp = f->capture_time - duration;
else
timestamp = 0;
GST_BUFFER_TIMESTAMP (*buffer) = timestamp;
GST_BUFFER_DURATION (*buffer) = duration;
capture_frame_free (f); capture_frame_free (f);
...@@ -614,6 +663,22 @@ gst_decklink_video_src_start_streams (GstElement * element) ...@@ -614,6 +663,22 @@ gst_decklink_video_src_start_streams (GstElement * element)
&& (GST_STATE (self) == GST_STATE_PLAYING && (GST_STATE (self) == GST_STATE_PLAYING
|| GST_STATE_PENDING (self) == GST_STATE_PLAYING)) { || GST_STATE_PENDING (self) == GST_STATE_PLAYING)) {
GST_DEBUG_OBJECT (self, "Starting streams"); GST_DEBUG_OBJECT (self, "Starting streams");
// Need to unlock to get the clock time
g_mutex_unlock (&self->input->lock);
// Current times of internal and external clock when we go to
// playing. We need this to convert the pipeline running time
// to the running time of the hardware
//
// We can't use the normal base time for the external clock
// because we might go to PLAYING later than the pipeline
self->internal_base_time = gst_clock_get_internal_time (self->input->clock);
self->external_base_time =
gst_clock_get_internal_time (GST_ELEMENT_CLOCK (self));
g_mutex_lock (&self->input->lock);
res = self->input->input->StartStreams (); res = self->input->input->StartStreams ();
if (res != S_OK) { if (res != S_OK) {
GST_ELEMENT_ERROR (self, STREAM, FAILED, GST_ELEMENT_ERROR (self, STREAM, FAILED,
...@@ -652,6 +717,18 @@ gst_decklink_video_src_change_state (GstElement * element, ...@@ -652,6 +717,18 @@ gst_decklink_video_src_change_state (GstElement * element,
self->input->clock, TRUE)); self->input->clock, TRUE));
self->flushing = FALSE; self->flushing = FALSE;
break; break;
case GST_STATE_CHANGE_PAUSED_TO_PLAYING:{
GstClock *clock;
clock = gst_element_get_clock (GST_ELEMENT_CAST (self));
if (clock && clock != self->input->clock) {
gst_clock_set_master (self->input->clock, clock);
}
if (clock)
gst_object_unref (clock);
break;
}
default: default:
break; break;
} }
...@@ -665,6 +742,7 @@ gst_decklink_video_src_change_state (GstElement * element, ...@@ -665,6 +742,7 @@ gst_decklink_video_src_change_state (GstElement * element,
gst_element_post_message (element, gst_element_post_message (element,
gst_message_new_clock_lost (GST_OBJECT_CAST (element), gst_message_new_clock_lost (GST_OBJECT_CAST (element),
self->input->clock)); self->input->clock));
gst_clock_set_master (self->input->clock, NULL);
g_mutex_lock (&self->input->lock); g_mutex_lock (&self->input->lock);
self->input->clock_start_time = GST_CLOCK_TIME_NONE; self->input->clock_start_time = GST_CLOCK_TIME_NONE;
self->input->clock_last_time = 0; self->input->clock_last_time = 0;
...@@ -689,6 +767,8 @@ gst_decklink_video_src_change_state (GstElement * element, ...@@ -689,6 +767,8 @@ gst_decklink_video_src_change_state (GstElement * element,
(NULL), ("Failed to stop streams: 0x%08x", res)); (NULL), ("Failed to stop streams: 0x%08x", res));
ret = GST_STATE_CHANGE_FAILURE; ret = GST_STATE_CHANGE_FAILURE;
} }
self->internal_base_time = GST_CLOCK_TIME_NONE;
self->external_base_time = GST_CLOCK_TIME_NONE;
break; break;
} }
case GST_STATE_CHANGE_PAUSED_TO_PLAYING:{ case GST_STATE_CHANGE_PAUSED_TO_PLAYING:{
......
...@@ -65,6 +65,9 @@ struct _GstDecklinkVideoSrc ...@@ -65,6 +65,9 @@ struct _GstDecklinkVideoSrc
GQueue current_frames; GQueue current_frames;
guint buffer_size; guint buffer_size;
GstClockTime internal_base_time;
GstClockTime external_base_time;
}; };
struct _GstDecklinkVideoSrcClass struct _GstDecklinkVideoSrcClass
...@@ -73,6 +76,8 @@ struct _GstDecklinkVideoSrcClass ...@@ -73,6 +76,8 @@ struct _GstDecklinkVideoSrcClass
}; };
GType gst_decklink_video_src_get_type (void); GType gst_decklink_video_src_get_type (void);
void gst_decklink_video_src_convert_to_external_clock (GstDecklinkVideoSrc * self,
GstClockTime * timestamp, GstClockTime * duration);
G_END_DECLS G_END_DECLS
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment