From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
To: linux-kernel@vger.kernel.org
Cc: mingo@elte.hu, laijs@cn.fujitsu.com, dipankar@in.ibm.com,
akpm@linux-foundation.org, mathieu.desnoyers@polymtl.ca,
josh@joshtriplett.org, dvhltc@us.ibm.com, niv@us.ibm.com,
tglx@linutronix.de, peterz@infradead.org, rostedt@goodmis.org,
Valdis.Kletnieks@vt.edu, dhowells@redhat.com,
"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
Subject: [PATCH tip/core/rcu 09/13] idr: apply lockdep-based diagnostics to rcu_dereference() uses
Date: Thu, 11 Feb 2010 16:00:35 -0800 [thread overview]
Message-ID: <1265932839-25899-9-git-send-email-paulmck@linux.vnet.ibm.com> (raw)
In-Reply-To: <20100212000016.GA25781@linux.vnet.ibm.com>
Because idr can be used with any of a number of locks or with any
flavor of RCU, just disable the lockdep-based diagnostics. If idr
needs diagnostics, the check expression will need to be passed into
the relevant idr primitives as an additional argument.
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
---
lib/idr.c | 8 ++++----
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/lib/idr.c b/lib/idr.c
index 1cac726..4877fce 100644
--- a/lib/idr.c
+++ b/lib/idr.c
@@ -502,7 +502,7 @@ void *idr_find(struct idr *idp, int id)
int n;
struct idr_layer *p;
- p = rcu_dereference(idp->top);
+ p = rcu_dereference_raw(idp->top);
if (!p)
return NULL;
n = (p->layer+1) * IDR_BITS;
@@ -517,7 +517,7 @@ void *idr_find(struct idr *idp, int id)
while (n > 0 && p) {
n -= IDR_BITS;
BUG_ON(n != p->layer*IDR_BITS);
- p = rcu_dereference(p->ary[(id >> n) & IDR_MASK]);
+ p = rcu_dereference_raw(p->ary[(id >> n) & IDR_MASK]);
}
return((void *)p);
}
@@ -550,7 +550,7 @@ int idr_for_each(struct idr *idp,
struct idr_layer **paa = &pa[0];
n = idp->layers * IDR_BITS;
- p = rcu_dereference(idp->top);
+ p = rcu_dereference_raw(idp->top);
max = 1 << n;
id = 0;
@@ -558,7 +558,7 @@ int idr_for_each(struct idr *idp,
while (n > 0 && p) {
n -= IDR_BITS;
*paa++ = p;
- p = rcu_dereference(p->ary[(id >> n) & IDR_MASK]);
+ p = rcu_dereference_raw(p->ary[(id >> n) & IDR_MASK]);
}
if (p) {
--
1.6.6
next prev parent reply other threads:[~2010-02-12 0:03 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-02-12 0:00 [PATCH tip/core/rcu 0/13] rcu: add lockdep checking, doc update, dyntick GP acceleration Paul E. McKenney
2010-02-12 0:00 ` [PATCH tip/core/rcu 01/13] rcu: introduce lockdep-based checking to RCU read-side primitives Paul E. McKenney
2010-02-12 0:00 ` [PATCH tip/core/rcu 02/13] rcu: add lockdep-enabled variants of rcu_dereference() Paul E. McKenney
2010-02-12 4:13 ` Eric Dumazet
2010-02-12 0:00 ` [PATCH tip/core/rcu 03/13] rcu: integrate rcu_dereference_check() message into lockdep Paul E. McKenney
2010-02-12 0:00 ` [PATCH tip/core/rcu 04/13] rcu: disable lockdep checking in RCU list-traversal primitives Paul E. McKenney
2010-02-12 0:00 ` [PATCH tip/core/rcu 05/13] net: add checking to rcu_dereference() primitives Paul E. McKenney
2010-02-12 4:15 ` Eric Dumazet
2010-02-14 8:23 ` Ingo Molnar
2010-02-15 7:18 ` David Miller
2010-02-14 8:34 ` Michał Mirosław
2010-02-14 8:50 ` Eric Dumazet
2010-02-12 0:00 ` [PATCH tip/core/rcu 06/13] sched: use lockdep-based checking on rcu_dereference() Paul E. McKenney
2010-02-14 10:12 ` Peter Zijlstra
2010-02-14 17:48 ` Paul E. McKenney
2010-02-12 0:00 ` [PATCH tip/core/rcu 07/13] vfs: apply lockdep-based checking to rcu_dereference() uses Paul E. McKenney
2010-02-14 10:12 ` Peter Zijlstra
2010-02-14 17:37 ` Paul E. McKenney
2010-02-12 0:00 ` [PATCH tip/core/rcu 08/13] radix-tree: disable RCU lockdep checking in radix tree Paul E. McKenney
2010-02-12 0:00 ` Paul E. McKenney [this message]
2010-02-12 0:00 ` [PATCH tip/core/rcu 10/13] security: apply lockdep-based checking to rcu_dereference() uses Paul E. McKenney
2010-02-12 0:00 ` [PATCH tip/core/rcu 11/13] rcu: documentation update for CONFIG_PROVE_RCU Paul E. McKenney
2010-02-12 0:00 ` [PATCH tip/core/rcu 12/13] rcu: fix citation of Mathieu's dissertation Paul E. McKenney
2010-02-12 0:00 ` [PATCH tip/core/rcu 13/13] rcu: accelerate grace period if last non-dynticked CPU Paul E. McKenney
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=1265932839-25899-9-git-send-email-paulmck@linux.vnet.ibm.com \
--to=paulmck@linux.vnet.ibm.com \
--cc=Valdis.Kletnieks@vt.edu \
--cc=akpm@linux-foundation.org \
--cc=dhowells@redhat.com \
--cc=dipankar@in.ibm.com \
--cc=dvhltc@us.ibm.com \
--cc=josh@joshtriplett.org \
--cc=laijs@cn.fujitsu.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mathieu.desnoyers@polymtl.ca \
--cc=mingo@elte.hu \
--cc=niv@us.ibm.com \
--cc=peterz@infradead.org \
--cc=rostedt@goodmis.org \
--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
all inboxes | Powered by JetHome®