Home » Mailing list archives » linux-kernel » 2008 » August » 4

Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock()

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Dave Jones <davej@...>
To: Peter Zijlstra <a.p.zijlstra@...>
Cc: Roland Dreier <rdreier@...>, Linus Torvalds <torvalds@...>, David Miller <davem@...>, <jeremy@...>, <hugh@...>, <mingo@...>, <akpm@...>, <linux-kernel@...>
Subject: Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock()
Date: Monday, August 4, 2008 - 10:53 am

On Mon, Aug 04, 2008 at 04:32:12PM +0200, Peter Zijlstra wrote:
 > On Mon, 2008-08-04 at 07:26 -0700, Roland Dreier wrote:
 > > > No more than 48 locks (mutexes, rwlocks, spinlock, RCU, everything
 > >  > covered by lockdep) held by any one code-path; including nested
 > >  > interrupt contexts.
 > > 
 > > Does that mean that something like the new mm_take_all_locks() operation
 > > is going to explode if someone tries to use it with lockdep on?
 > 
 > Gah - yes, clearly nobody tried this.. :-/
 > 
 > Just looking at the code it will not only run into this limit, but it
 > would warn about recursion on the second file/anon vma due to utter lack
 > of annotation.
 > 
 > Why are people still developing without lockdep?

More puzzling, is why hasn't this triggered in the Fedora rawhide kernels,
which do have lockdep enabled.

	Dave

-- 
http://www.codemonkey.org.uk
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
Messages in current thread:
[RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Peter Zijlstra, (Mon Aug 4, 9:03 am)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Peter Zijlstra, (Thu Aug 7, 7:25 am)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Jeremy Fitzhardinge, (Mon Aug 4, 2:06 pm)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Peter Zijlstra, (Mon Aug 4, 2:54 pm)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Jeremy Fitzhardinge, (Mon Aug 4, 3:26 pm)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Linus Torvalds, (Mon Aug 4, 3:31 pm)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Steven Rostedt, (Wed Oct 8, 11:27 am)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Nick Piggin, (Wed Oct 8, 11:52 am)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Steven Rostedt, (Wed Oct 8, 1:18 pm)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Linus Torvalds, (Wed Oct 8, 11:43 am)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Steven Rostedt, (Wed Oct 8, 12:03 pm)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Linus Torvalds, (Wed Oct 8, 12:19 pm)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Steven Rostedt, (Wed Oct 8, 12:53 pm)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Jeremy Fitzhardinge, (Mon Aug 4, 4:16 pm)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Peter Zijlstra, (Mon Aug 4, 3:39 pm)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Roland Dreier, (Mon Aug 4, 10:07 am)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Peter Zijlstra, (Mon Aug 4, 10:19 am)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Roland Dreier, (Mon Aug 4, 10:26 am)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Peter Zijlstra, (Mon Aug 4, 10:32 am)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Dave Jones, (Mon Aug 4, 10:53 am)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Peter Zijlstra, (Mon Aug 4, 10:56 am)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Andrea Arcangeli, (Mon Aug 4, 12:26 pm)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Peter Zijlstra, (Mon Aug 4, 12:38 pm)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), David Miller, (Mon Aug 4, 5:32 pm)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Andrea Arcangeli, (Mon Aug 4, 1:27 pm)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Peter Zijlstra, (Mon Aug 4, 2:48 pm)
Re: [RFC][PATCH 7/7] lockdep: spin_lock_nest_lock(), Andrea Arcangeli, (Mon Aug 4, 1:46 pm)
[PATCH] workaround minor lockdep bug triggered by mm_take_al..., Andrea Arcangeli, (Mon Aug 4, 1:57 pm)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., Peter Zijlstra, (Mon Aug 4, 2:48 pm)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., Andrea Arcangeli, (Mon Aug 4, 4:15 pm)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., Peter Zijlstra, (Mon Aug 4, 4:37 pm)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., Andrea Arcangeli, (Mon Aug 4, 5:09 pm)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., Roland Dreier, (Mon Aug 4, 10:00 pm)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., Andrea Arcangeli, (Mon Aug 4, 10:18 pm)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., Roland Dreier, (Tue Aug 5, 8:02 am)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., Andrea Arcangeli, (Tue Aug 5, 8:20 am)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., David Miller, (Mon Aug 4, 5:57 pm)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., Arjan van de Ven, (Mon Aug 4, 5:27 pm)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., Andrea Arcangeli, (Mon Aug 4, 5:54 pm)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., Pekka Enberg, (Mon Aug 4, 5:14 pm)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., Andrea Arcangeli, (Mon Aug 4, 5:30 pm)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., Arjan van de Ven, (Mon Aug 4, 5:42 pm)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., Andrea Arcangeli, (Mon Aug 4, 6:30 pm)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., Arjan van de Ven, (Mon Aug 4, 7:38 pm)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., Andrea Arcangeli, (Mon Aug 4, 8:47 pm)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., Andrew Morton, (Mon Aug 4, 5:41 pm)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., Andrea Arcangeli, (Mon Aug 4, 6:12 pm)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., Roland Dreier, (Mon Aug 4, 2:56 pm)
Re: [PATCH] workaround minor lockdep bug triggered by mm_tak..., Peter Zijlstra, (Mon Aug 4, 3:05 pm)