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 755612 - gtk: Add property to create a window and only dispatch to main thread if that is set
gtk: Add property to create a window and only dispatch to main thread if that...
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
1.5.90
Other Linux
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-09-25 08:13 UTC by Sebastian Dröge (slomo)
Modified: 2018-11-03 15:04 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Sebastian Dröge (slomo) 2015-09-25 08:13:24 UTC 
See summary. Dispatching to the main thread and blocking, especially during state changes, can easily cause deadlocks. Currently we only do that because of the window that is created by the sinks if the user did not embed the widget anywhere. No normal application needs this but all applications now have to live in fear of deadlocks.
Comment 1 Matthew Waters (ystreet00) 2015-09-25 08:24:27 UTC 
The other case for needing the dispatch to main thread is for getting the GL context/display from Gtk so it's not just the 'create own window' case.
Comment 2 Sebastian Dröge (slomo) 2015-09-25 08:29:03 UTC 
That's done in get_widget(), right? Any sane application will get the widget from the sink in its main thread, and before changing the state of anything :)
Comment 3 Matthew Waters (ystreet00) 2015-09-25 08:32:54 UTC 
Currently no.  You don't need the GL context to get the widget ;)

Function trace is as follows
change_state (NULL_to_READY)
gtk_gst_gl_sink_start()
gtk_gst_gl_widget_init_winsys()
<dspatch to main context>
_get_gl_context()
Comment 4 GStreamer system administrator 2018-11-03 15:04:43 UTC 
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/gstreamer/gst-plugins-good/issues/226.