aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatthew Wilcox2018-08-28 16:13:16 -0400
committerMatthew Wilcox2018-10-21 10:46:46 -0400
commitd6427f8179b5dd65eb468c61fc8cc24657c336c9 (patch)
treebe783289bf2c41109ad5386fc62cc7973a0e770e
parent47e0fab2b15155e33fdff777c791bebfd5855bbc (diff)
xarray: Move multiorder account test in-kernel
Move this test to the in-kernel test suite, and enhance it to test several different orders. Signed-off-by: Matthew Wilcox <willy@infradead.org>
-rw-r--r--lib/test_xarray.c32
-rw-r--r--tools/testing/radix-tree/multiorder.c24
2 files changed, 32 insertions, 24 deletions
diff --git a/lib/test_xarray.c b/lib/test_xarray.c
index ca86141641cb..38cab4ccb24e 100644
--- a/lib/test_xarray.c
+++ b/lib/test_xarray.c
@@ -1019,6 +1019,37 @@ static noinline void check_workingset(struct xarray *xa, unsigned long index)
XA_BUG_ON(xa, !xa_empty(xa));
}
+/*
+ * Check that the pointer / value / sibling entries are accounted the
+ * way we expect them to be.
+ */
+static noinline void check_account(struct xarray *xa)
+{
+#ifdef CONFIG_XARRAY_MULTI
+ unsigned int order;
+
+ for (order = 1; order < 12; order++) {
+ XA_STATE(xas, xa, 1 << order);
+
+ xa_store_order(xa, 0, order, xa, GFP_KERNEL);
+ xas_load(&xas);
+ XA_BUG_ON(xa, xas.xa_node->count == 0);
+ XA_BUG_ON(xa, xas.xa_node->count > (1 << order));
+ XA_BUG_ON(xa, xas.xa_node->nr_values != 0);
+
+ xa_store_order(xa, 1 << order, order, xa_mk_value(1 << order),
+ GFP_KERNEL);
+ XA_BUG_ON(xa, xas.xa_node->count != xas.xa_node->nr_values * 2);
+
+ xa_erase(xa, 1 << order);
+ XA_BUG_ON(xa, xas.xa_node->nr_values != 0);
+
+ xa_erase(xa, 0);
+ XA_BUG_ON(xa, !xa_empty(xa));
+ }
+#endif
+}
+
static noinline void check_destroy(struct xarray *xa)
{
unsigned long index;
@@ -1068,6 +1099,7 @@ static int xarray_checks(void)
check_xa_alloc();
check_find(&array);
check_find_entry(&array);
+ check_account(&array);
check_destroy(&array);
check_move(&array);
check_create_range(&array);
diff --git a/tools/testing/radix-tree/multiorder.c b/tools/testing/radix-tree/multiorder.c
index 0436554a099a..dc27a3da210a 100644
--- a/tools/testing/radix-tree/multiorder.c
+++ b/tools/testing/radix-tree/multiorder.c
@@ -356,29 +356,6 @@ void multiorder_tagged_iteration(void)
item_kill_tree(&tree);
}
-static void multiorder_account(void)
-{
- RADIX_TREE(tree, GFP_KERNEL);
- struct radix_tree_node *node;
- void **slot;
-
- item_insert_order(&tree, 0, 5);
-
- __radix_tree_insert(&tree, 1 << 5, 5, xa_mk_value(5));
- __radix_tree_lookup(&tree, 0, &node, NULL);
- assert(node->count == node->nr_values * 2);
- radix_tree_delete(&tree, 1 << 5);
- assert(node->nr_values == 0);
-
- __radix_tree_insert(&tree, 1 << 5, 5, xa_mk_value(5));
- __radix_tree_lookup(&tree, 1 << 5, &node, &slot);
- assert(node->count == node->nr_values * 2);
- __radix_tree_replace(&tree, node, slot, NULL);
- assert(node->nr_values == 0);
-
- item_kill_tree(&tree);
-}
-
bool stop_iteration = false;
static void *creator_func(void *ptr)
@@ -457,7 +434,6 @@ void multiorder_checks(void)
multiorder_tag_tests();
multiorder_iteration();
multiorder_tagged_iteration();
- multiorder_account();
multiorder_iteration_race();
radix_tree_cpu_dead(0);