[gstreamer-bugs] [Bug 599870] [API] micro-optimisation: add GST_DEBUG_FUNCPTR_FULL
GStreamer (bugzilla.gnome.org)
bugzilla at gnome.org
Wed Oct 28 00:45:07 PDT 2009
https://bugzilla.gnome.org/show_bug.cgi?id=599870
GStreamer | gstreamer (core) | git
Edward Hervey <bilboed> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |bilboed at gmail.com
--- Comment #3 from Edward Hervey <bilboed at gmail.com> 2009-10-28 07:45:05 UTC ---
(In reply to comment #1)
> Makes sense... letting instantiation block on a global mutex is probably not
> the best idea :)
>
> Changing that mutex to a RW lock and letting _gst_debug_register_funcptr()
> first take a R lock to check if it already is inserted and only take a W lock
> if it isn't might be a cleaner fix though.
Use a rwlock will be even worse, since the glib rwlock isn't a native
implementation, but instead takes two regular locks... so best case scenario
you'd be now taking two lock, and worst case scenario four :)
--
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.
More information about the Gstreamer-bugs
mailing list