Commit 0a385639 authored by Matthew Wilcox's avatar Matthew Wilcox

test_ida: Move ida_check_leaf

Convert to new API and move to kernel space.  Take the opportunity to
test the situation a little more thoroughly (ie at different offsets).
Signed-off-by: default avatarMatthew Wilcox <willy@infradead.org>
parent 06b01113
...@@ -25,11 +25,36 @@ void ida_dump(struct ida *ida) { } ...@@ -25,11 +25,36 @@ void ida_dump(struct ida *ida) { }
} \ } \
} while (0) } while (0)
/*
* Check what happens when we fill a leaf and then delete it. This may
* discover mishandling of IDR_FREE.
*/
static void ida_check_leaf(struct ida *ida, unsigned int base)
{
unsigned long i;
for (i = 0; i < IDA_BITMAP_BITS; i++) {
IDA_BUG_ON(ida, ida_alloc_min(ida, base, GFP_KERNEL) !=
base + i);
}
ida_destroy(ida);
IDA_BUG_ON(ida, !ida_is_empty(ida));
IDA_BUG_ON(ida, ida_alloc(ida, GFP_KERNEL) != 0);
IDA_BUG_ON(ida, ida_is_empty(ida));
ida_free(ida, 0);
IDA_BUG_ON(ida, !ida_is_empty(ida));
}
static int ida_checks(void) static int ida_checks(void)
{ {
DEFINE_IDA(ida); DEFINE_IDA(ida);
IDA_BUG_ON(&ida, !ida_is_empty(&ida)); IDA_BUG_ON(&ida, !ida_is_empty(&ida));
ida_check_leaf(&ida, 0);
ida_check_leaf(&ida, 1024);
ida_check_leaf(&ida, 1024 * 64);
printk("IDA: %u of %u tests passed\n", tests_passed, tests_run); printk("IDA: %u of %u tests passed\n", tests_passed, tests_run);
return (tests_run != tests_passed) ? 0 : -EINVAL; return (tests_run != tests_passed) ? 0 : -EINVAL;
......
...@@ -336,32 +336,6 @@ void ida_check_nomem(void) ...@@ -336,32 +336,6 @@ void ida_check_nomem(void)
IDA_BUG_ON(&ida, !ida_is_empty(&ida)); IDA_BUG_ON(&ida, !ida_is_empty(&ida));
} }
/*
* Check what happens when we fill a leaf and then delete it. This may
* discover mishandling of IDR_FREE.
*/
void ida_check_leaf(void)
{
DEFINE_IDA(ida);
int id;
unsigned long i;
for (i = 0; i < IDA_BITMAP_BITS; i++) {
assert(ida_pre_get(&ida, GFP_KERNEL));
assert(!ida_get_new(&ida, &id));
assert(id == i);
}
ida_destroy(&ida);
assert(ida_is_empty(&ida));
assert(ida_pre_get(&ida, GFP_KERNEL));
assert(!ida_get_new(&ida, &id));
assert(id == 0);
ida_destroy(&ida);
assert(ida_is_empty(&ida));
}
/* /*
* Check handling of conversions between exceptional entries and full bitmaps. * Check handling of conversions between exceptional entries and full bitmaps.
*/ */
...@@ -560,7 +534,6 @@ void user_ida_checks(void) ...@@ -560,7 +534,6 @@ void user_ida_checks(void)
ida_destroy(&ida); ida_destroy(&ida);
assert(ida_is_empty(&ida)); assert(ida_is_empty(&ida));
ida_check_leaf();
ida_check_max(); ida_check_max();
ida_check_conv(); ida_check_conv();
ida_check_random(); ida_check_random();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment