public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca>
To: Alexey Dobriyan <adobriyan@gmail.com>
Cc: Peter Zijlstra <peterz@infradead.org>,
	Masami Hiramatsu <mhiramat@redhat.com>,
	Steven Rostedt <rostedt@goodmis.org>,
	"Frank Ch. Eigler" <fche@redhat.com>, Ingo Molnar <mingo@elte.hu>,
	LKML <linux-kernel@vger.kernel.org>,
	systemtap-ml <systemtap@sources.redhat.com>,
	Hideo AOKI <haoki@redhat.com>
Subject: Re: [RFC] Tracepoint proposal
Date: Sun, 22 Jun 2008 14:27:05 -0400	[thread overview]
Message-ID: <20080622182705.GA23301@Krystal> (raw)
In-Reply-To: <20080622175928.GA5022@martell.zuzino.mipt.ru>

* Alexey Dobriyan (adobriyan@gmail.com) wrote:
> On Sun, Jun 22, 2008 at 01:11:35PM -0400, Mathieu Desnoyers wrote:
> > Tracepoint proposal
> > 
> > - Tracepoint infrastructure
> >   - In-kernel users
> >   - Complete typing, verified by the compiler
> >   - Dynamically linked and activated
> > 
> > - Marker infrastructure
> >   - Exported API to userland
> >   - Basic types only
> > 
> > - Dynamic vs static
> >   - In-kernel probes are dynamically linked, dynamically activated, connected to
> >     tracepoints. Type verification is done at compile-time. Those in-kernel
> >     probes can be a probe extracting the information to put in a marker or a
> >     specific in-kernel tracer such as ftrace.
> >   - Information sinks (LTTng, SystemTAP) are dynamically connected to the
> >     markers inserted in the probes and are dynamically activated.
> > 
> > - Near instrumentation site vs in a separate tracer module
> > 
> > A probe module, only if provided with the kernel tree, could connect to internal
> > tracing sites. This argues for keeping the tracepoing probes near the
> > instrumentation site code. However, if a tracer is general purpose and exports
> > typing information to userspace through some mechanism, it should only export
> > the "basic type" information and could be therefore shipped outside of the
> > kernel tree.
> > 
> > In-kernel probes should be integrated to the kernel tree. They would be close to
> > the instrumented kernel code and would translate between the in-kernel
> > instrumentation and the "basic type" exports. Other in-kernel probes could
> > provide a different output (statistics available through debugfs for instance).
> > ftrace falls into this category.
> > 
> > Generic or specialized information "sinks" (LTTng, systemtap) could be connected
> > to the markers put in tracepoint probes to extract the information to userspace.
> > They would extract both typing information and the per-tracepoint execution
> > information to userspace.
> > 
> > Therefore, the code would look like :
> > 
> > kernel/sched.c:
> > 
> > #include "sched-trace.h"
> > 
> > schedule()
> > {
> >   ...
> >   trace_sched_switch(prev, next);
> >   ...
> > }
> 
> Once this is accepted you're going to add hundreds of such calls to every
> core subsystem, right?
> 

The LTTng instrumentation has about 125 of such calls. Tests have
revealed that adding such dormant tracepoints to the kernel often
increase kernel performances rather than decreasing it (see the ia64
benchmarks posted on lkml a few weeks ago).

The core subsystem maintainers are being involved in the process.
Actually, marking up the source code has the interesting effect of
letting knowledgeable people influence the trace point decisions.

Mathieu

-- 
Mathieu Desnoyers
OpenPGP key fingerprint: 8CD5 52C3 8E3C 4140 715F  BA06 3F25 A8FE 3BAE 9A68

  reply	other threads:[~2008-06-22 18:27 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-06-20 17:03 [RFC][Patch 2/2] markers: example of irq regular kernel markers Masami Hiramatsu
2008-06-20 17:45 ` Mathieu Desnoyers
2008-06-20 19:34   ` Masami Hiramatsu
2008-06-21 10:12     ` KOSAKI Motohiro
2008-06-21 14:36       ` Steven Rostedt
2008-06-21 14:53         ` Frank Ch. Eigler
2008-06-21 15:07           ` Steven Rostedt
2008-06-21 16:13             ` Peter Zijlstra
2008-06-21 18:02               ` Frank Ch. Eigler
2008-06-22  4:31                 ` Masami Hiramatsu
2008-06-23  2:19                   ` KOSAKI Motohiro
2008-06-21 19:39             ` Frank Ch. Eigler
2008-06-22  4:00       ` Masami Hiramatsu
2008-06-20 20:07   ` Peter Zijlstra
2008-06-22 17:11     ` [RFC] Tracepoint proposal Mathieu Desnoyers
2008-06-22 17:59       ` Alexey Dobriyan
2008-06-22 18:27         ` Mathieu Desnoyers [this message]
2008-06-24  0:20           ` Alexey Dobriyan
2008-06-24  4:01             ` Masami Hiramatsu
2008-06-24  7:15               ` Takashi Nishiie
2008-06-24 11:55                 ` Frank Ch. Eigler
2008-06-24 16:04                 ` Masami Hiramatsu
2008-06-24 16:21                   ` KOSAKI Motohiro
2008-06-24 17:01                     ` Masami Hiramatsu
2008-06-24 17:46                       ` Mathieu Desnoyers
2008-06-25 23:52                       ` [RFC PATCH] Kernel Tracepoints Mathieu Desnoyers
2008-06-26 21:02                         ` Masami Hiramatsu
2008-06-27 13:14                           ` Mathieu Desnoyers
2008-06-27 22:45                             ` Masami Hiramatsu
2008-06-30 15:43                               ` Mathieu Desnoyers
2008-06-27 13:15                           ` Mathieu Desnoyers
2008-06-30 19:38                             ` Masami Hiramatsu
2008-06-27 13:30                           ` Mathieu Desnoyers
2008-06-27 20:58                             ` Masami Hiramatsu
2008-06-30 15:40                               ` Mathieu Desnoyers
2008-06-30 19:58                                 ` Masami Hiramatsu
2008-07-03 15:12                                   ` Mathieu Desnoyers
2008-07-03 18:51                                     ` Masami Hiramatsu
2008-06-27 13:36                           ` [RFC PATCH] Kernel Tracepoints (update) Mathieu Desnoyers
2008-07-03 15:27                             ` Masami Hiramatsu
2008-07-03 15:47                               ` Mathieu Desnoyers
2008-07-03 18:18                               ` Mathieu Desnoyers
2008-07-03 18:46                                 ` Masami Hiramatsu
2008-06-25 23:55                       ` [RFC PATCH] Tracepoint sched probes Mathieu Desnoyers
2008-06-24  3:09       ` [RFC] Tracepoint proposal Masami Hiramatsu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20080622182705.GA23301@Krystal \
    --to=mathieu.desnoyers@polymtl.ca \
    --cc=adobriyan@gmail.com \
    --cc=fche@redhat.com \
    --cc=haoki@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mhiramat@redhat.com \
    --cc=mingo@elte.hu \
    --cc=peterz@infradead.org \
    --cc=rostedt@goodmis.org \
    --cc=systemtap@sources.redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox