qtdemux: track streams that are EOS on push mode to finish earlier

When the segment has a defined stop position, qtdemux should check
when streams reach this position and mark those as EOS. When all
streams are EOS it will return GST_FLOW_EOS to upstream to allow
the pipeline to finish instead of continuously consume buffers
from upstream that are not useful for the segment.

https://bugzilla.gnome.org/show_bug.cgi?id=707530
This commit is contained in:
Thiago Santos 2013-09-05 00:17:16 -03:00
parent 33cf8b679d
commit be0eeae491

View file

@ -89,6 +89,8 @@
#define QTDEMUX_SECONDS_FROM_1904_TO_1970 (((1970 - 1904) * (guint64) 365 + \ #define QTDEMUX_SECONDS_FROM_1904_TO_1970 (((1970 - 1904) * (guint64) 365 + \
QTDEMUX_LEAP_YEARS_FROM_1904_TO_1970) * QTDEMUX_SECONDS_PER_DAY) QTDEMUX_LEAP_YEARS_FROM_1904_TO_1970) * QTDEMUX_SECONDS_PER_DAY)
#define STREAM_IS_EOS(s) (s->time_position == -1)
GST_DEBUG_CATEGORY (qtdemux_debug); GST_DEBUG_CATEGORY (qtdemux_debug);
/*typedef struct _QtNode QtNode; */ /*typedef struct _QtNode QtNode; */
@ -1858,6 +1860,7 @@ gst_qtdemux_reset (GstQTDemux * qtdemux, gboolean hard)
qtdemux->streams[n]->last_ret = GST_FLOW_OK; qtdemux->streams[n]->last_ret = GST_FLOW_OK;
qtdemux->streams[n]->sent_eos = FALSE; qtdemux->streams[n]->sent_eos = FALSE;
qtdemux->streams[n]->segment_seqnum = 0; qtdemux->streams[n]->segment_seqnum = 0;
qtdemux->streams[n]->time_position = 0;
} }
} }
} }
@ -3720,7 +3723,7 @@ gst_qtdemux_sync_streams (GstQTDemux * demux)
if (demux->pullbased) { if (demux->pullbased) {
/* loop mode is sample time based */ /* loop mode is sample time based */
if (stream->time_position != -1) if (!STREAM_IS_EOS (stream))
continue; continue;
} else { } else {
/* push mode is byte position based */ /* push mode is byte position based */
@ -4155,7 +4158,8 @@ gst_qtdemux_loop_state_movie (GstQTDemux * qtdemux)
&& qtdemux->segment.stop <= min_time && qtdemux->segment.stop <= min_time
&& qtdemux->streams[index]->on_keyframe)) { && qtdemux->streams[index]->on_keyframe)) {
GST_DEBUG_OBJECT (qtdemux, "we reached the end of our segment."); GST_DEBUG_OBJECT (qtdemux, "we reached the end of our segment.");
goto eos; qtdemux->streams[index]->time_position = -1;
goto eos_stream;
} }
/* gap events for subtitle streams */ /* gap events for subtitle streams */
@ -4879,7 +4883,7 @@ gst_qtdemux_chain (GstPad * sinkpad, GstObject * parent, GstBuffer * inbuf)
} }
} }
/* Figure out which stream this is packet belongs to */ /* Figure out which stream this packet belongs to */
for (i = 0; i < demux->n_streams; i++) { for (i = 0; i < demux->n_streams; i++) {
stream = demux->streams[i]; stream = demux->streams[i];
if (stream->sample_index >= stream->n_samples) if (stream->sample_index >= stream->n_samples)
@ -4902,24 +4906,50 @@ gst_qtdemux_chain (GstPad * sinkpad, GstObject * parent, GstBuffer * inbuf)
} }
/* Put data in a buffer, set timestamps, caps, ... */ /* Put data in a buffer, set timestamps, caps, ... */
sample = &stream->samples[stream->sample_index];
if (G_LIKELY (!(STREAM_IS_EOS (stream)))) {
outbuf = gst_adapter_take_buffer (demux->adapter, demux->neededbytes); outbuf = gst_adapter_take_buffer (demux->adapter, demux->neededbytes);
GST_DEBUG_OBJECT (demux, "stream : %" GST_FOURCC_FORMAT, GST_DEBUG_OBJECT (demux, "stream : %" GST_FOURCC_FORMAT,
GST_FOURCC_ARGS (stream->fourcc)); GST_FOURCC_ARGS (stream->fourcc));
g_return_val_if_fail (outbuf != NULL, GST_FLOW_ERROR); g_return_val_if_fail (outbuf != NULL, GST_FLOW_ERROR);
sample = &stream->samples[stream->sample_index];
dts = QTSAMPLE_DTS (stream, sample); dts = QTSAMPLE_DTS (stream, sample);
pts = QTSAMPLE_PTS (stream, sample); pts = QTSAMPLE_PTS (stream, sample);
duration = QTSAMPLE_DUR_DTS (stream, sample, dts); duration = QTSAMPLE_DUR_DTS (stream, sample, dts);
keyframe = QTSAMPLE_KEYFRAME (stream, sample); keyframe = QTSAMPLE_KEYFRAME (stream, sample);
/* check for segment end */
if (G_UNLIKELY (demux->segment.stop != -1
&& demux->segment.stop <= pts && stream->on_keyframe)) {
GST_DEBUG_OBJECT (demux, "we reached the end of our segment.");
stream->time_position = -1; /* this means EOS */
/* check if all streams are eos */
ret = GST_FLOW_EOS;
for (i = 0; i < demux->n_streams; i++) {
if (!STREAM_IS_EOS (demux->streams[i])) {
ret = GST_FLOW_OK;
break;
}
}
if (ret == GST_FLOW_EOS) {
GST_DEBUG_OBJECT (demux, "All streams are EOS, signal upstream");
goto eos;
}
} else {
ret = gst_qtdemux_decorate_and_push_buffer (demux, stream, outbuf, ret = gst_qtdemux_decorate_and_push_buffer (demux, stream, outbuf,
dts, pts, duration, keyframe, dts, demux->offset); dts, pts, duration, keyframe, dts, demux->offset);
}
/* combine flows */ /* combine flows */
ret = gst_qtdemux_combine_flows (demux, stream, ret); ret = gst_qtdemux_combine_flows (demux, stream, ret);
} else {
/* skip this data, stream is EOS */
gst_adapter_flush (demux->adapter, demux->neededbytes);
}
stream->sample_index++; stream->sample_index++;
stream->offset_in_sample = 0; stream->offset_in_sample = 0;