Re: [PATCH 0/11] LTTng-core (basic tracing infrastructure) 0.5.108

Andrew Morton
2006-09-15T20:14:27Z

On Fri, 15 Sep 2006 20:19:07 +0200
Ingo Molnar <mingo@elte.hu> wrote:

>
> * Andrew Morton <akpm@osdl.org> wrote:
>
> > What Karim is sharing with us here (yet again) is the real in-field
> > experience of real users (ie: not kernel developers).
>
> well, Jes has that experience and Thomas too.

systemtap and ltt are the only full-scale tracing tools which target
sysadmins and applciation developers of which I am aware..

> > I mean, on one hand we have people explaining what they think a
> > tracing facility should and shouldn't do, and on the other hand we
> > have a guy who has been maintaining and shipping exactly that thing to
> > (paying!) customers for many years.
>
> so does Thomas and Jes. So what's the point?

My point is that I respect Karim and Frank's experience. I in fact
disagree with them (or at least, I want to). But they've been there, and I
haven't. So I listen.

> i judge LTT by its current code quality, not by its proponents shouting
> volume - and that quality is still quite poor at the moment. (and then
> there are the conceptual problems too, outlined numerous times) I have
> quoted specific example(s) for that in this thread. Furthermore, LTT
> does this:
>
> 246 files changed, 26207 insertions(+), 71 deletions(-)
>
> and this gives me the shivers, for all the reasons i outlined.
>

In the bit of text which you snipped I was agreeing with this...

Look, if Karim and Frank (who I assume is a systemtap developer) think that
we need static tracepoints then I have no reason to disagree with them.
What I would propose is that:

a) Those tracepoints be integrated one at a time on well-understood
grounds of necessity. Tracepoints _should_ be added dynamically. But
if there are instances where that's not working and cannot be made to
work then OK, in we go.

b) Saying "we need the static tracepoints because the line numbers keep
on changing" is not, repeat not a justification for static tracepoints.
It's a SMOP to develop tracepoint-adding code which can handle line
numbers changing. lwall did it.

c) Any static tracepoints should be seen as corner-case augmentation of
existing dynamic tracing framework(s). IOW: I see no justification at
this time for adding complete new second set of backend
accumulation/reporting/management infrastructure (ie: LTT core).

Shorter version: I agree with Frank.