Skip to content

Commit 7e73eb0

Browse files
author
Matthew Wilcox
committed
idr: Add missing __rcu annotations
Where we use the radix tree iteration macros, we need to annotate 'slot' with __rcu. Make sure we don't forget any new places in the future with the same CFLAGS check used for radix-tree.c. Signed-off-by: Matthew Wilcox <[email protected]>
1 parent d7b6272 commit 7e73eb0

File tree

2 files changed

+8
-7
lines changed

2 files changed

+8
-7
lines changed

lib/Makefile

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ lib-y := ctype.o string.o vsprintf.o cmdline.o \
2525
earlycpio.o seq_buf.o nmi_backtrace.o nodemask.o win_minmax.o
2626

2727
CFLAGS_radix-tree.o += -DCONFIG_SPARSE_RCU_POINTER
28+
CFLAGS_idr.o += -DCONFIG_SPARSE_RCU_POINTER
2829

2930
lib-$(CONFIG_MMU) += ioremap.o
3031
lib-$(CONFIG_SMP) += cpumask.o

lib/idr.c

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ static DEFINE_SPINLOCK(simple_ida_lock);
2828
*/
2929
int idr_alloc(struct idr *idr, void *ptr, int start, int end, gfp_t gfp)
3030
{
31-
void **slot;
31+
void __rcu **slot;
3232
struct radix_tree_iter iter;
3333

3434
if (WARN_ON_ONCE(start < 0))
@@ -98,7 +98,7 @@ int idr_for_each(const struct idr *idr,
9898
int (*fn)(int id, void *p, void *data), void *data)
9999
{
100100
struct radix_tree_iter iter;
101-
void **slot;
101+
void __rcu **slot;
102102

103103
radix_tree_for_each_slot(slot, &idr->idr_rt, &iter, 0) {
104104
int ret = fn(iter.index, rcu_dereference_raw(*slot), data);
@@ -123,7 +123,7 @@ EXPORT_SYMBOL(idr_for_each);
123123
void *idr_get_next(struct idr *idr, int *nextid)
124124
{
125125
struct radix_tree_iter iter;
126-
void **slot;
126+
void __rcu **slot;
127127

128128
slot = radix_tree_iter_find(&idr->idr_rt, &iter, *nextid);
129129
if (!slot)
@@ -151,7 +151,7 @@ EXPORT_SYMBOL(idr_get_next);
151151
void *idr_replace(struct idr *idr, void *ptr, int id)
152152
{
153153
struct radix_tree_node *node;
154-
void **slot = NULL;
154+
void __rcu **slot = NULL;
155155
void *entry;
156156

157157
if (WARN_ON_ONCE(id < 0))
@@ -250,7 +250,7 @@ EXPORT_SYMBOL(idr_replace);
250250
int ida_get_new_above(struct ida *ida, int start, int *id)
251251
{
252252
struct radix_tree_root *root = &ida->ida_rt;
253-
void **slot;
253+
void __rcu **slot;
254254
struct radix_tree_iter iter;
255255
struct ida_bitmap *bitmap;
256256
unsigned long index;
@@ -350,7 +350,7 @@ void ida_remove(struct ida *ida, int id)
350350
struct ida_bitmap *bitmap;
351351
unsigned long *btmp;
352352
struct radix_tree_iter iter;
353-
void **slot;
353+
void __rcu **slot;
354354

355355
slot = radix_tree_iter_lookup(&ida->ida_rt, &iter, index);
356356
if (!slot)
@@ -396,7 +396,7 @@ EXPORT_SYMBOL(ida_remove);
396396
void ida_destroy(struct ida *ida)
397397
{
398398
struct radix_tree_iter iter;
399-
void **slot;
399+
void __rcu **slot;
400400

401401
radix_tree_for_each_slot(slot, &ida->ida_rt, &iter, 0) {
402402
struct ida_bitmap *bitmap = rcu_dereference_raw(*slot);

0 commit comments

Comments
 (0)