From: Peter Zijlstra <peterz@infradead.org>
To: Mathieu Desnoyers <compudj@krystal.dyndns.org>
Cc: prasad@linux.vnet.ibm.com, jan.kiszka@web.de,
Gian Lorenzo Meocci <glmeocci@gmail.com>,
ltt-dev@lists.casi.polymtl.ca, linux-kernel@vger.kernel.org,
mbligh@google.com, Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [ltt-dev] trace a futex
Date: Fri, 05 Dec 2008 17:01:39 +0100 [thread overview]
Message-ID: <1228492899.20520.4.camel@twins> (raw)
In-Reply-To: <20081205122822.GA25816@Krystal>
[ resend with a real email address for Thomas ]
On Fri, 2008-12-05 at 07:28 -0500, Mathieu Desnoyers wrote:
> * Peter Zijlstra (peterz@infradead.org) wrote:
> >
> > FWIW, the ftrace infrastructure has on many an occasion (even before it
> > was called ftrace and specific to -rt) helped in debugging and fixing
> > futex races.
> >
>
> Hrm, I'm not sure futex races is the key aspect of interest here.
> Knowing which amount of pthread mutex lock calls ends up calling the
> scheduler looks a bit more like the topic brought by this particular
> use-case. Therefore, correlating the information from the nptl with the
> kernel information would be useful.
We already have the scheduler instrucmentation, so correlating that to
known futex calls (from userspace) shouldn't be too hard.
> Is lockdep called when a futex is taken ?
lockdep only does kernel-internal locks - so no.
> Should we add instrumentation
> (tracepoints) to futex.c ? If yes, was there specific instrumentation
> you used with ftrace that should be added ?
Not sure we should, Thomas did the bulk of that ftrace debugging, Thomas
do you think it would be worthwhile to add some tracepoints in there?
prev parent reply other threads:[~2008-12-05 16:01 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <8d94e9280812021142j2eb5ca72td730410e97ccd9a1@mail.gmail.com>
[not found] ` <20081202195015.GA25792@Krystal>
[not found] ` <8d94e9280812021215m6a63d6c2l4e64acd26274953d@mail.gmail.com>
[not found] ` <20081202215519.GA734@Krystal>
[not found] ` <4935B25F.7010701@web.de>
2008-12-03 5:26 ` [ltt-dev] trace a futex Mathieu Desnoyers
2008-12-03 18:01 ` K.Prasad
2008-12-04 8:53 ` Peter Zijlstra
2008-12-05 12:28 ` Mathieu Desnoyers
2008-12-05 15:58 ` Peter Zijlstra
2008-12-05 16:01 ` Peter Zijlstra [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=1228492899.20520.4.camel@twins \
--to=peterz@infradead.org \
--cc=compudj@krystal.dyndns.org \
--cc=glmeocci@gmail.com \
--cc=jan.kiszka@web.de \
--cc=linux-kernel@vger.kernel.org \
--cc=ltt-dev@lists.casi.polymtl.ca \
--cc=mbligh@google.com \
--cc=prasad@linux.vnet.ibm.com \
--cc=tglx@linutronix.de \
/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