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 797020 - decodebin3-parse: take input locking when updating group-id
decodebin3-parse: take input locking when updating group-id
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
1.14.0
Other Linux
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2018-08-24 08:32 UTC by HoonHee Lee
Modified: 2018-11-03 12:09 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
decodebin3-parse: take input locking when updating group-id (1.11 KB, patch)
2018-08-24 08:35 UTC, HoonHee Lee
none Details | Review

Description HoonHee Lee 2018-08-24 08:32:20 UTC
Dear All
 
I've encountered issue to play dual video stream (BL/EL).
The issue is that group-id is changed in decodebin3-parse and it is not equal in between video and audio in streamsyncronizer.
So, streamsynchronizer could not send segment and buffer to downstream(basesink).
 
The root cause it that group-id is updated without mutex locking.
 
===========================================
0:00:32.817321099 10434 0xefd0ba00 DEBUG             decodebin3 gstdecodebin3-parse.c:237:parse_chain_output_probe:<parsebin0:src_0> Got event stream-start
0:00:32.817865349 10434 0xefd0ba00 FIXME             decodebin3 gstdecodebin3-parse.c:246:parse_chain_output_probe:<parsebin0:src_0> Consider implementing group-id handling on stream-start event
0:00:32.817900932 10434 0xefd0ba00 DEBUG             decodebin3 gstdecodebin3-parse.c:251:parse_chain_output_probe:<parsebin0:src_0> Got stream-start, group_id:4, input 0x194d18
0:00:32.818515182 10434 0xf0601a30 DEBUG             decodebin3 gstdecodebin3-parse.c:237:parse_chain_output_probe:<parsebin0:src_1> Got event stream-start
0:00:32.818548641 10434 0xf0601a30 DEBUG             decodebin3 gstdecodebin3-parse.c:251:parse_chain_output_probe:<parsebin0:src_1> Got stream-start, group_id:3, input 0x194d18
0:00:32.818587557 10434 0xf0601a30 WARN              decodebin3 gstdecodebin3.c:908:set_input_group_id:<decodebin3-0> Group id changed (4 -> 3) on input 0x194d18 
0:00:32.818611682 10434 0xf0601a30 DEBUG             decodebin3 gstdecodebin3.c:915:set_input_group_id:<decodebin3-0> Setting current group id to 3
0:00:32.818869057 10434 0xf0601a30 DEBUG             decodebin3 gstdecodebin3-parse.c:237:parse_chain_output_probe:<parsebin0:src_1> Got event caps
0:00:32.818924432 10434 0xf0601a30 DEBUG             decodebin3 gstdecodebin3-parse.c:281:parse_chain_output_probe:<parsebin0:src_1> caps audio/x-eac3, framed=(boolean)true, rate=(int)48000, channels=(int)2, alignment=(string)iec61937
0:00:32.818289849 10434 0xefd0ba00 DEBUG             decodebin3 gstdecodebin3.c:915:set_input_group_id:<decodebin3-0> Setting current group id to 4
===========================================================
Comment 1 HoonHee Lee 2018-08-24 08:35:27 UTC
Created attachment 373451 [details] [review]
decodebin3-parse: take input locking when updating group-id

Dear All
 
Please review my patch.
 
Thanks.
Comment 2 GStreamer system administrator 2018-11-03 12:09:36 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-base/issues/479.