[Bug 778763] New: adaptivedemux: Add pad probe for upstream event/query

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Thu Feb 16 14:41:56 UTC 2017


https://bugzilla.gnome.org/show_bug.cgi?id=778763

            Bug ID: 778763
           Summary: adaptivedemux: Add pad probe for upstream event/query
    Classification: Platform
           Product: GStreamer
           Version: git master
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: Normal
         Component: gst-plugins-bad
          Assignee: gstreamer-bugs at lists.freedesktop.org
          Reporter: sh.yang at lge.com
        QA Contact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---

Add probe to fix deadlock and bug on seeking.

* About deadlock
Assume that seek event was forwarded from a srcpad, and demux is handling it
now.
At the same time, any queries/event which take manifest lock might be forwarded
from the other pads. Since _handle_seek_event() takes manifest lock,
the queries/event can never be handled until _handle_seek_event() is finished.
Also, demux cannot push flush event since the pads are stream locked by
the queries/event.

* Delay handle seek until preroll done
Because subclass handles seeking only for existing stream (not preparing
stream),
wait on preroll.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.


More information about the gstreamer-bugs mailing list