Bug 179356 - [GStreamer] Failure in gst_gl_context_thread_add
Summary: [GStreamer] Failure in gst_gl_context_thread_add
Status: RESOLVED DUPLICATE of bug 224972
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: Other
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-11-06 18:07 PST by Michael Catanzaro
Modified: 2021-04-23 06:10 PDT (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2017-11-06 18:07:21 PST
There's another thread inside _mesa_glsl_compile_shader at the same time. Maybe that's bad?

Thread 1 (Thread 0x2b8428401700 (LWP 5903)):
#0  0x00002b83a02d292f in exec_node_remove () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/Mesa/src/glsl/list.h:170
#1  _ZN9exec_node6removeEv () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/Mesa/src/glsl/list.h:248
#2  _Z31optimize_dead_builtin_variablesP9exec_list16ir_variable_mode () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/Mesa/src/glsl/opt_dead_builtin_variables.cpp:79
#3  0x00002b83a0293cfb in _mesa_glsl_compile_shader () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/Mesa/src/glsl/glsl_parser_extras.cpp:1647
#4  0x00002b83a015fbba in compile_shader () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/Mesa/src/mesa/main/shaderapi.c:980
#5  0x00002b831fceb3d2 in _compile_shader () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/gst-plugins-bad-1.10.5/gst-libs/gst/gl/gstglslstage.c:486
#6  0x00002b831fcda990 in gst_gl_context_thread_add () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/gst-plugins-bad-1.10.5/gst-libs/gst/gl/gstglcontext.c:1537
#7  0x00002b831fcebe07 in gst_glsl_stage_compile () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/gst-plugins-bad-1.10.5/gst-libs/gst/gl/gstglslstage.c:542
#8  0x00002b831fce81eb in gst_gl_shader_compile_attach_stage () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/gst-plugins-bad-1.10.5/gst-libs/gst/gl/gstglshader.c:603
#9  0x00002b831fcee03f in _create_shader () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/gst-plugins-bad-1.10.5/gst-libs/gst/gl/gstglcolorconvert.c:1830
#10 _init_convert () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/gst-plugins-bad-1.10.5/gst-libs/gst/gl/gstglcolorconvert.c:2028
#11 _do_convert () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/gst-plugins-bad-1.10.5/gst-libs/gst/gl/gstglcolorconvert.c:2368
#12 0x00002b831fcf2683 in _run_message_sync () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/gst-plugins-bad-1.10.5/gst-libs/gst/gl/gstglwindow.c:601
#13 0x00002b831fcf2622 in _run_message_async () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/gst-plugins-bad-1.10.5/gst-libs/gst/gl/gstglwindow.c:668
#14 0x00002b8320cd15ca in g_main_dispatch () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/glib-2.52.1/glib/gmain.c:3212
#15 g_main_context_dispatch () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/glib-2.52.1/glib/gmain.c:3865
#16 0x00002b8320cd1948 in g_main_context_iterate () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/glib-2.52.1/glib/gmain.c:3938
#17 0x00002b8320cd1c62 in g_main_loop_run () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/glib-2.52.1/glib/gmain.c:4134
#18 0x00002b831fcf26f5 in gst_gl_window_default_run () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/gst-plugins-bad-1.10.5/gst-libs/gst/gl/gstglwindow.c:527
#19 0x00002b831fcdb95c in gst_gl_context_create_thread () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/gst-plugins-bad-1.10.5/gst-libs/gst/gl/gstglcontext.c:1273
#20 0x00002b8320cf8315 in g_thread_proxy () at /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/DependenciesGTK/Source/glib-2.52.1/glib/gthread.c:784
#21 0x00002b8323f2b494 in start_thread (arg=0x2b8428401700) at pthread_create.c:333
#22 0x00002b832510793f in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:97

Full backtrace here: https://build.webkit.org/results/GTK%20Linux%2064-bit%20Release%20(Tests)/r224517%20(4057)/imported/w3c/web-platform-tests/mediacapture-fromelement/capture-crash-log.txt

It's crashing imported/w3c/web-platform-tests/mediacapture-fromelement/capture.html so I'll update the expectations for that accordingly.
Comment 1 Michael Catanzaro 2017-11-06 18:28:40 PST
Note: it's flaky.
Comment 2 Philippe Normand 2018-02-07 10:12:57 PST
I can't make it fail here. Perhaps it's no longer flaky after the update to gst 1.12?
Comment 4 Michael Catanzaro 2018-02-07 11:50:10 PST
I believe lots of tests were failing with this crash, BTW, so many that I wondered if we should undo the switch to GStreamerGL. I wonder if we have other bug reports for this.
Comment 5 Philippe Normand 2018-02-08 01:05:12 PST
Was this ever reported to Mesa and/or GStreamer bugzilla?
Comment 6 Michael Catanzaro 2018-02-08 08:12:51 PST
(In reply to Philippe Normand from comment #5)
> Was this ever reported to Mesa and/or GStreamer bugzilla?

Nope.
Comment 7 Xabier Rodríguez Calvar 2021-03-24 09:22:23 PDT
I don't see a crash, just a failure.
Comment 8 Philippe Normand 2021-03-24 09:36:16 PDT
Yeah well this specific trace is 3 years old, we did several Mesa and GStreamer updates since :)

Maybe update the TestExpectations then?
Comment 9 Xabier Rodríguez Calvar 2021-03-25 03:26:02 PDT
(In reply to Philippe Normand from comment #8)
> Yeah well this specific trace is 3 years old, we did several Mesa and
> GStreamer updates since :)
> 
> Maybe update the TestExpectations then?

It does not appear as crash, just as failure, so I'll update the bug summary
Comment 10 Philippe Normand 2021-04-23 06:10:39 PDT
Test is already marked as failing in the generic TestExpectations so no need to have another one in our ports.

Also since we don't support video.captureStream() anyway, I don't see how this test could pass, hence duplicate of bug 224972.

*** This bug has been marked as a duplicate of bug 224972 ***