Re: [PATCH v3] timer: implement lockdep deadlock detection

From: Johannes Berg
Date: Wed Jan 28 2009 - 06:00:09 EST


On Wed, 2009-01-28 at 11:13 +0100, Peter Zijlstra wrote:
> On Wed, 2009-01-28 at 10:54 +0100, Johannes Berg wrote:
> > On Wed, 2009-01-28 at 09:20 +0100, Peter Zijlstra wrote:
> > > On Tue, 2009-01-27 at 19:57 +0100, Johannes Berg wrote:
> > > > This modifies the timer code in a way to allow lockdep to detect
> > > > deadlocks resulting from a lock being taken in the timer function
> > > > as well as around the del_timer_sync() call.
> > > >
> > > > Signed-off-by: Johannes Berg <johannes@xxxxxxxxxxxxxxxx>
> > >
> > > Nice, thanks!
> >
> > I actually got "trying to register non-static key" on my powerpc64
> > machine. Is there a possibility that functions are not static??
>
> Hmm, weird, afaict static_obj() includes both text and data, for the
> core kernel as well as modules.

Yeah, I'd think it should. I'll run it by the powerpc list when I get it
again, it only seems to happen very rarely.

johannes

Attachment: signature.asc
Description: This is a digitally signed message part