From: nils.carlson@ericsson.com (Nils Carlson)
Subject: [ltt-dev] [UST] Instrumentation API changes done
Date: Wed, 27 Apr 2011 16:24:01 +0200 [thread overview]
Message-ID: <alpine.DEB.2.00.1104271621300.15457@as68123.uab.ericsson.se> (raw)
In-Reply-To: <20110422183308.GA28636@Krystal>
Hi,
On Fri, 22 Apr 2011, Mathieu Desnoyers wrote:
> Hi,
>
> I would appreciate if you could have a look at the API change
> implementation I pushed into UST. For now, it should cover the most
> critical parts. In its current state, I think the ust_marker() API is
> pretty much final, so people can start using it for debug-only in their
> application and expect it not to change. For the Tracepoints, the
> DECLARE/DEFINE_TRACEPOINT will eventually become internal to
> TRACEPOINT_EVENT(). We can expect further changes to the
> TRACEPOINT_EVENT() API, as we will very likely need to change the other
> macros that come with it.
>
The changes look good to me. No real comments. I'll have a look at the
documentation when I can.
> While we are there, I am tempted to kill TRACEPOINT_EVENT_FN(), because
> it was only needed for specific syscall tracing, and I don't really
> expect to let the instrumentation to be controlled so closely by the
> application code in user-space. Thoughts ?
Nope. No thoughts, just emptiness here... ;-)
> Also, I'd really like to push a UST 0.13 out the door with the new
> instrumentation ASAP. At least people will be able to start using
> ust_mark() without fear that the carpet will be removed from under their
> feet. ;-)
I think it will have to wait a few days at least, but then we should be
ready.
/Nils
> Thanks,
>
> Mathieu
>
> --
> Mathieu Desnoyers
> Operating System Efficiency R&D Consultant
> EfficiOS Inc.
> http://www.efficios.com
>
> _______________________________________________
> ltt-dev mailing list
> ltt-dev at lists.casi.polymtl.ca
> http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
>
prev parent reply other threads:[~2011-04-27 14:24 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-22 18:33 Mathieu Desnoyers
2011-04-27 14:24 ` Nils Carlson [this message]
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=alpine.DEB.2.00.1104271621300.15457@as68123.uab.ericsson.se \
--to=nils.carlson@ericsson.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