[lttng-dev] [PATCH lttng-ust] Add trace instrumentation for some pthread functions.

Mathieu Desnoyers mathieu.desnoyers at efficios.com
Fri Aug 2 21:28:31 EDT 2013


* Stefan Seefeld (stefan at seefeld.name) wrote:
> On 07/31/2013 06:00 PM, Jérémie Galarneau wrote:
> > Hi Stefan,
> >
> > Please inline your patches in the future as it makes the review process simpler.
> > There are a number of stylistic problems, mostly misplaced braces.
> >
> > You can run your patches through the checkpatch.pl script found in
> > lttng-tools/extras. It will spot most coding-style related problems
> > right away.
> 
> OK, will do, once the patch is approved functionally.

Jeremie is right: incorrect coding style makes the review process harder
for us. Once the coding style is right, we can do the functional review
of the patch more efficiently.

Please resubmit after coding style fixes.

Thanks,

Mathieu

> 
> > Also, I think lttng-ust-pthread-mutex.c may be a more appropriate name
> > than pthread.c
> 
> Wouldn't that just be redundant ? (And I actually expect the file to
> grow instrumentation that's not mutex-specific.)
> 
> Thanks,
>     Stefan
> 
> -- 
> 
>       ...ich hab' noch einen Koffer in Berlin...
> 
> 
> _______________________________________________
> lttng-dev mailing list
> lttng-dev at lists.lttng.org
> http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

-- 
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com



More information about the lttng-dev mailing list