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 701309 - [sdk-0.10.1] Patch for GES to resolve random blocking states
[sdk-0.10.1] Patch for GES to resolve random blocking states
Status: RESOLVED NOTGNOME
Product: GStreamer
Classification: Platform
Component: gst-editing-services
unspecified
Other Linux
: Normal normal
: NONE
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-05-30 20:32 UTC by Eloi BAIL
Modified: 2013-05-30 22:03 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch (19.65 KB, patch)
2013-05-30 20:32 UTC, Eloi BAIL 		none Details | Review

Description Eloi BAIL 2013-05-30 20:32:25 UTC 
Created attachment 245676 [details] [review]
patch

Hi,

I worked on deadlock seen 1/10 with Gstreamer Editing Service.
The pad-added and no-more-pad signal can be emited from any thread
so we have to protect our code from that.

A commit has already been pushed on master 1.0 by Thibault Saunier that helped me to find a good fix : http://cgit.freedesktop.org/gstreamer/gst-editing-services/commit/?id=7e121ff167c900faae46154f076ff5a35d49f938


I port the patch on gstreamer SDK (see in attachement).

Could you please include it in the tree ?

Feel free to ask me if you have any questions.

Thanks,

Eloi
Comment 1 Tim-Philipp Müller 2013-05-30 22:03:50 UTC 
Those 0.10 builds are not maintained by the GStreamer project, but 3rd parties. Their bug tracker is on bugs.freedesktop.org, please re-file the patch there, thanks.