[Bug 760584] adaptivedemux: tests: hls seek tests sometimes time out

[GStreamer (GNOME Bugzilla)]

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

--- Comment #5 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
I would say it's exactly the same thing. We have two state changes happening at
the same time in different threads.

For the other bug, it seems to be fixed by reverting (as then sending the next
seek event is blocked until the state change to PLAYING is finished).

commit b427997119a2b6aacbeb550f729936f8b963e24b
Author: Sebastian Dröge <sebastian at centricular.com>
Date:   Thu Dec 10 11:35:05 2015 +0200

    element: Don't hold state lock all the time while sending an event

    This lock seems to exist only to prevent elements from changing states
while
    events are being processed. However events are going to be processed
    nonetheless in those elements if sent directly via pads, so protection must
    already be implemented inside the elements for event handling if it is
needed.

    As such having the lock here is not very useful and is actually causing
    various deadlocks in different situations as described in
    https://bugzilla.gnome.org/show_bug.cgi?id=744040


That only seems like a workaround though, specific to seek events. There seems
to be a generic race condition somewhere with finishing asynchronous state
changes and loss of state at the same time.

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