Skip to content

Commit 77d30b1

Browse files
committed
aio: fix rcu sparse warnings introduced by ioctx table lookup patch
Sseveral sparse warnings were caused by missing rcu_dereference() annotations for dereferencing mm->ioctx_table. Thankfully, none of those were actual bugs as the deref was protected by a spin lock in all instances. Signed-off-by: Benjamin LaHaise <bcrl@kvack.org> Reported-by: Fengguang Wu <fengguang.wu@intel.com>
1 parent 79bd1bc commit 77d30b1

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

fs/aio.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -470,7 +470,7 @@ static int ioctx_add_table(struct kioctx *ctx, struct mm_struct *mm)
470470
struct aio_ring *ring;
471471

472472
spin_lock(&mm->ioctx_lock);
473-
table = mm->ioctx_table;
473+
table = rcu_dereference(mm->ioctx_table);
474474

475475
while (1) {
476476
if (table)
@@ -498,7 +498,7 @@ static int ioctx_add_table(struct kioctx *ctx, struct mm_struct *mm)
498498
table->nr = new_nr;
499499

500500
spin_lock(&mm->ioctx_lock);
501-
old = mm->ioctx_table;
501+
old = rcu_dereference(mm->ioctx_table);
502502

503503
if (!old) {
504504
rcu_assign_pointer(mm->ioctx_table, table);
@@ -622,7 +622,7 @@ static void kill_ioctx(struct mm_struct *mm, struct kioctx *ctx)
622622
struct kioctx_table *table;
623623

624624
spin_lock(&mm->ioctx_lock);
625-
table = mm->ioctx_table;
625+
table = rcu_dereference(mm->ioctx_table);
626626

627627
WARN_ON(ctx != table->table[ctx->id]);
628628
table->table[ctx->id] = NULL;

0 commit comments

Comments
 (0)