After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 632291 - discoverer: sparse tracks cause prerolling to hang till timeout
discoverer: sparse tracks cause prerolling to hang till timeout
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal normal
: 0.10.33
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on: 600648
Blocks:
 
 
Reported: 2010-10-16 10:38 UTC by djericjames
Modified: 2011-03-08 17:31 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
discoverer: Don't wait for subtitle streams to preroll (2.08 KB, patch)
2011-03-08 17:31 UTC, Arun Raghavan
committed Details | Review

Description djericjames 2010-10-16 10:38:51 UTC
When I attempt to import/extract with MediaExport an mkv file with multiple audio tracks, rygel hangs and stops the import.  In specific Italian and English together cause the observed behavior.

This was with Ubuntu version 10.10 with rygel version 0.8.0-1.
Comment 1 Arun Raghavan 2010-10-17 14:30:43 UTC
The hang is due to some bug in (presumably) a gstreamer plugin. metadata extraction goes fine with gstreamer from git, so updating gst* (or waiting for the next release should fix the hang). Nevertheless, discovery takes 10 seconds on my machine, which it should not. Will investigate further.
Comment 2 Jens Georg 2011-01-02 21:06:34 UTC
I think this is still valid; moreover the timeout for the async variant in the GstDiscoverer included in gupnp-dlna still doesn't work with latest release, rendering meta-data extraction unusable
Comment 3 Arun Raghavan 2011-01-04 05:13:20 UTC
Just a quick update on this:

a. The next release of gupnp-dlna will fix the problem with timeouts (we'll use GstDiscoverer from gst-plugins-base which has fixed the problem)

b. I have a fix for the underlying issue, which is that subtitle streams, being sparse, can hold up preroll in GstDiscoverer. I've got a patch for this [1], but it needs quite a bit of testing before it can be used

[1] http://git.collabora.co.uk/?p=user/arun/gst-plugins-base.git;a=shortlog;h=refs/heads/subtitles-fix
Comment 4 Zeeshan Ali 2011-01-10 01:00:06 UTC
(In reply to comment #3)
> Just a quick update on this:
> 
> a. The next release of gupnp-dlna will fix the problem with timeouts (we'll use
> GstDiscoverer from gst-plugins-base which has fixed the problem)

  So with gupnp-dlna 0.5.0 out the door, half the problem is solved? Should we move this to gst-plugins-base?
Comment 5 Arun Raghavan 2011-01-10 04:23:44 UTC
Yes, it makes sense to move this to gst-plugins-base now.
Comment 6 Arun Raghavan 2011-03-08 17:07:25 UTC
We can tackle this once we have demuxers signalling sparse streams (as started on bug 600648).
Comment 7 Arun Raghavan 2011-03-08 17:31:54 UTC
Pushing a fix for this till we have the sparse stream event and support for it in various demuxers.

The following fix has been pushed:
9ffb885 discoverer: Don't wait for subtitle streams to preroll
Comment 8 Arun Raghavan 2011-03-08 17:31:59 UTC
Created attachment 182847 [details] [review]
discoverer: Don't wait for subtitle streams to preroll

Subtitle streams being parse can cause the pipeline to wait indefinitely
to PREROLL. This makes subtitle streams got to PAUSED even if no data is
available. This should not be a cause for concern as we don't expect to
get much data for subtitle streams other than language tags from the
container.