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 747840 - validate: glvideomixer sometimes fails
validate: glvideomixer sometimes fails
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-devtools
unspecified
Other Linux
: Normal normal
: 1.5.2
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-04-14 10:23 UTC by Guillaume Desmottes
Modified: 2015-06-24 14:44 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
validate: set GST_GL_XINITTHREADS (1.05 KB, patch)
2015-04-14 10:33 UTC, Guillaume Desmottes
none Details | Review
validate: set GST_GL_XINITTHREADS (1.04 KB, patch)
2015-04-14 10:52 UTC, Guillaume Desmottes
none Details | Review

Description Guillaume Desmottes 2015-04-14 10:23:31 UTC
I often get this error when trying to run, for example, validate.file.glvideomixer.simple.fast_forward.synchronized

Pipeline started
[xcb] Unknown request in queue while dequeuing
[xcb] Most likely this is a multi-threaded client and XInitThreads has not been called
[xcb] Aborting, sorry about that.
lt-gst-validate-1.0: xcb_io.c:179: dequeue_pending_request: Assertion `!xcb_xlib_unknown_req_in_deq' failed.


Setting GST_GL_XINITTHREADS seems to fix the issue (see bug#731525). Is that something we should add to the tests env?
Comment 1 Sebastian Dröge (slomo) 2015-04-14 10:25:44 UTC
Yes
Comment 2 Guillaume Desmottes 2015-04-14 10:33:01 UTC
Created attachment 301521 [details] [review]
validate: set GST_GL_XINITTHREADS

This ensure that XInitThreads is called and so gl contexts are properly
initialized.

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

Signed-off-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Comment 3 Thibault Saunier 2015-04-14 10:42:22 UTC
Review of attachment 301521 [details] [review]:

::: validate/launcher/baseclasses.py
@@ +337,3 @@
+        # Ensure XInitThreads is called, see bgo#731525
+        self.proc_env['GST_GL_XINITTHREADS'] = '1'
+        self.add_env_variable('GST_GL_XINITTHREADS')

I think you should rather do it in the GstValidateTest than Test itself.
Comment 4 Guillaume Desmottes 2015-04-14 10:52:05 UTC
Created attachment 301523 [details] [review]
validate: set GST_GL_XINITTHREADS

This ensure that XInitThreads is called and so gl contexts are properly
initialized.

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

Signed-off-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Comment 5 Thibault Saunier 2015-04-14 10:58:04 UTC
I just made sure GST_GL_XINITTHREADS was printed in the command (if no value is specified in add_env_variable,
it gets the value from os.environ, which was not set here).

Attachment 301523 [details] pushed as 8a6494d - validate: set GST_GL_XINITTHREADS