From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2006-04-21 16:32:55
|
Do not reply to this via email (we are currently unable to handle email responses and they get discarded). You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=3D339326 GStreamer | gstreamer (core) | Ver: HEAD CVS Summary: pads not set to FLUSHING when created Product: GStreamer Version: HEAD CVS Platform: Other OS/Version: Linux Status: UNCONFIRMED Severity: major Priority: Normal Component: gstreamer (core) AssignedTo: gst...@li... ReportedBy: wi...@fl... QAContact: gst...@li... GNOME version: Unspecified GNOME milestone: Unspecified newly created pads are not set to the FLUSHING state. This means that a n= ewly created element has pads that can accept data, even in the NULL or READY = state. Setting the pads to FLUSHING initially causes regressions in demuxers tha= t add pads dynamically in PAUSED/PLAYING but forget to activate the pad, so the= y need to be fixed first. --=20 Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=3Demail ------- You are receiving this mail because: ------- You are the QA contact for the bug. You are the assignee for the bug. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2006-07-09 20:00:46
|
Do not reply to this via email (we are currently unable to handle email responses and they get discarded). You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=3D339326 GStreamer | gstreamer (core) | Ver: HEAD CVS Edward Hervey changed: What |Removed |Added -------------------------------------------------------------------------= --- CC| |bi...@bi... Status|UNCONFIRMED |NEW Ever Confirmed|0 |1 ------- Comment #1 from Edward Hervey 2006-07-09 13:43 UTC ------- Example where it causes issues: * An element creates a New pad (not activated, flushing) in READY. * Someone does a gst_pad_set_blocking(newpad, TRUE, callback) * The element's state is set to lock (gst_element_set_locked_state (TRUE)= ) * we switch the element's state to PAUSED, which puts it to PAUSED withou= t calling the state_change function (and the pad activation). * We unblock the state (gst_element_set_locked_state (FALSE)) * We change the state of the element to READY ** The state_change function tries to deactivate the pad, but since it's already not-activated, it won't call the function that unsets the FLUSHIN= G flag ** The element send an EOS ** The pad blocks *** LOCKS in state_change --=20 Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=3Demail ------- You are receiving this mail because: ------- You are the QA contact for the bug. You are the assignee for the bug. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2006-10-06 16:24:57
|
Do not reply to this via email (we are currently unable to handle email responses and they get discarded). You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=3D339326 GStreamer | gstreamer (core) | Ver: HEAD CVS ------- Comment #2 from Wim Taymans 2006-10-06 16:24 UTC ------- Created an attachment (id=3D74154) --> (http://bugzilla.gnome.org/attachment.cgi?id=3D74154&action=3Dview) patch this patch sets the pad to flushing initially and gives a warning when a flushing pad is added to a running element. this should help diagnose offenders. --=20 Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=3Demail |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2006-12-15 16:02:04
|
Do not reply to this via email (we are currently unable to handle email responses and they get discarded). You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=3D339326 GStreamer | gstreamer (core) | Ver: HEAD CVS Wim Taymans changed: What |Removed |Added -------------------------------------------------------------------------= --- Attachment #74154|0 |1 is obsolete| | Status|NEW |RESOLVED Resolution| |FIXED Target Milestone|HEAD |0.10.12 ------- Comment #3 from Wim Taymans 2006-12-15 16:01 UTC ------- * gst/gstelement.c: (gst_element_add_pad): * gst/gstghostpad.c: (gst_ghost_pad_new_full): * gst/gstpad.c: (gst_pad_init): Set pads to FLUSHING when they are created. Check, warn and fix w= hen a demuxer adds an inactive pad to itself when running. Fixes #33932= 6. --=20 Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=3Demail |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-01-05 18:52:20
|
Do not reply to this via email (we are currently unable to handle email responses and they get discarded). You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=3D339326 GStreamer | gstreamer (core) | Ver: HEAD CVS Jan Schmidt changed: What |Removed |Added -------------------------------------------------------------------------= --- CC| |th...@ma...= m ------- Comment #4 from Jan Schmidt 2007-01-05 18:52 UTC ------- You missed a case, that is now impossible to do correctly: * Running element adds a SINK pad to itself (e.g: a Muxer with request = pads, or multiqueue). * By the new logic, an element must do gst_pad_set_active before adding= the pad. * Activating the pad causes the _activate handlers to be called, for st= uff like activating pull mode when it can, but the pad isn't linked to anythi= ng and has no parent yet because it hasn't been added to the element. The same thing applies to SRC pads with activate handlers, but those aren= 't common. I think that instead of warning when a flushing pad is added to a running element, the pad should just be activated, because both add-to-element-then-activate or activate-then-add-to-element are both wro= ng - the activation needs to happen atomically when adding the pad. --=20 Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=3Demail |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-03-30 19:02:48
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=339326 GStreamer | gstreamer (core) | Ver: HEAD CVS Olivier Crete changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |te...@te... Status|RESOLVED |REOPENED Resolution|FIXED | ------- Comment #5 from Olivier Crete 2007-03-30 19:02 UTC ------- Its very annoying that you change the API/ABI sematics like that in the middle of a stable series. If I set the pads to active before adding them, it breaks some of my elements on 0.10.9, so I need to maintain two separate versions now. Please fix . -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=339326. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-04-02 15:31:41
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=339326 GStreamer | gstreamer (core) | Ver: HEAD CVS ------- Comment #6 from Jan Schmidt 2007-04-02 15:31 UTC ------- What exactly does the breakage look like with 0.10.9? So far we haven't found any situations where the plugins can't work properly with GStreamer both before and after the change - it's either been a bug in the plugin, or an obscure case that can be easily done another way. Either way, there's been several releases of GStreamer with the change already made, so changing it back won't help you. -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=339326. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-04-02 16:17:45
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=339326 GStreamer | gstreamer (core) | Ver: HEAD CVS ------- Comment #7 from Olivier Crete 2007-04-02 16:17 UTC ------- I have one example with one of our plugins, rtpbin. Source is at: http://projects.collabora.co.uk/darcs/farsight/gst-plugins-farsight/ext/jrtp/ The patch that makes it work with recent gstreamer but breaks with 0.10.9 is http://projects.collabora.co.uk/~tester/gst-plugins-farsight-set-active.patch The second example is in farsight itself, which uses a regular bin and ghostpads. Source code is at http://projects.collabora.co.uk/darcs/farsight/farsight/plugins/rtp/rtpstream.c The patch is at: http://projects.collabora.co.uk/~tester/farsight-set-active.patch And obviously, without the patches, it doesn't work with the newer gst. -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=339326. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-04-03 22:58:49
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=339326 GStreamer | gstreamer (core) | Ver: HEAD CVS ------- Comment #8 from Olivier Crete 2007-04-03 22:58 UTC ------- I think I found the problem... It has to do with ghostpads and bins. One case is, I have a bin (actually a rtpbin), which is either in the NULL or PAUSED state to which I add an element and then create a sink ghostpad from that element's sinkpad. If I set the ghostpad active before adding it to the bin, it breaks with 0.10.9, if I don't 0.10.12 complains. The second case is similar, I have a bin which may be in the PAUSED state, to which I add a new element (which is in the NULL state), then create a ghostpad, add it to the bin and use it to link it to an element outside the bin. If I set the ghostpad active before adding it to the bin, data doesn't flow in 0.10.9, if I set it active after adding to the bin, 0.10.12 outputs a warning (this is from farsight code). In both cases, I can aleviate the problem by setting the ghostpad active only if the pad it refers to is active, but that still outputs a warning on 0.10.12. -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=339326. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-04-04 12:23:26
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=339326 GStreamer | gstreamer (core) | Ver: HEAD CVS ------- Comment #9 from Tim-Philipp Müller 2007-04-04 12:23 UTC ------- > In both cases, I can aleviate the problem by setting the ghostpad active only > if the pad it refers to is active, but that still outputs a warning on 0.10.12. As a work-around, you could also check the core version at runtime with gst_version() and skip the activation for earlier cores. -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=339326. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-05-01 20:14:11
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=339326 GStreamer | gstreamer (core) | Ver: HEAD CVS Philippe Khalaf changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |phi...@co.... | |uk ------- Comment #10 from Philippe Khalaf 2007-05-01 20:14 UTC ------- I would also like to add that the helper functions in gstutils.c do not activate ghost pads before adding them to the bin, therefore, it causes warnings when using functions such as gst_element_link_filtered(). -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=339326. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-06-19 14:50:34
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=339326 GStreamer | gstreamer (core) | Ver: HEAD CVS Wim Taymans changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|pads not set to FLUSHING |gstutils.c pad link helper |when created |functions don't activate | |pads properly -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=339326. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2009-04-15 23:11:48
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=339326 GStreamer | gstreamer (core) | Ver: git Tim-Philipp Müller changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |t....@ze... ------- Comment #11 from Tim-Philipp Müller 2009-04-15 23:11 UTC ------- However annoying the semantic change may have been, we've had g_warnings for this in core for ages now, so I think we should just go and fix this in whatever way is best. Is any of this still relevant? -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=339326. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2009-04-15 23:12:15
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=339326 GStreamer | gstreamer (core) | Ver: git Tim-Philipp Müller changed: What |Removed |Added ---------------------------------------------------------------------------- Status|REOPENED |NEEDINFO -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=339326. |
From: GStreamer (bugzilla.gnome.o. <bug...@gn...> - 2009-09-10 07:21:05
|
https://bugzilla.gnome.org/show_bug.cgi?id=339326 Sebastian Dröge <slomo> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |sl...@ci... --- Comment #12 from Sebastian Dröge <sl...@ci...> 2009-09-10 07:20:47 UTC --- Wim or someone else, what shall be done about this bug now? -- Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. You are the assignee for the bug. |
From: GStreamer (bugzilla.gnome.o. <bug...@gn...> - 2010-01-16 16:24:48
|
https://bugzilla.gnome.org/show_bug.cgi?id=339326 GStreamer | gstreamer (core) | git --- Comment #13 from Sebastian Dröge <sl...@ci...> 2010-01-16 16:24:35 UTC --- Ping? -- Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. You are the assignee for the bug. |
From: GStreamer (bugzilla.gnome.o. <bug...@gn...> - 2010-03-24 17:48:36
|
https://bugzilla.gnome.org/show_bug.cgi?id=339326 GStreamer | gstreamer (core) | git Tobias Mueller <gnome-bugs> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |gnome-bugs@auftrags-killer. | |org --- Comment #14 from Tobias Mueller <gno...@au...> 2010-03-24 17:48:23 UTC --- Maybe Olivier can tell whether this is still an issue. If not, I'm all for closing as OBSOLETE. -- Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. You are the assignee for the bug. |
From: GStreamer (bugzilla.gnome.o. <bug...@gn...> - 2010-05-21 15:26:20
|
https://bugzilla.gnome.org/show_bug.cgi?id=339326 GStreamer | gstreamer (core) | git Olivier Crete (Tester) <tester> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |RESOLVED Resolution| |OBSOLETE --- Comment #15 from Olivier Crete (Tester) <te...@te...> 2010-05-21 15:26:05 UTC --- I guess this works for everyone now.. So lets close it -- Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. You are the assignee for the bug. |