diff options
| author | Matthew Wilcox (Oracle) <willy@infradead.org> | 2021-04-01 07:44:48 -0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2021-04-16 11:38:14 +0200 |
| commit | 0cdf4c3ea95ace38f3a3de982c37f7eaa31f67f5 (patch) | |
| tree | 305645fc3cc36d9c8d67bdb11902124cab8c80cc | |
| parent | f8ac5cb2f42959c864c47794de12592639aafe9e (diff) | |
| download | linux-0cdf4c3ea95ace38f3a3de982c37f7eaa31f67f5.tar.gz linux-0cdf4c3ea95ace38f3a3de982c37f7eaa31f67f5.tar.bz2 linux-0cdf4c3ea95ace38f3a3de982c37f7eaa31f67f5.zip | |
idr test suite: Take RCU read lock in idr_find_test_1
[ Upstream commit 703586410da69eb40062e64d413ca33bd735917a ]
When run on a single CPU, this test would frequently access already-freed
memory. Due to timing, this bug never showed up on multi-CPU tests.
Reported-by: Chris von Recklinghausen <crecklin@redhat.com>
Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
| -rw-r--r-- | tools/testing/radix-tree/idr-test.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/tools/testing/radix-tree/idr-test.c b/tools/testing/radix-tree/idr-test.c index 44ceff95a9b3..4a9b451b7ba0 100644 --- a/tools/testing/radix-tree/idr-test.c +++ b/tools/testing/radix-tree/idr-test.c @@ -306,11 +306,15 @@ void idr_find_test_1(int anchor_id, int throbber_id) BUG_ON(idr_alloc(&find_idr, xa_mk_value(anchor_id), anchor_id, anchor_id + 1, GFP_KERNEL) != anchor_id); + rcu_read_lock(); do { int id = 0; void *entry = idr_get_next(&find_idr, &id); + rcu_read_unlock(); BUG_ON(entry != xa_mk_value(id)); + rcu_read_lock(); } while (time(NULL) < start + 11); + rcu_read_unlock(); pthread_join(throbber, NULL); |