Thiago Santos [Thu, 14 Feb 2013 20:39:35 +0000 (17:39 -0300)]
dashdemux: properly reset stream status after a flush
The stream eos/end of period status must also be reset after a flushing
seek. This makes seeking after EOS work.
Andre Moreira Magalhaes (andrunko) [Wed, 13 Feb 2013 04:13:23 +0000 (02:13 -0200)]
dashdemux: keep a list of streams periods
Keep a list of streams per period so that the download loop can keep
downloading while the stream loop is still pushing old period's data.
Andre Moreira Magalhaes (andrunko) [Wed, 13 Feb 2013 00:54:32 +0000 (22:54 -0200)]
dashdemux: Properly parse base uri from mpd.
Andre Moreira Magalhaes (andrunko) [Wed, 13 Feb 2013 00:53:16 +0000 (22:53 -0200)]
dashdemux: Remove unused members.
Andre Moreira Magalhaes (andrunko) [Tue, 12 Feb 2013 20:50:51 +0000 (18:50 -0200)]
dashdemux: Only send newsegment on seek.
Thiago Santos [Fri, 8 Feb 2013 05:55:05 +0000 (02:55 -0300)]
dashdemux: send segments that overwrite previous ones
This is only done for seeks, so it is fine
Andre Moreira Magalhaes (andrunko) [Thu, 7 Feb 2013 23:58:37 +0000 (21:58 -0200)]
dashdemux: switch bitrates without switching pads
Andre Moreira Magalhaes (andrunko) [Thu, 7 Feb 2013 18:22:13 +0000 (16:22 -0200)]
dashdemux: Remove unusued method.
Andre Moreira Magalhaes (andrunko) [Thu, 7 Feb 2013 16:09:44 +0000 (14:09 -0200)]
dashdemux: Set stream need_header after seek and has_data_queued to FALSE when flushing stream.
Thiago Santos [Thu, 7 Feb 2013 06:09:28 +0000 (03:09 -0300)]
dashdemux: do not pause/start the stream loop on playing/paused changes
The demuxer shouldn't care about paused/playing, it should act the
same in both states
Thiago Santos [Wed, 6 Feb 2013 16:31:18 +0000 (13:31 -0300)]
dashdemux: remove property that isn't used anymore
Remove some pieces of code that are from an old property
that isn't used anymore
Thiago Santos [Tue, 5 Feb 2013 20:38:06 +0000 (17:38 -0300)]
dashdemux: add a downloadrate utility
A small struct that keeps a short history of fragment download bitrates
to have an average measure of N last fragments instead of using only
the last downloaded bitrate
Thiago Santos [Tue, 5 Feb 2013 17:24:45 +0000 (14:24 -0300)]
dashdemux: sprinkling some more log
Thiago Santos [Tue, 5 Feb 2013 17:22:51 +0000 (14:22 -0300)]
dashdemux: change bitrates based on the stream bitrate
Do not use a global bitrate as the sizes of the fragments matter
when calculating the download rate as the connection setup time is
also being taken into the download duration, a smaller fragment
will have a lower bitrate than a larger one.
This avoids switching the bitrates for streams frequently because
of bitrate mismatches
Thiago Santos [Tue, 5 Feb 2013 03:45:06 +0000 (00:45 -0300)]
dashdemux: refactor to use the uridownloader lib instead of internal copy
Andre Moreira Magalhaes (andrunko) [Tue, 5 Feb 2013 03:29:28 +0000 (01:29 -0200)]
dashdemux: Make sure switch_pads won't leak any pad.
Andre Moreira Magalhaes (andrunko) [Tue, 5 Feb 2013 00:58:32 +0000 (22:58 -0200)]
dashdemux: Properly stop download and stream tasks where appropriate.
Thiago Santos [Fri, 1 Feb 2013 05:12:25 +0000 (02:12 -0300)]
dashdemux: avoid an assertion when stopping the pipeline
Avoids asserting when dataqueues are setting to flushing due
to a pipeline stop during a pad switch.
Thiago Santos [Fri, 1 Feb 2013 05:10:15 +0000 (02:10 -0300)]
dashdemux: download the next fragment with smaller timestamp
Instead of downloading 1 fragment per stream per download loop,
select the stream with the earlier timestamp and get a fragment
only for that one.
The old algorithm would lead to problems when the fragment durations
were too different for streams.
Thiago Santos [Fri, 1 Feb 2013 03:37:33 +0000 (00:37 -0300)]
dashdemux: track segments in the demuxer
Use a GstSegment to track the current segment information in the
demuxer.
Thiago Santos [Fri, 1 Feb 2013 03:33:00 +0000 (00:33 -0300)]
dashdemux: remove buffering message emission from dashdemux
dashdemux shouldn't emit the buffering message as that can pause
the pipeline. It has no proper knowledge of the downstream buffering
status so it can pause the pipeline when it isn't necessary. It should
have an internal buffer for downloading the streams ahead of playback,
but that shouldn't make it able to stop the pipeline for buffering.
A particular case in which this is bad is when a pad switch happens
(changing bitrates for example), the new pads dashdemux creates
will get linked to demuxers and new queues will be created,
these queues are initially empty and dashdemux will quickly
drain its buffers by pushing them to those queues. So it
would have no more buffers internally and would emit a
buffering message with a low ratio, causing the pipeline
to pause when it wouldn't be necessary.
Thiago Santos [Wed, 30 Jan 2013 20:31:04 +0000 (17:31 -0300)]
dashdemux: removing unused code
Cleaning up by removing unused code and variables
Thiago Santos [Wed, 30 Jan 2013 20:27:47 +0000 (17:27 -0300)]
dashdemux: re-enable and fix adaptive switching
It is still unstable, but at least it works when the switches aren't
happening after every fragment
Thiago Santos [Tue, 29 Jan 2013 18:58:50 +0000 (15:58 -0300)]
dashdemux: correctly signal EOS on manifest end
Put EOS on the streams queues after the last fragment from the
last period for each stream. This way we keep it serialized
with the buffers and it will work when streams have different
ending times
Thiago Santos [Tue, 29 Jan 2013 16:08:27 +0000 (13:08 -0300)]
dashdemux: detect end-of-period correctly
Detect end of period when there are no more buffers on any of the
streams
Thiago Santos [Tue, 29 Jan 2013 16:04:01 +0000 (13:04 -0300)]
dashdemux: Use the smallest queue value to define buffering state
The smallest queue should be used to prevent blocking the download
thread when a stream has too much data buffered, leaving the other
streams starving from fragments
Thiago Santos [Tue, 29 Jan 2013 14:22:52 +0000 (11:22 -0300)]
dash: fix parsing of streams to get correct fragment durations
Use the current stream base time instead of the 0th stream base time
to get default duration for fragments.
Thiago Santos [Tue, 29 Jan 2013 04:06:52 +0000 (01:06 -0300)]
dashdemux: block waiting for buffers instead of busy wait
Make the streaming loop block waiting for buffers on the queues instead
of busy waiting eating cpu continuously on a queue length check
Thiago Santos [Mon, 28 Jan 2013 22:43:46 +0000 (19:43 -0300)]
dashdemux: fix seeking on multiple streams
Each stream has its own durations and timestamps, the fragment number
is different for each stream when seeking, so the seek has to be done
for all streams, rather than on a single stream and propagated to
others
Thiago Santos [Mon, 28 Jan 2013 21:52:04 +0000 (18:52 -0300)]
dashdemux: Replace GQueue by GstDataQueue
GstDataQueue has proper locking and provides functions to limit the
size of the queue. Also has blocking calls that are useful to
our multithread scenario in Dash.
Thiago Santos [Mon, 28 Jan 2013 15:28:29 +0000 (12:28 -0300)]
dashdemux: move the buffers queues to the streams
Store the buffers separately for each stream, this is clearer than
having a queue with a list of buffers. It also allows easier selection
of buffers to push in later refactors
Thiago Santos [Mon, 28 Jan 2013 13:46:27 +0000 (10:46 -0300)]
dashdemux: refactor streams data to its own struct
Keeps code more organized and similar to what other demuxers
usually do
Thiago Santos [Fri, 25 Jan 2013 16:04:15 +0000 (13:04 -0300)]
dash: do not try to print a pointer as a string
Thiago Santos [Fri, 25 Jan 2013 15:45:18 +0000 (12:45 -0300)]
dashdemux: Remove timing code that delays pushing of fragments
Fragments should be pushed ASAP as downstream should be responsible for
doing the syncrhonization and proper buffering.
This has the great side effect of fixing most of the seeking A/V sync issues.
Thiago Santos [Fri, 25 Jan 2013 12:36:35 +0000 (09:36 -0300)]
dash: fixing build warnings
Also running through gst-indent
Louis-Francis Ratté-Boulianne [Wed, 23 Jan 2013 21:43:12 +0000 (16:43 -0500)]
gstdashdemux: add need_header member to know whether the initialization fragment should be pushed
Louis-Francis Ratté-Boulianne [Wed, 23 Jan 2013 18:20:25 +0000 (13:20 -0500)]
dashdemux: don't wait for the whole duration of a fragment when seeking
Louis-Francis Ratté-Boulianne [Wed, 23 Jan 2013 18:19:21 +0000 (13:19 -0500)]
dashdemux: switch pads and send header file when a newsegment event is sent
Louis-Francis Ratté-Boulianne [Wed, 16 Jan 2013 18:58:52 +0000 (13:58 -0500)]
dashdemux: various fixes
David Corvoysier [Thu, 20 Dec 2012 08:04:28 +0000 (09:04 +0100)]
Applied gst-indent
David Corvoysier [Tue, 18 Dec 2012 10:21:22 +0000 (11:21 +0100)]
Replaced URL pointing to obsolete Klagenfurt MPD by a new one
Gianluca Gennari [Mon, 17 Dec 2012 14:46:32 +0000 (15:46 +0100)]
mpdparser: fix escape sequence in URL template resolving
Gianluca Gennari [Mon, 17 Dec 2012 14:39:10 +0000 (15:39 +0100)]
mpdparser: do not switch pads when bitstreamSwitching = true
Gianluca Gennari [Mon, 17 Dec 2012 14:19:33 +0000 (15:19 +0100)]
dashdemux: complete support for manifest file updates
Gianluca Gennari [Mon, 17 Dec 2012 14:12:58 +0000 (15:12 +0100)]
dashdemux: add support for manifest file updates
- the MPD file is updated in the download loop (only if we have a "dynamic" MPD and minimumUpdatePeriod is valid);
- properly LOCK/UNLOCK the GstMpdClient;
Gianluca Gennari [Mon, 17 Dec 2012 14:04:45 +0000 (15:04 +0100)]
dashdemux: new API to set/get segment index and period index
TODO: rework segment selection to support Representations or Adaptation Sets with segments not aligned
Gianluca Gennari [Mon, 17 Dec 2012 14:00:52 +0000 (15:00 +0100)]
dashdemux: new API to load the stream Period with a given index
this avoids to fiddle with stream internals in the code
Gianluca Gennari [Mon, 17 Dec 2012 13:54:42 +0000 (14:54 +0100)]
dashdemux: implement seeking across Periods
David Corvoysier [Wed, 21 Nov 2012 13:14:34 +0000 (14:14 +0100)]
Applied gst-indent
David Corvoysier [Wed, 21 Nov 2012 13:14:21 +0000 (14:14 +0100)]
By default, do not forward pad queries upstream, because it may have unforessen side-effects (forwarding the new 1.0 GST_CAPS query would prevent pad linking for instance)
Olivier Blin [Mon, 12 Nov 2012 09:54:35 +0000 (10:54 +0100)]
plugin: register as dashdemux instead of fragmented
This fixes conflicts with the HLS plugin, which is also named
fragmented.
When building its registry, gstreamer was picking one or the other
between hls and dashdemux.
Gianluca Gennari [Thu, 25 Oct 2012 15:55:02 +0000 (17:55 +0200)]
dashdemux: check if we have received the MPD file in the proper way
Gianluca Gennari [Thu, 25 Oct 2012 15:53:53 +0000 (17:53 +0200)]
mpdparser: silence warning
Olivier Blin [Thu, 25 Oct 2012 15:15:09 +0000 (17:15 +0200)]
dashdemux: fix printing buffering time and download rate/size
This was broken on armv7.
Olivier Blin [Thu, 25 Oct 2012 15:17:35 +0000 (17:17 +0200)]
dashdemux: fix unused-but-set variable
Olivier Blin [Thu, 25 Oct 2012 14:49:21 +0000 (16:49 +0200)]
dashdemux: use new gst_mpd_client_get_next_fragment_duration API
This fixes build that has been broken by commit
fb9aeac6552021b176a4c4bd07265e02a0b70e0f.
gst_mpd_client_get_target_duration has been removed, and
gst_mpd_client_get_next_fragment_duration should be used instead.
Gianluca Gennari [Wed, 24 Oct 2012 15:28:40 +0000 (17:28 +0200)]
dashdemux: remove FIXME comment
pad switching across multiple periods works perfectly fine now
and it is done only when there is a change in the CAPS
Gianluca Gennari [Wed, 24 Oct 2012 14:55:20 +0000 (16:55 +0200)]
dashdemux: send new-segment events to all pads
Gianluca Gennari [Wed, 24 Oct 2012 14:30:01 +0000 (16:30 +0200)]
dashdemux: reworked the API to retrieve fragment timestamp, fragment duration and media presentation duration
This was necessary to support variable-duration Fragments.
in the new API:
- gst_mpd_client_get_current_position returns the timestamp of the NEXT fragment to download;
- gst_mpd_client_get_next_fragment_duration returns the duration of the next fragment to download;
- gst_mpd_client_get_media_presentation_duration returns the mediaPresentationDuration from the MPD file;
also there is a new internal parser function:
- gst_mpd_client_get_segment_duration extracts the constant segment duration from the MPD file
(only used when there is no SegmentTimeline syntax element in the current representation)
In gst_mpd_client_get_next_fragment, we set the timestamp/duration of the fragment just downloaded
copying the values from the corresponding GstMediaSegment.
TODO: rework SEEKING to support seeking across different Periods.
Gianluca Gennari [Wed, 24 Oct 2012 12:50:54 +0000 (14:50 +0200)]
dashdemux: use effective segment duration to reschedule the streaming task
do not assume segment duration is constant
Conflicts:
ext/dash/gstdashdemux.c
Gianluca Gennari [Wed, 24 Oct 2012 12:24:05 +0000 (14:24 +0200)]
dashdemux: reworked gst_dash_demux_get_buffering_time() to return the effective buffering time
do not assume the Fragment duration is constant
Gianluca Gennari [Wed, 24 Oct 2012 09:49:51 +0000 (11:49 +0200)]
dashdemux: complete support for Media Presentations with several Periods
- Periods are played in sequence, from PeriodStart to PeriodEnd
- seamless switching from one Period to the next one works fine;
- the 'new-segment' generation is broken, so if we need to switch pads for a new Period there is a crash;
Gianluca Gennari [Mon, 22 Oct 2012 16:12:30 +0000 (18:12 +0200)]
mpdparser: initial support for Media Presentations made of several Periods
- build a list of the available Periods with their start and duration time
- add the list of GstStreamPeriod in the GstMpdClient data struct
- remove cur_period from GstMpdClient and introduce an API to get the current GstStreamPeriod
- several API clean-ups
Olivier Blin [Wed, 24 Oct 2012 16:10:47 +0000 (18:10 +0200)]
mpdparser: support BaseURL for SegmentURL using mediaRange
This is used in big_buck_bunny_CappedVBR_Multirate_4s_0_AudioVideo_MainProfile.mpd
Olivier Blin [Wed, 24 Oct 2012 16:04:07 +0000 (18:04 +0200)]
mpdparser: factorize adding range parameter to URL
Gianluca Gennari [Fri, 19 Oct 2012 18:12:09 +0000 (20:12 +0200)]
dashdemux: reworked caps detection
also reworked the API to extract audio/video parameters from the manifest file
(mimeType, width, height, rate, num channels)
Gianluca Gennari [Fri, 19 Oct 2012 17:03:41 +0000 (19:03 +0200)]
dashdemux: do not send bogus duration messages when mediaPresentationDuration is unknown
Gianluca Gennari [Fri, 19 Oct 2012 16:45:30 +0000 (18:45 +0200)]
mpdparser: support SegmentTimeline syntax also for list-based media segment generation
Gianluca Gennari [Fri, 19 Oct 2012 15:47:51 +0000 (17:47 +0200)]
mpdparser: add support for SegmentTimeline syntax elements
build the list of segments to be played using the SegmentTimeline syntax, if present
bugfixes:
- for dynamic MPD files, when mediaPresentationDuration is not present use minimumUpdatePeriod instead
- do not add a spurious '$' when building an URL from a template like "$Bandwidth$/init.mp4v"
- introduce gst_mpd_client_add_media_segment() to avoid code duplication
David Corvoysier [Fri, 19 Oct 2012 13:42:51 +0000 (15:42 +0200)]
Fixed wrong index in fragments
Gianluca Gennari [Fri, 19 Oct 2012 10:01:03 +0000 (12:01 +0200)]
mpdparser: fix memory leak in gst_mpd_client_get_next_fragment_set()
we have to free next_fragment_uri
David Corvoysier [Fri, 19 Oct 2012 08:50:16 +0000 (10:50 +0200)]
Removed unused function
David Corvoysier [Fri, 19 Oct 2012 08:43:49 +0000 (10:43 +0200)]
Seek event handling cleanup
Gianluca Gennari [Thu, 18 Oct 2012 16:49:53 +0000 (18:49 +0200)]
dashdemux: fix playing of single-segment media streams
other fixes:
- fixed a buffering bug: now we stop buffering when we reach the end of manifest
- now gst_mpd_client_get_target_duration() always returns a valid duration
(in case of single-segment streams, we return either Period duration or mediaPresentation duration)
TODO: support SegmentTimeline
Gianluca Gennari [Wed, 17 Oct 2012 16:46:22 +0000 (18:46 +0200)]
uridownloader: deleted useless declaration of gst_uri_downloader_free()
Gianluca Gennari [Wed, 17 Oct 2012 16:45:25 +0000 (18:45 +0200)]
uridownloader: when possible, reuse existing element to download a new URI
previously a new element was allocated for each media segment, and it was never freed
Gianluca Gennari [Wed, 17 Oct 2012 16:43:26 +0000 (18:43 +0200)]
dashdemux: unref caps in function need_add_header()
Gianluca Gennari [Wed, 17 Oct 2012 16:42:02 +0000 (18:42 +0200)]
dashdemux: fix extraction of SegmentTemplate->initialization in gst_mpd_client_get_next_header()
now the following MPD files are properly supported:
http://dash.edgesuite.net/adobe/hdworld_dash/HDWorld-1600.mpd
http://dash.edgesuite.net/adobe/hdworld_dash/HDWorld.mpd
Gianluca Gennari [Wed, 17 Oct 2012 16:40:02 +0000 (18:40 +0200)]
dashdemux: fix memory leak in gst_dash_demux_get_next_header()
David Corvoysier [Wed, 17 Oct 2012 13:52:19 +0000 (15:52 +0200)]
We now only accept actual manifest files, and not any XML file
David Corvoysier [Wed, 17 Oct 2012 13:50:33 +0000 (15:50 +0200)]
Added a typefind helper for DASH: this commit should be reverted as soon as a proper patch has been submitted to gst-plugins-base
David Corvoysier [Wed, 17 Oct 2012 08:02:39 +0000 (10:02 +0200)]
Fixed state machine (again)
David Corvoysier [Tue, 16 Oct 2012 14:46:17 +0000 (16:46 +0200)]
Code refactoring
David Corvoysier [Tue, 16 Oct 2012 14:34:10 +0000 (16:34 +0200)]
Fixed state machine
David Corvoysier [Tue, 16 Oct 2012 11:43:39 +0000 (13:43 +0200)]
Improve consistency of the tasks shutdown code
David Corvoysier [Tue, 16 Oct 2012 10:51:44 +0000 (12:51 +0200)]
Added missing g_list_free
David Corvoysier [Tue, 16 Oct 2012 09:44:26 +0000 (11:44 +0200)]
Do not answer to URI queries
David Corvoysier [Tue, 16 Oct 2012 09:39:18 +0000 (11:39 +0200)]
Try queries upstream if we cannot process them
David Corvoysier [Tue, 16 Oct 2012 09:26:03 +0000 (11:26 +0200)]
Renamed demux->next_stream in demux->next_push
David Corvoysier [Tue, 16 Oct 2012 09:23:26 +0000 (11:23 +0200)]
Inlined one-line utility funciton that was called only once
David Corvoysier [Tue, 16 Oct 2012 07:10:00 +0000 (09:10 +0200)]
Fixed regression after the introduction of the multi-screen support, preventing the pipeline from stopping properly
David Corvoysier [Mon, 15 Oct 2012 14:41:28 +0000 (16:41 +0200)]
Modified log level of shutdown message
David Corvoysier [Mon, 15 Oct 2012 13:08:53 +0000 (15:08 +0200)]
Comments: We don't select a single representation, but a representation for each stream Code: cleanup of gst_get_next_fragment (also renamed in gst_get_next_fragment_set)
David Corvoysier [Mon, 15 Oct 2012 12:25:27 +0000 (14:25 +0200)]
Removed debug trace
David Corvoysier [Mon, 15 Oct 2012 12:14:53 +0000 (14:14 +0200)]
Added introduction and general design notes
David Corvoysier [Mon, 15 Oct 2012 11:54:17 +0000 (13:54 +0200)]
Fixed wrong comment
David Corvoysier [Mon, 15 Oct 2012 11:52:31 +0000 (13:52 +0200)]
Documented gst_dash_demux_download_loop
David Corvoysier [Mon, 15 Oct 2012 11:35:15 +0000 (13:35 +0200)]
Commented gst_dash_demux_stream_loop
David Corvoysier [Mon, 15 Oct 2012 11:17:12 +0000 (13:17 +0200)]
Commented needs_pad_switch
David Corvoysier [Mon, 15 Oct 2012 11:09:49 +0000 (13:09 +0200)]
Documented the representation switch mechanism