diff options
author | Matthew Wilcox <willy@infradead.org> | 2018-12-06 08:19:13 -0500 |
---|---|---|
committer | Matthew Wilcox <willy@infradead.org> | 2018-12-06 08:26:16 -0500 |
commit | eff3860bbfedbac6edac57fb0d7f3a60e860c1c3 (patch) | |
tree | 99b3f3e6321a16cfba84319c4a94a1aa653e68d9 /tools/testing/radix-tree/Makefile | |
parent | cf76c364a1e1e5224af80edf70a1e3023e1fcf8c (diff) | |
download | linux-stable-eff3860bbfedbac6edac57fb0d7f3a60e860c1c3.tar.gz linux-stable-eff3860bbfedbac6edac57fb0d7f3a60e860c1c3.tar.bz2 linux-stable-eff3860bbfedbac6edac57fb0d7f3a60e860c1c3.zip |
radix tree: Don't return retry entries from lookup
Commit 66ee620f06f9 ("idr: Permit any valid kernel pointer to be stored")
changed the radix tree lookup so that it stops when reaching the bottom
of the tree. However, the condition was added in the wrong place,
making it possible to return retry entries to the caller. Reorder the
tests to check for the retry entry before checking whether we're at the
bottom of the tree. The retry entry should never be found in the tree
root, so it's safe to defer the check until the end of the loop.
Add a regression test to the test-suite to be sure this doesn't come
back.
Fixes: 66ee620f06f9 ("idr: Permit any valid kernel pointer to be stored")
Reported-by: Greg Kurz <groug@kaod.org>
Signed-off-by: Matthew Wilcox <willy@infradead.org>
Diffstat (limited to 'tools/testing/radix-tree/Makefile')
-rw-r--r-- | tools/testing/radix-tree/Makefile | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tools/testing/radix-tree/Makefile b/tools/testing/radix-tree/Makefile index acf1afa01c5b..397d6b612502 100644 --- a/tools/testing/radix-tree/Makefile +++ b/tools/testing/radix-tree/Makefile @@ -7,6 +7,7 @@ LDLIBS+= -lpthread -lurcu TARGETS = main idr-test multiorder xarray CORE_OFILES := xarray.o radix-tree.o idr.o linux.o test.o find_bit.o bitmap.o OFILES = main.o $(CORE_OFILES) regression1.o regression2.o regression3.o \ + regression4.o \ tag_check.o multiorder.o idr-test.o iteration_check.o benchmark.o ifndef SHIFT |