[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20080419180306.GA29968@redhat.com>
Date: Sat, 19 Apr 2008 14:03:06 -0400
From: "Frank Ch. Eigler" <fche@...hat.com>
To: Peter Zijlstra <a.p.zijlstra@...llo.nl>
Cc: prasad@...ux.vnet.ibm.com, linux-kernel@...r.kernel.org,
tglx@...utronix.de, mingo@...e.hu, mathieu.desnoyers@...ymtl.ca
Subject: Re: [RFC PATCH 1/2] Marker probes in futex.c
Hi -
On Sat, Apr 19, 2008 at 04:52:26PM +0200, Peter Zijlstra wrote:
> > It's not just that - it's a whole package including easy creation of
> > new markers, the code that manages their activation and deactivation,
> > the tool code that connects up to receive new events *and parameters*.
> > [...]
> I'm thinking the two use-cases are confused here. So we have
>
> a) permanent markers
> b) ad-hoc debug markers
OTOH I think this is a false dichotomy. Debugging is not only done by
a subsystem maintainer during the merge/rc period. When something
goes wrong on a deployed machine, problem diagnosis requires data,
which ideally should be gathered as non-intrusively as possible - that
means no recompiling / rebooting, and ideally very little slowdown.
I bet that many of those markers you might consider "ad-hoc" would in
fact have some post-release diagnostic value. Now, maybe in the case
of futexes, the kernel makes no sophisticated decisions that may need
subsequent review. Maybe all the internals are directly calculable
from the results visible at the system call level (which threads block
and which return). But this is certainly not so for many other parts
of the kernel.
With markers, you don't have to make this an agonizing decision. You
just insert markers, regardless of whether it's high-level, universal,
"permanent"; or whether it's low-level, diagnostic, temporary. The
same consumer tools will work for them all.
- FChE
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists