[Bug 721300] tee: locking issues

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=721300
  GStreamer | gstreamer (core) | git

--- Comment #1 from Andrey Utkin <me at andrey-utkin.pp.ua> 2014-01-01 13:00:04 UTC ---
Created an attachment (id=265096)
 View: https://bugzilla.gnome.org/attachment.cgi?id=265096
 Review: https://bugzilla.gnome.org/review?bug=721300&attachment=265096

partially fixing patch

After looking at above (see "tee: needs fixing locking in gsttee.c?"), i've
made such a patch and now i am running with it.
It allows me to release requestpad safely from main thread.
But if i release it from BLOCK probe callback, the callback is called from
within handle_data() and hangs on secondary GST_TEE_DYN_LOCK, obviously because
it is not recursive. Could we afford recursive mutex?

Before this patch, GST_TEE_DYN_LOCK was used only in release_pad(), which does
not make sense.

-- 
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.