From: Andi Kleen <andi@firstfloor.org>
To: Daniel Gutson <daniel.gutson@tallertechnologies.com>
Cc: Leonardo Boquillon <leonardo.boquillon@tallertechnologies.com>,
linux-perf-users@vger.kernel.org
Subject: Re: Ability for classifying of measurements\
Date: Wed, 04 May 2016 14:41:01 -0700 [thread overview]
Message-ID: <87mvo5pklu.fsf@tassilo.jf.intel.com> (raw)
In-Reply-To: <CAF5HaEXP1vvDod5eRnY2DYvTzeZDC_w15SbNpPEdosbv52qbAQ@mail.gmail.com> (Daniel Gutson's message of "Wed, 4 May 2016 14:20:37 -0300")
Daniel Gutson <daniel.gutson@tallertechnologies.com> writes:
> I kept thinking about all this, and I changed my mind towards a
> simpler (a first step) yet useful approach.
> In the same way people spread some printfs for old school debugging, I
> keep the idea of some perf API made available
> for the application (which calls depend on some flag, say -DNPERF
> similar to -DNDEBUG, so if the macro is
> not defined, calls have no effects).
> The first relatively easy (again, but yet useful) feature that comes
> to my mind, is the ability (from the app) to
> label phases of the perf recording, including enabling, disabling, and
> labelling.
ftrace has a labelling feature like this (/sys/kernel/debug/tracing_marker)
But you can just do it on your own by defining some uprobes with
perf probe for unique points and tracing them.
The main challenge would be good user tooling support.
For enabling/disabling:
it works for processes that do self monitoring today.
For non self monitoring there were perf patches at some point, but they
were rejected because it wasn't clear if a process should be able
to opt out of monitoring on its own.
I suppose it may be ok with an explicit opt-in on the perf side.
BTW I suspect the main motivation why you need this is because
perf doesn't have a time line to select what time region to browse.
The information is actually all in the perf.data, the UI is just
not good. I sometimes now use vtune (which reads perf files)
and has a time line to work around this.
At some point it would be nice to add a time line to perf. Unfortunately
I don't think it would work well with the terminal UI, so would
need to be done in the gui mode.
-Andi
--
ak@linux.intel.com -- Speaking for myself only
next prev parent reply other threads:[~2016-05-04 21:41 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-19 20:51 Ability for classifying of measurements Leonardo Boquillon
2016-04-20 13:08 ` Andi Kleen
[not found] ` <CAF5HaEXOwn0_b=an2RR2FT++ktRXLpAP2WHPE7O=kdVKVRYKVQ@mail.gmail.com>
2016-04-20 16:08 ` Daniel Gutson
2016-04-20 17:47 ` Ability for classifying of measurements\ Andi Kleen
2016-05-04 17:20 ` Daniel Gutson
2016-05-04 17:26 ` Daniel Gutson
2016-05-04 21:41 ` Andi Kleen [this message]
2016-05-04 21:56 ` Daniel Gutson
2016-05-04 23:19 ` Vince Weaver
2016-05-04 23:50 ` Andi Kleen
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=87mvo5pklu.fsf@tassilo.jf.intel.com \
--to=andi@firstfloor.org \
--cc=daniel.gutson@tallertechnologies.com \
--cc=leonardo.boquillon@tallertechnologies.com \
--cc=linux-perf-users@vger.kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.