Thread (26 messages) 26 messages, 6 authors, 2007-08-08

Re: [PATCH -rt 1/9] preempt rcu: check for underflow

flat view

From: Ingo Molnar <hidden>
Date: 2007-07-30 09:22:32
Also in: lkml

* Daniel Walker [off-list ref] wrote:
quoted hunk ↗ jump to hunk
+	/*
+	 * If our rcu_read_lock_nesting went negative, likely
+	 * something is wrong..
+	 */
+	WARN_ON(current->rcu_read_lock_nesting < 0);
have you actually caught any rcu locking problem this way? Double 
unlocks should be caught by lockdep already, at a higher level.

in any case i've added a slightly different form of this change to the 
-rt queue that will also check for counter overflows. But i'm not sure 
we want to litter the code with trivial checks like this, so i'm keeping 
it separate and if it does not trigger anything real i'll remove it.

	Ingo
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help