public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/11] LTTng-core (basic tracing infrastructure) 0.5.108
@ 2006-09-14  3:38 Mathieu Desnoyers
  2006-09-14 11:27 ` Ingo Molnar
  0 siblings, 1 reply; 284+ messages in thread
From: Mathieu Desnoyers @ 2006-09-14  3:38 UTC (permalink / raw)
  To: linux-kernel, Christoph Hellwig, Andrew Morton, Ingo Molnar,
	Greg Kroah-Hartman, Thomas Gleixner, Tom Zanussi
  Cc: ltt-dev, Michel Dagenais

Hi,

Following an advice Christoph gave me this summer, submitting a smaller,
easier to review patch should make everybody happier. Here is a stripped
down version of LTTng : I removed everything that would make the code
review reluctant (especially kernel instrumentation and kernel state dump
module). I plan to release this "core" version every few LTTng releases
and post it to LKML.

Comments and reviews are very welcome.

See http://ltt.polymtl.ca > QUICKSTART for information about creating your own
instrumentation set.


Mathieu


OpenPGP public key:              http://krystal.dyndns.org:8080/key/compudj.gpg
Key fingerprint:     8CD5 52C3 8E3C 4140 715F  BA06 3F25 A8FE 3BAE 9A68 

^ permalink raw reply	[flat|nested] 284+ messages in thread
* Re: [PATCH 0/11] LTTng-core (basic tracing infrastructure) 0.5.108
@ 2006-09-15  3:10 James Dickens
  0 siblings, 0 replies; 284+ messages in thread
From: James Dickens @ 2006-09-15  3:10 UTC (permalink / raw)
  To: lkml

Static probe points in the mainline kernel should not be there for
kernel programmers. Any kernel programmer that is interested in an
event that a static probe would trace, could with a little work use
kprobes, Systemtap, printk,  statements or numerous other methods and
accomplish the same thing most likely with less impact on the kernel.

If you allow static probe points, do them for the people that use your
code,  If static probing is to work in the mainline kernel, its
necessary for everyone to see the value of them.

I came up with some simple rules that may help the adoption of static
probe points in the kernel. They answer a lot of issues I read in
other reads.

Some simple rules for Static Probing:

- If the probe is not enabled, it turns into a NOP. No probes are
enabled by default
- Each programmer should provide this as a service to the user.
- There should be at most a 1000 static probe points in the entire
kernel including modules, drivers, etc.
- Probes should not pass out any more information than what a user
would need. If the user needs more he needs to find another way to get
it, perhaps dynamic probing.
- If any part of the kernel has more than a dozen probe points there
are too many.
- If a probe would be of little use to a user/sysadmin it should be
removed from the mainline kernel.
- Yes, if a probe point is in the code you are working on, the role of
maintaining it falls on you.
- If you notice your code is doing something that matches a statically
probed event (.i.e. your network driver dropped a packet), it's your
responsibility to add the necessary probe in your code.
- If "you" need a probe that would not be needed except for debugging
your code, use one of the other methods mentioned above, or remove it
before your code is submitted to the mainline kernel.


Some example static probe points

Task going is being moved on to a cpu.
Task moving off a cpu

Start of an IO
End of an IO

Network packet received
Packet dropped.

Various lock activities
Lock taken
Spin lock taken


James Dickens
uadmin.blogspot.com

^ permalink raw reply	[flat|nested] 284+ messages in thread
* Re: [PATCH 0/11] LTTng-core (basic tracing infrastructure) 0.5.108
@ 2006-09-15  9:17 Richard J Moore
  0 siblings, 0 replies; 284+ messages in thread
From: Richard J Moore @ 2006-09-15  9:17 UTC (permalink / raw)
  To: linux-kernel


Ingo Molnar wrote:

> > I don't think anyone is saying that static tracepoints do not have
> > their limitations, or that dynamic tracepointing is useless. But
> > that's not the point ... why can't we have one infrastructure that
> > supports both? Preferably in a fairly simple, consistent way.
>
> primarily because i fail to see any property of static tracers that are
> not met by dynamic tracers. So to me dynamic tracers like SystemTap are
> a superset of static tracers.

There is one example whethere dynamic tracing is difficult or very messy to
implement and that's for tracepoints needed during system and device
initialization. In this sense dynamic is not a practical superset of
static. However I believe the tooling, for dynamic trace should work for
static as well.

- -
Richard J Moore
IBM Advanced Linux Response Team - Linux Technology Centre
MOBEX: 264807; Mobile (+44) (0)7739-875237
Office: (+44) (0)1962-817072


^ permalink raw reply	[flat|nested] 284+ messages in thread
* Re: [PATCH 0/11] LTTng-core (basic tracing infrastructure) 0.5.108
@ 2006-09-15 17:14 Chuck Ebbert
  2006-09-15 18:32 ` Alan Cox
  2006-09-16 10:46 ` Jes Sorensen
  0 siblings, 2 replies; 284+ messages in thread
From: Chuck Ebbert @ 2006-09-15 17:14 UTC (permalink / raw)
  To: Alan Cox
  Cc: Greg Kroah-Hartman, linux-kernel, Roman Zippel, Jes Sorensen,
	Paul Mundt, Karim Yaghmour, Ingo Molnar, Mathieu Desnoyers,
	Christoph Hellwig, Andrew Morton, Thomas Gleixner, Tom Zanussi,
	ltt-dev, Michel Dagenais

In-Reply-To: <1158331071.29932.63.camel@localhost.localdomain>

On Fri, 15 Sep 2006 15:37:51 +0100, Alan Cox wrote:

> > $ grep KPROBES arch/*/Kconf*
> > arch/i386/Kconfig:config KPROBES
> > arch/ia64/Kconfig:config KPROBES
> > arch/powerpc/Kconfig:config KPROBES
> > arch/sparc64/Kconfig:config KPROBES
> > arch/x86_64/Kconfig:config KPROBES
>
> Send patches. The fact nobody has them implemented on your platform
> isn't a reason to implement something else, quite the reverse in fact.

Yes, but the point is: until that's done you can't claim kprobes is a
valid tracing tool for everyone.

And things like net/ipv4/tcp_probe.c shouldn't be generally implemented
until every arch is supported.

-- 
Chuck

^ permalink raw reply	[flat|nested] 284+ messages in thread
* Re: [PATCH 0/11] LTTng-core (basic tracing infrastructure) 0.5.108
@ 2006-09-25 15:20 Chuck Ebbert
  2006-09-25 15:39 ` Ingo Molnar
  0 siblings, 1 reply; 284+ messages in thread
From: Chuck Ebbert @ 2006-09-25 15:20 UTC (permalink / raw)
  To: Ingo Molnar; +Cc: linux-kernel

In-Reply-To: <20060918151713.GA11495@elte.hu>

On Mon, 18 Sep 2006 17:17:13 +0200, Ingo Molnar wrote:

> yeah - and i dont think the kprobes overhead is a fundamental thing - i 
> posted a few kprobes-speedup patches as a reply to your measurements.

Where is the source code for the kprobes benchmarks you used?

-- 
Chuck


^ permalink raw reply	[flat|nested] 284+ messages in thread

end of thread, other threads:[~2006-09-25 15:47 UTC | newest]

Thread overview: 284+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-09-14  3:38 [PATCH 0/11] LTTng-core (basic tracing infrastructure) 0.5.108 Mathieu Desnoyers
2006-09-14 11:27 ` Ingo Molnar
2006-09-14 13:40   ` Roman Zippel
2006-09-14 13:55     ` Ingo Molnar
2006-09-14 14:33       ` Roman Zippel
2006-09-14 15:26         ` Michel Dagenais
2006-09-14 17:48           ` Ingo Molnar
2006-09-15 15:04             ` Mathieu Desnoyers
2006-09-14 18:08           ` Nick Piggin
2006-09-14 18:38             ` Karim Yaghmour
2006-09-14 17:13         ` Ingo Molnar
2006-09-14 17:55           ` Roman Zippel
2006-09-14 18:15             ` Ingo Molnar
2006-09-14 18:35               ` Mathieu Desnoyers
2006-09-14 18:54               ` Karim Yaghmour
2006-09-15  9:20                 ` Jes Sorensen
2006-09-15 12:38                   ` Karim Yaghmour
2006-09-15 12:32                     ` Jes Sorensen
2006-09-15 14:09                       ` Karim Yaghmour
2006-09-15 14:30                         ` Jes Sorensen
2006-09-15 15:12                           ` Karim Yaghmour
2006-09-16 10:41                             ` Jes Sorensen
2006-09-16 15:28                               ` Karim Yaghmour
2006-09-18  8:57                                 ` Jes Sorensen
2006-09-18 14:48                                   ` Ingo Molnar
2006-09-18 15:37                                     ` Karim Yaghmour
2006-09-15 13:20                     ` Paul Mundt
2006-09-15 13:41                       ` Roman Zippel
2006-09-15 13:44                         ` Jes Sorensen
2006-09-15 14:03                           ` Roman Zippel
2006-09-15 14:37                             ` Alan Cox
2006-09-15 14:34                               ` Roman Zippel
2006-09-15 13:57                         ` Paul Mundt
2006-09-15 14:17                           ` Karim Yaghmour
2006-09-15 14:13                             ` Jes Sorensen
2006-09-15 14:31                               ` Karim Yaghmour
2006-09-15 14:28                                 ` Paul Mundt
2006-09-15 14:46                                   ` Martin J. Bligh
2006-09-15 15:22                                     ` Alan Cox
2006-09-15 15:47                                       ` Martin J. Bligh
2006-09-15 14:51                                   ` Karim Yaghmour
2006-09-15 15:00                                     ` Thomas Gleixner
2006-09-15 15:28                                       ` Karim Yaghmour
2006-09-15 18:16                                       ` Andrew Morton
2006-09-15 18:19                                         ` Ingo Molnar
2006-09-15 19:26                                           ` Karim Yaghmour
2006-09-15 19:43                                           ` Roman Zippel
2006-09-15 20:05                                             ` Ingo Molnar
2006-09-15 20:22                                               ` Mathieu Desnoyers
2006-09-15 21:08                                                 ` Jose R. Santos
2006-09-15 21:25                                                   ` Mathieu Desnoyers
2006-09-15 22:02                                                     ` Jose R. Santos
2006-09-15 22:03                                                   ` Ingo Molnar
2006-09-15 22:32                                                     ` Karim Yaghmour
2006-09-15 22:43                                                       ` Ingo Molnar
2006-09-15 23:33                                                         ` Karim Yaghmour
2006-09-15 23:52                                                           ` Ingo Molnar
2006-09-16  2:24                                                             ` Karim Yaghmour
2006-09-15 23:53                                                           ` Ingo Molnar
2006-09-16  2:51                                                             ` Karim Yaghmour
2006-09-15 22:59                                                     ` Frank Ch. Eigler
2006-09-15 23:40                                                       ` Karim Yaghmour
2006-09-15 23:17                                                     ` Jose R. Santos
2006-09-15 21:32                                                 ` Ingo Molnar
2006-09-15 21:58                                                   ` Mathieu Desnoyers
2006-09-15 22:19                                                     ` Ingo Molnar
2006-09-15 22:45                                                       ` Karim Yaghmour
2006-09-16  9:59                                                 ` Jes Sorensen
2006-09-16 17:24                                                   ` Mathieu Desnoyers
2006-09-16 17:35                                                     ` Ingo Molnar
2006-09-16 17:56                                                       ` Mathieu Desnoyers
2006-09-16 19:10                                                         ` Ingo Molnar
2006-09-16 19:37                                                           ` Ingo Molnar
2006-09-16 20:29                                                             ` [patch] kprobes: optimize branch placement Ingo Molnar
2006-09-16 20:43                                                               ` Ingo Molnar
2006-09-16 21:44                                                                 ` Karim Yaghmour
2006-09-16 22:57                                                                   ` Andrew Morton
2006-09-16 23:24                                                                     ` Ingo Molnar
2006-09-17  2:44                                                                       ` Karim Yaghmour
2006-09-17  1:59                                                                     ` Karim Yaghmour
2006-09-17 12:15                                                                       ` Mike Galbraith
2006-09-16 22:56                                                                 ` Andrew Morton
2006-09-16 23:30                                                                   ` Ingo Molnar
2006-09-18 13:52                                                                     ` Ananth N Mavinakayanahalli
2006-09-16 22:55                                                               ` Andrew Morton
2006-09-17 10:13                                                             ` [PATCH 0/11] LTTng-core (basic tracing infrastructure) 0.5.108 Frederik Deweerdt
2006-09-17 14:00                                                               ` Ingo Molnar
2006-09-16 19:51                                                           ` Karim Yaghmour
2006-09-16 23:40                                                         ` Ingo Molnar
2006-09-17  5:33                                                           ` Mathieu Desnoyers
2006-09-16 18:11                                                       ` Karim Yaghmour
2006-09-16 17:44                                                         ` Ingo Molnar
2006-09-16 18:15                                                           ` Karim Yaghmour
2006-09-18  8:18                                                             ` Jes Sorensen
2006-09-16 17:55                                                     ` Karim Yaghmour
2006-09-18  8:21                                                       ` Jes Sorensen
2006-09-18  8:33                                                     ` Jes Sorensen
2006-09-18 15:01                                                       ` Mathieu Desnoyers
2006-09-16 17:30                                                   ` Mathieu Desnoyers
2006-09-18  8:15                                                     ` Jes Sorensen
2006-09-18 14:53                                                       ` Mathieu Desnoyers
2006-09-18 15:17                                                         ` Ingo Molnar
2006-09-18 16:54                                                           ` Mathieu Desnoyers
2006-09-15 21:12                                               ` Roman Zippel
2006-09-15 21:08                                                 ` Ingo Molnar
2006-09-15 20:13                                           ` Andrew Morton
2006-09-15 21:49                                             ` Jose R. Santos
2006-09-16 10:19                                             ` Jes Sorensen
2006-09-16 16:05                                               ` Karim Yaghmour
2006-09-17  4:54                                                 ` Ganesan Rajagopal
2006-09-18  8:13                                                 ` Jes Sorensen
2006-09-18 14:46                                                   ` Mathieu Desnoyers
2006-09-18 17:06                                                   ` Martin Bligh
2006-09-20 14:17                                                     ` Jes Sorensen
2006-09-15 19:35                                         ` Thomas Gleixner
2006-09-15 19:40                                           ` Ingo Molnar
2006-09-15 19:56                                           ` Karim Yaghmour
2006-09-15 20:23                                             ` Thomas Gleixner
2006-09-15 20:40                                               ` Roman Zippel
2006-09-15 20:48                                                 ` Ingo Molnar
2006-09-15 21:17                                                   ` Karim Yaghmour
2006-09-15 21:15                                                     ` Ingo Molnar
2006-09-15 21:56                                                       ` Karim Yaghmour
2006-09-15 21:27                                                   ` Roman Zippel
2006-09-15 21:51                                                     ` Ingo Molnar
2006-09-15 22:15                                                       ` Karim Yaghmour
2006-09-15 22:53                                                       ` Roman Zippel
2006-09-15 23:14                                                         ` Ingo Molnar
2006-09-15 23:49                                                           ` Nicholas Miell
2006-09-15 23:57                                                             ` Ingo Molnar
2006-09-16  0:41                                                               ` Nicholas Miell
2006-09-16  0:31                                                           ` Roman Zippel
2006-09-16  8:20                                                             ` Ingo Molnar
2006-09-16  8:21                                                             ` Ingo Molnar
2006-09-16  8:21                                                             ` Ingo Molnar
2006-09-16  8:22                                                             ` Ingo Molnar
2006-09-16 19:58                                                               ` Roman Zippel
2006-09-16 22:50                                                                 ` Ingo Molnar
2006-09-16 23:00                                                                 ` Ingo Molnar
2006-09-17  1:15                                                                   ` Roman Zippel
2006-09-17  8:42                                                                     ` Ingo Molnar
2006-09-17 15:16                                                                       ` Roman Zippel
2006-09-17 15:25                                                                         ` Ingo Molnar
2006-09-17 16:02                                                                           ` Roman Zippel
2006-09-17 16:45                                                                             ` Ingo Molnar
2006-09-17 16:59                                                                             ` Nick Piggin
2006-09-17 17:26                                                                               ` Roman Zippel
2006-09-17 17:56                                                                                 ` Nick Piggin
2006-09-17 18:59                                                                                   ` Roman Zippel
2006-09-17 21:23                                                                                     ` Ingo Molnar
2006-09-17 21:52                                                                                       ` Roman Zippel
2006-09-17 22:27                                                                                         ` Ingo Molnar
2006-09-17 21:40                                                                                     ` Ingo Molnar
2006-09-18  8:43                                                                                     ` Jes Sorensen
2006-09-17 21:32                                                                                   ` Ingo Molnar
2006-09-17 19:23                                                                                 ` Ingo Molnar
2006-09-17 19:45                                                                                   ` Roman Zippel
2006-09-17 20:56                                                                                     ` Ingo Molnar
2006-09-17 21:36                                                                                       ` Roman Zippel
2006-09-17 22:13                                                                                         ` Ingo Molnar
2006-09-16 23:14                                                                 ` Ingo Molnar
2006-09-17 14:19                                                                   ` Frank Ch. Eigler
2006-09-17 15:31                                                                     ` Ingo Molnar
2006-09-17 17:15                                                                       ` Mathieu Desnoyers
     [not found]                                                                   ` <y0mu036eglz.fsf@ton.toronto.redhat.com>
2006-09-17 15:00                                                                     ` Ingo Molnar
2006-09-16  8:23                                                             ` Ingo Molnar
2006-09-16  8:23                                                             ` Ingo Molnar
2006-09-16  8:23                                                             ` Ingo Molnar
2006-09-15 21:05                                               ` Karim Yaghmour
2006-09-15 21:17                                                 ` Thomas Gleixner
2006-09-15 21:31                                                   ` Karim Yaghmour
2006-09-15 20:00                                         ` Mathieu Desnoyers
2006-09-15 20:27                                           ` Jose R. Santos
2006-09-15 20:37                                         ` Alan Cox
2006-09-15 20:26                                           ` Mathieu Desnoyers
2006-09-15 20:51                                           ` Karim Yaghmour
2006-09-17 17:53                                           ` Mathieu Desnoyers
2006-09-17 19:35                                           ` printk instrumentation with LTTng Mathieu Desnoyers
2006-09-15 15:24                                     ` [PATCH 0/11] LTTng-core (basic tracing infrastructure) 0.5.108 Alan Cox
2006-09-15 15:23                                       ` Karim Yaghmour
2006-09-15 14:39                                 ` Jes Sorensen
2006-09-15 15:04                                   ` Karim Yaghmour
2006-09-14 19:40               ` Tim Bird
2006-09-14 20:00                 ` Ingo Molnar
2006-09-14 20:46                   ` Karim Yaghmour
2006-09-19 12:05                     ` Christoph Hellwig
2006-09-14 21:02                   ` Roman Zippel
2006-09-15 11:40                 ` Alan Cox
2006-09-15 11:46                   ` Roman Zippel
2006-09-15 12:38                     ` Alan Cox
2006-09-15 12:39                       ` Roman Zippel
2006-09-15 13:41                         ` Alan Cox
2006-09-15 13:34                           ` Roman Zippel
2006-09-15 14:41                             ` Alan Cox
2006-09-15 14:35                               ` Karim Yaghmour
2006-09-15 14:58                                 ` Alan Cox
2006-09-15 14:57                                   ` Karim Yaghmour
2006-09-15 17:49                                     ` Andrew Morton
2006-09-15 18:20                                       ` Karim Yaghmour
2006-09-15 17:01                                   ` Tim Bird
2006-09-15 17:08                                   ` Frank Ch. Eigler
2006-09-15 17:57                                     ` Andrew Morton
2006-09-15 18:31                                     ` Alan Cox
2006-09-15 18:12                                       ` Ingo Molnar
2006-09-15 19:10                                         ` Roman Zippel
2006-09-15 19:10                                           ` Ingo Molnar
2006-09-15 20:05                                           ` Thomas Gleixner
2006-09-15 20:35                                             ` Roman Zippel
2006-09-15 21:44                                             ` Tim Bird
2006-09-19 12:29                                           ` Christoph Hellwig
2006-09-19 13:17                                             ` Roman Zippel
2006-09-15 18:24                                       ` Frank Ch. Eigler
2006-09-15 18:23                                         ` Ingo Molnar
2006-09-15 18:18                                   ` Martin Bligh
2006-09-15 18:10                           ` Jose R. Santos
2006-09-15 19:49                             ` Mathieu Desnoyers
2006-09-15 20:54                               ` Jose R. Santos
2006-09-15 21:42                                 ` Karim Yaghmour
2006-09-15 21:46                                 ` Mathieu Desnoyers
2006-09-19 15:05                                   ` Jose R. Santos
2006-09-19 15:30                                     ` Mathieu Desnoyers
2006-09-19 16:39                                       ` Jose R. Santos
2006-09-19 18:03                                         ` Mathieu Desnoyers
2006-09-15 17:45                       ` Andrew Morton
2006-09-15 18:16                         ` Karim Yaghmour
2006-09-15 19:20                           ` Jose R. Santos
2006-09-15 19:59                           ` Andrew Morton
2006-09-15 20:24                             ` Karim Yaghmour
2006-09-15 20:25                               ` Thomas Gleixner
2006-09-14 19:47               ` Roman Zippel
2006-09-14 20:24                 ` Ingo Molnar
2006-09-14 20:54                   ` Roman Zippel
2006-09-14 21:08                     ` Daniel Walker
2006-09-14 21:30                       ` Roman Zippel
2006-09-14 22:15                         ` Ingo Molnar
2006-09-14 23:39                           ` Roman Zippel
2006-09-14 23:43                             ` Ingo Molnar
2006-09-15  0:27                               ` Roman Zippel
2006-09-15  1:47                   ` Mathieu Desnoyers
2006-09-15  5:47                     ` Vara Prasad
2006-09-14 18:12           ` Karim Yaghmour
2006-09-14 20:25           ` Martin Bligh
2006-09-14 20:34             ` Ingo Molnar
2006-09-14 20:55               ` Martin Bligh
2006-09-14 21:31                 ` Ingo Molnar
2006-09-14 22:25                   ` Martin Bligh
2006-09-14 22:36                     ` Ingo Molnar
2006-09-14 22:59                       ` Martin Bligh
2006-09-14 23:19                         ` Ingo Molnar
2006-09-15  0:19                           ` Nicholas Miell
2006-09-15  1:04                           ` Martin J. Bligh
2006-09-15 12:38                             ` Ingo Molnar
2006-09-15  7:00                         ` Vara Prasad
2006-09-15 15:37                       ` Michel Dagenais
2006-09-19 12:08                 ` Christoph Hellwig
2006-09-14 21:07               ` Roman Zippel
2006-09-15  9:29               ` Jes Sorensen
2006-09-14 17:51         ` Karim Yaghmour
2006-09-14 15:19       ` Mathieu Desnoyers
2006-09-14 19:39         ` Frank Ch. Eigler
2006-09-15 17:13         ` Jose R. Santos
2006-09-14 15:02   ` Mathieu Desnoyers
2006-09-14 15:14   ` Martin J. Bligh
2006-09-14 17:43     ` Ingo Molnar
2006-09-14 18:25       ` Karim Yaghmour
2006-09-14 20:03       ` Martin Bligh
2006-09-14 20:14         ` Ingo Molnar
2006-09-14 20:40           ` Martin Bligh
2006-09-14 21:05           ` Michel Dagenais
2006-09-14 22:23             ` Ingo Molnar
2006-09-14 22:46               ` Martin Bligh
2006-09-14 22:56                 ` Ingo Molnar
2006-09-14 19:03     ` grundig
2006-09-14 19:21       ` Karim Yaghmour
2006-09-14 19:48     ` Frank Ch. Eigler
2006-09-15 16:32       ` Jose R. Santos
2006-09-19 11:59   ` Christoph Hellwig
  -- strict thread matches above, loose matches on Subject: below --
2006-09-15  3:10 James Dickens
2006-09-15  9:17 Richard J Moore
2006-09-15 17:14 Chuck Ebbert
2006-09-15 18:32 ` Alan Cox
2006-09-16 10:46 ` Jes Sorensen
2006-09-25 15:20 Chuck Ebbert
2006-09-25 15:39 ` Ingo Molnar

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox