mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2024-11-23 10:11:08 +00:00
Code Issues 1 Projects Releases Wiki Activity

decklink: keep the internal time we send to decklink always increasing

Browse source 
Otherwise decklink seems to hold onto the latest frame and may cause a
flash in the output if the played sequence has a framerate less than the
video output.

Scenario is play for N seconds, pause, flushing seek to some other position,
play again.  Each of the play sequences would normally start at 0 with
the decklink time.  As a result, the latest frame from the previous sequence
is kept alive waiting for it's timestamp to pass before either dropping
(if a subsequent frame in the new sequence overrides it) or displayed
causing the out of place frame to be displayed.

This is also supported by the debug logs from the decklink video sink
element where a ScheduledFrameCompleted() callback would not occur for
the frame until the above had happened.

It was timing related as to whether the frame was displayed based
on the decklink refresh cycle (which seems to be 16ms here),
when the frame was scheduled by the sink and the difference between
the 'time since vblank' of the two play requests (and thus start times
of scheduled playback).
This commit is contained in:
Matthew Waters 2018-07-08 09:54:04 -05:00
parent bd2920ca2d
commit a9b40eebc0
2 changed files with 6 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
sys/decklink/gstdecklinkvideosink.cpp
View file

@ -605,6 +605,7 @@ gst_decklink_video_sink_convert_to_internal_clock (GstDecklinkVideoSink * self,
} else {
GST_LOG_OBJECT (self, "No clock conversion needed, same clocks");
}
*timestamp += self->scheduled_stop_time;
}
static GstFlowReturn
@ -956,6 +957,7 @@ gst_decklink_video_sink_start_scheduled_playback (GstElement * element)
self->output->started = TRUE;
self->output->clock_restart = TRUE;
self->output->clock_offset = self->scheduled_stop_time;
// Need to unlock to get the clock time
g_mutex_unlock (&self->output->lock);
@ -1024,6 +1026,8 @@ gst_decklink_video_sink_stop_scheduled_playback (GstDecklinkVideoSink * self)
self->output->output->IsScheduledPlaybackRunning (&active);
} while (active);
}
if (start_time > 0)
self->scheduled_stop_time = start_time;
self->internal_base_time = GST_CLOCK_TIME_NONE;
self->external_base_time = GST_CLOCK_TIME_NONE;
g_mutex_unlock (&self->output->lock);

2
sys/decklink/gstdecklinkvideosink.h
View file

@ -61,6 +61,8 @@ struct _GstDecklinkVideoSink
GstClockTime internal_base_time;
GstClockTime external_base_time;
GstClockTime scheduled_stop_time;
GstDecklinkOutput *output;
};