[gstreamer-bugs] [Bug 599870] [API] micro-optimisation: add GST_DEBUG_FUNCPTR_FULL

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Fri Nov 6 07:27:18 PST 2009


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

--- Comment #11 from Sebastian Dröge <slomo at circular-chaos.org> 2009-11-06 15:27:13 UTC ---
(In reply to comment #8)
> (In reply to comment #6)
> > > 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.
> > 
> > Depends on whether the issue was primarily contention (doubtful?) or the hash
> > table lookups I guess.

> Maybe we could adjust the macro to be a G_STMT with the volatile inside.

That's probably the best solution... any reason not to do that? :)

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