[PATCH] tevent patches

Amitay Isaacs amitay at gmail.com
Fri Feb 28 01:27:31 MST 2014


On Thu, Feb 27, 2014 at 8:18 PM, Volker Lendecke
<Volker.Lendecke at sernet.de>wrote:

> On Thu, Feb 27, 2014 at 09:03:30AM +0100, Stefan (metze) Metzmacher wrote:
> > Hi Amitay,
> >
> > > Here are some improvements to tevent tracing functionality.  This was
> > > especially useful for debugging CTDB and finding out what events are
> being
> > > triggered often.
> > >
> > > The patches are also available in my tree:
> >
> > We can't make trace point an compile time optional feature,
> > smbd relies on this feature.
> >
> > Can you add a TEVENT_HAS_HANDLER_TRACE_POINTS define similar to
> > TEVENT_HAS_LOOP_ONCE_TRACE_POINTS?
>
> Just brainstorming....
>
> Would it be possible to make these tracepoints if not used
> 0-cost when a handler is executed? If a caller wants to have
> trace points, can we wrap the handlers in another function
> at tevent_add_fd time & friends?
>

Here's second attempt at tracing patches. This one implements 0-cost
handler when trace points are not used.
Also added a define that can be used for configuration checks.

Amitay.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: tevent.patches
Type: application/octet-stream
Size: 15882 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20140228/b51e7350/attachment.obj>


More information about the samba-technical mailing list