From: Frederic Weisbecker <fweisbec@gmail.com>
To: Hitoshi Mitake <mitake@dcl.info.waseda.ac.jp>
Cc: linux-kernel@vger.kernel.org, h.mitake@gmail.com,
Ingo Molnar <mingo@elte.hu>,
Peter Zijlstra <a.p.zijlstra@chello.nl>,
Paul Mackerras <paulus@samba.org>,
Arnaldo Carvalho de Melo <acme@redhat.com>,
Jens Axboe <jens.axboe@oracle.com>,
Jason Baron <jbaron@redhat.com>,
Xiao Guangrong <xiaoguangrong@cn.fujitsu.com>
Subject: Re: [PATCH] perf lock: track only specified threads
Date: Fri, 7 May 2010 02:49:47 +0200 [thread overview]
Message-ID: <20100507004945.GA8069@nowhere> (raw)
In-Reply-To: <1273138376-23323-1-git-send-email-mitake@dcl.info.waseda.ac.jp>
On Thu, May 06, 2010 at 06:32:56PM +0900, Hitoshi Mitake wrote:
> I implemented the feature of tracking only specified threads to perf lock.
> With -t option, users can specify which threads should be tracked.
>
> Example of usage:
> | % sudo ./perf lock info -t # info -t is convenient with this feature
> | Thread ID: comm
> | 0: swapper
> | 1: init
> | 12: migration/3
> | 13: ksoftirqd/3
> | 27: events/0
> | 28: events/1
> | 29: events/2
> | 30: events/3
> | 31: events/4
> | 857: kondemand/0
> | 858: kondemand/1
> | 859: kondemand/2
> | ...
> | % sudo ./perf lock -t 27,28,29,30,31 report # track only these threads
> | Name acquired contended total wait (ns) max wait (ns) min wait (ns)
I'm not sure we want such per thread granularity filtering. I'm not
sure it will be very useful. But per process yeah.
And actually we should do that on tracing time rather than on post-processing.
This will lower the tracing overhead a lot.
Ideally I think we need:
./perf lock record ls -R /
This would trace locks taken by this instance of ls only, ie: drop the -a
if we pass a command line.
What do you think?
next prev parent reply other threads:[~2010-05-07 0:49 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-04-24 2:05 [GIT PULL] perf tools updates Frederic Weisbecker
2010-04-24 2:05 ` [PATCH 1/9] perf lock: Fix state machine to recognize lock sequence Frederic Weisbecker
2010-04-24 10:43 ` [PATCH] perf lock: add "info" subcommand for dumping misc information Hitoshi Mitake
2010-04-24 10:46 ` Hitoshi Mitake
2010-04-24 10:46 ` [PATCH v2] " Hitoshi Mitake
2010-04-24 13:41 ` Frederic Weisbecker
2010-04-30 18:49 ` Frederic Weisbecker
2010-05-03 5:11 ` Hitoshi Mitake
2010-05-03 5:12 ` [PATCH v3] " Hitoshi Mitake
2010-05-05 21:10 ` Frederic Weisbecker
2010-05-06 9:31 ` Hitoshi Mitake
2010-05-06 9:32 ` [PATCH] perf lock: track only specified threads Hitoshi Mitake
2010-05-07 0:49 ` Frederic Weisbecker [this message]
2010-05-08 8:02 ` Hitoshi Mitake
2010-05-08 8:10 ` [PATCH] perf lock: Drop "-a" option from set of default arguments to cmd_record() Hitoshi Mitake
2010-05-08 16:14 ` Frederic Weisbecker
2010-05-09 14:53 ` Hitoshi Mitake
2010-05-11 6:48 ` Frederic Weisbecker
2010-05-12 10:23 ` Hitoshi Mitake
2010-05-12 15:51 ` Frederic Weisbecker
2010-05-15 8:54 ` Hitoshi Mitake
2010-05-10 7:19 ` [tip:perf/core] perf lock: Add "info" subcommand for dumping misc information tip-bot for Hitoshi Mitake
2010-04-24 2:05 ` [PATCH 2/9] perf: Fix initialization bug in parse_single_tracepoint_event() Frederic Weisbecker
2010-04-24 2:05 ` [PATCH 3/9] perf: Generalize perf lock's sample event reordering to the session layer Frederic Weisbecker
2010-04-24 2:05 ` [PATCH 4/9] perf: Use generic sample reordering in perf sched Frederic Weisbecker
2010-04-24 2:05 ` [PATCH 5/9] perf: Use generic sample reordering in perf kmem Frederic Weisbecker
2010-04-24 2:05 ` [PATCH 6/9] perf: Use generic sample reordering in perf trace Frederic Weisbecker
2010-04-24 2:05 ` [PATCH 7/9] perf: Use generic sample reordering in perf timechart Frederic Weisbecker
2010-04-24 2:05 ` [PATCH 8/9] perf: Add a perf trace option to check samples ordering reliability Frederic Weisbecker
2010-04-24 16:13 ` Masami Hiramatsu
2010-04-25 18:08 ` Frederic Weisbecker
2010-04-24 2:05 ` [PATCH 9/9] perf: Some perf-kvm documentation edits Frederic Weisbecker
2010-04-24 2:27 ` [GIT PULL] perf tools updates Frederic Weisbecker
2010-04-27 9:15 ` Ingo Molnar
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=20100507004945.GA8069@nowhere \
--to=fweisbec@gmail.com \
--cc=a.p.zijlstra@chello.nl \
--cc=acme@redhat.com \
--cc=h.mitake@gmail.com \
--cc=jbaron@redhat.com \
--cc=jens.axboe@oracle.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=mitake@dcl.info.waseda.ac.jp \
--cc=paulus@samba.org \
--cc=xiaoguangrong@cn.fujitsu.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;
as well as URLs for NNTP newsgroup(s).