test_ida: Move ida_check_max
authorMatthew Wilcox <willy@infradead.org>
Mon, 18 Jun 2018 21:25:20 +0000 (17:25 -0400)
committerMatthew Wilcox <willy@infradead.org>
Wed, 22 Aug 2018 03:54:20 +0000 (23:54 -0400)
Convert to new API and move to kernel space.

Signed-off-by: Matthew Wilcox <willy@infradead.org>
lib/test_ida.c
tools/testing/radix-tree/idr-test.c

index 1988f91a20c833e2097e324fc14933eb40cc5674..44174ec9f5bf89ff07905a757ff2d5b6ab2f1758 100644 (file)
@@ -47,6 +47,28 @@ static void ida_check_leaf(struct ida *ida, unsigned int base)
        IDA_BUG_ON(ida, !ida_is_empty(ida));
 }
 
+/*
+ * Check allocations up to and slightly above the maximum allowed (2^31-1) ID.
+ * Allocating up to 2^31-1 should succeed, and then allocating the next one
+ * should fail.
+ */
+static void ida_check_max(struct ida *ida)
+{
+       unsigned long i, j;
+
+       for (j = 1; j < 65537; j *= 2) {
+               unsigned long base = (1UL << 31) - j;
+               for (i = 0; i < j; i++) {
+                       IDA_BUG_ON(ida, ida_alloc_min(ida, base, GFP_KERNEL) !=
+                                       base + i);
+               }
+               IDA_BUG_ON(ida, ida_alloc_min(ida, base, GFP_KERNEL) !=
+                               -ENOSPC);
+               ida_destroy(ida);
+               IDA_BUG_ON(ida, !ida_is_empty(ida));
+       }
+}
+
 static int ida_checks(void)
 {
        DEFINE_IDA(ida);
@@ -55,6 +77,7 @@ static int ida_checks(void)
        ida_check_leaf(&ida, 0);
        ida_check_leaf(&ida, 1024);
        ida_check_leaf(&ida, 1024 * 64);
+       ida_check_max(&ida);
 
        printk("IDA: %u of %u tests passed\n", tests_passed, tests_run);
        return (tests_run != tests_passed) ? 0 : -EINVAL;
index fef1f45b927bb5eded9657ed71e6a274ed86538e..bd9699327f95a910e52cb2345cd1821289c0ff7b 100644 (file)
@@ -396,33 +396,6 @@ void ida_check_conv(void)
        ida_destroy(&ida);
 }
 
-/*
- * Check allocations up to and slightly above the maximum allowed (2^31-1) ID.
- * Allocating up to 2^31-1 should succeed, and then allocating the next one
- * should fail.
- */
-void ida_check_max(void)
-{
-       DEFINE_IDA(ida);
-       int id, err;
-       unsigned long i, j;
-
-       for (j = 1; j < 65537; j *= 2) {
-               unsigned long base = (1UL << 31) - j;
-               for (i = 0; i < j; i++) {
-                       assert(ida_pre_get(&ida, GFP_KERNEL));
-                       assert(!ida_get_new_above(&ida, base, &id));
-                       assert(id == base + i);
-               }
-               assert(ida_pre_get(&ida, GFP_KERNEL));
-               err = ida_get_new_above(&ida, base, &id);
-               assert(err == -ENOSPC);
-               ida_destroy(&ida);
-               assert(ida_is_empty(&ida));
-               rcu_barrier();
-       }
-}
-
 void ida_check_random(void)
 {
        DEFINE_IDA(ida);
@@ -534,7 +507,6 @@ void user_ida_checks(void)
        ida_destroy(&ida);
        assert(ida_is_empty(&ida));
 
-       ida_check_max();
        ida_check_conv();
        ida_check_random();
        ida_simple_get_remove_test();