Mirror of the lttng-dev mailing list
 help / color / mirror / Atom feed
From: compudj@krystal.dyndns.org (Mathieu Desnoyers)
Subject: [ltt-dev] LTTng specialized probes
Date: Mon, 6 Oct 2008 10:11:13 -0400	[thread overview]
Message-ID: <20081006141113.GE1808@Krystal> (raw)
In-Reply-To: <532480950809240032t644448f7lc4fdc0dffca69b9@mail.gmail.com>

Hi,

I'm currently working towards getting LTTng in shape for what is
required for mainline. I got the "TLB-less" buffers and splice()
working last week. I then did some performance testing on the flight
recorder mode and noticed an optimization that's really worth doing :

LTTng "ltt-serialize.c", which parses the format strings and formats
data into the trace buffers takes a lot of CPU time. I tried only
keeping the size calculation (first pass on the format string) and
disabling the real data write and basically got something like :

(default LTTng instrumentation, very approximate numbers)

tbench no tracing : ~1900MB/s
       Markers enabled : ~1800MB/s
       with size calculation : ~1400MB/s
       size calc + data write : ~950MB/s

I then remembered I've done ltt-serialize in such a way that it can be
easily overridden by per-format string specialized callbacks.

Therefore, it would be worthwhile to create such specialized serializers
so the common cases can be made much faster. I think it will have a very
significant impact on performance.

It's simply a matter of creating a new .c kernel module in ltt/ and to
create structures similar to :

ltt-serialize.c :

struct ltt_available_probe default_probe = {
        .name = "default",
        .format = NULL,
        .probe_func = ltt_vtrace,
        .callbacks[0] = ltt_serialize_data,
};

Give it a non-null format string (just giving the types expected by the
callback), a good name, and a callback function, which implements the
specialized serialization. Note that kernel/marker.c currently expects
the format string to match exactly the marker format string, including
the type names, which should be changed. The type verification should
only check that the %X parameters are the same (and that there are the
same amount of arguments expected).

That should not be hard, but it's not what I plan to focus on next.
Anyone is willing to work on this ?

Mathieu

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




       reply	other threads:[~2008-10-06 14:11 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <5df78e1d0809231814i4b9b98eeyfb9746e5dbb9eb72@mail.gmail.com>
     [not found] ` <20080924072503.GA15570@bolzano.suse.de>
     [not found]   ` <532480950809240032t644448f7lc4fdc0dffca69b9@mail.gmail.com>
2008-10-06 14:11     ` Mathieu Desnoyers [this message]
2008-10-06 15:14       ` Martin Bligh
2008-10-06 15:26         ` Mathieu Desnoyers
2008-10-06 15:37           ` Martin Bligh
2008-10-06 15:56             ` Mathieu Desnoyers
2008-10-06 17:55       ` Jiaying Zhang
2008-10-06 17:55         ` Jiaying Zhang
2008-10-06 17:55         ` Jiaying Zhang
     [not found]         ` <5df78e1d0810071116i2e9790cdx8d5854dbfa50cfaf@mail.gmail.com>
2008-10-07 21:28           ` Michael Davidson
2008-10-07 21:28             ` Michael Davidson
2008-10-07 21:28             ` Michael Davidson
2008-10-08  0:07             ` Mathieu Desnoyers
2008-10-08 15:56               ` Jan Kiszka
2008-10-08 15:56                 ` Jan Kiszka
2008-10-08 15:56                 ` Jan Kiszka
2008-10-08 16:06                 ` Martin Bligh
2008-10-09  2:43                   ` Mathieu Desnoyers
2008-10-09  2:24                 ` Mathieu Desnoyers
2008-10-09  2:24                   ` Mathieu Desnoyers
2008-10-09  2:24                   ` Mathieu Desnoyers
2008-10-09  3:04               ` Michael Davidson
2008-10-09  3:04                 ` Michael Davidson
2008-10-09  3:04                 ` Michael Davidson
2008-10-09 15:28                 ` Mathieu Desnoyers
2008-10-09 15:39                   ` Martin Bligh
2008-10-09 16:15                     ` Mathieu Desnoyers
2008-10-09 16:28                       ` Martin Bligh
2008-10-09 16:55                         ` Mathieu Desnoyers
2008-10-08  0:26         ` Mathieu Desnoyers

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=20081006141113.GE1808@Krystal \
    --to=compudj@krystal.dyndns.org \
    /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