drm: kselftest for drm_mm and restricted color eviction
authorChris Wilson <chris@chris-wilson.co.uk>
Thu, 22 Dec 2016 08:36:22 +0000 (08:36 +0000)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Tue, 27 Dec 2016 13:16:57 +0000 (14:16 +0100)
Check that after applying the driver's color adjustment, restricted
eviction scanning finds a suitable hole.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/20161222083641.2691-20-chris@chris-wilson.co.uk
drivers/gpu/drm/selftests/drm_mm_selftests.h
drivers/gpu/drm/selftests/test-drm_mm.c

index 0a3a7e32e5f70fb3bce97221352fe6e3282a47b6..6a4575fdc1c08863d2bec323077c5aa442d52895 100644 (file)
@@ -20,3 +20,4 @@ selftest(evict_range, igt_evict_range)
 selftest(topdown, igt_topdown)
 selftest(color, igt_color)
 selftest(color_evict, igt_color_evict)
+selftest(color_evict_range, igt_color_evict_range)
index 6584e4ce96af3d664e61563ef2ebc15f97267e74..fb8f7056225bfe0d1da49991b5fe7c3e80d64ce4 100644 (file)
@@ -1855,6 +1855,7 @@ out:
 }
 
 static int evict_color(struct drm_mm *mm,
+                      u64 range_start, u64 range_end,
                       struct evict_node *nodes,
                       unsigned int *order,
                       unsigned int count,
@@ -1868,7 +1869,9 @@ static int evict_color(struct drm_mm *mm,
        struct drm_mm_node tmp;
        int err;
 
-       drm_mm_init_scan(mm, size, alignment, color);
+       drm_mm_init_scan_with_range(mm,
+                                   size, alignment, color,
+                                   range_start, range_end);
        if (!evict_nodes(mm,
                         nodes, order, count,
                         &evict_list))
@@ -1886,6 +1889,12 @@ static int evict_color(struct drm_mm *mm,
                return err;
        }
 
+       if (tmp.start < range_start || tmp.start + tmp.size > range_end) {
+               pr_err("Inserted [address=%llu + %llu] did not fit into the request range [%llu, %llu]\n",
+                      tmp.start, tmp.size, range_start, range_end);
+               err = -EINVAL;
+       }
+
        if (colors_abutt(&tmp))
                err = -EINVAL;
 
@@ -1954,7 +1963,7 @@ static int igt_color_evict(void *ignored)
        for (mode = evict_modes; mode->name; mode++) {
                for (n = 1; n <= total_size; n <<= 1) {
                        drm_random_reorder(order, total_size, &prng);
-                       err = evict_color(&mm,
+                       err = evict_color(&mm, 0, U64_MAX,
                                          nodes, order, total_size,
                                          n, 1, color++,
                                          mode);
@@ -1967,7 +1976,7 @@ static int igt_color_evict(void *ignored)
 
                for (n = 1; n < total_size; n <<= 1) {
                        drm_random_reorder(order, total_size, &prng);
-                       err = evict_color(&mm,
+                       err = evict_color(&mm, 0, U64_MAX,
                                          nodes, order, total_size,
                                          total_size/2, n, color++,
                                          mode);
@@ -1984,7 +1993,7 @@ static int igt_color_evict(void *ignored)
                        DRM_MM_BUG_ON(!nsize);
 
                        drm_random_reorder(order, total_size, &prng);
-                       err = evict_color(&mm,
+                       err = evict_color(&mm, 0, U64_MAX,
                                          nodes, order, total_size,
                                          nsize, n, color++,
                                          mode);
@@ -2010,6 +2019,105 @@ err:
        return ret;
 }
 
+static int igt_color_evict_range(void *ignored)
+{
+       DRM_RND_STATE(prng, random_seed);
+       const unsigned int total_size = 8192;
+       const unsigned int range_size = total_size / 2;
+       const unsigned int range_start = total_size / 4;
+       const unsigned int range_end = range_start + range_size;
+       const struct insert_mode *mode;
+       unsigned long color = 0;
+       struct drm_mm mm;
+       struct evict_node *nodes;
+       struct drm_mm_node *node, *next;
+       unsigned int *order, n;
+       int ret, err;
+
+       /* Like igt_color_evict(), but limited to small portion of the full
+        * drm_mm range.
+        */
+
+       ret = -ENOMEM;
+       nodes = vzalloc(total_size * sizeof(*nodes));
+       if (!nodes)
+               goto err;
+
+       order = drm_random_order(total_size, &prng);
+       if (!order)
+               goto err_nodes;
+
+       ret = -EINVAL;
+       drm_mm_init(&mm, 0, 2*total_size - 1);
+       mm.color_adjust = separate_adjacent_colors;
+       for (n = 0; n < total_size; n++) {
+               if (!expect_insert(&mm, &nodes[n].node,
+                                  1, 0, color++,
+                                  &insert_modes[0])) {
+                       pr_err("insert failed, step %d\n", n);
+                       goto out;
+               }
+       }
+
+       for (mode = evict_modes; mode->name; mode++) {
+               for (n = 1; n <= range_size; n <<= 1) {
+                       drm_random_reorder(order, range_size, &prng);
+                       err = evict_color(&mm, range_start, range_end,
+                                         nodes, order, total_size,
+                                         n, 1, color++,
+                                         mode);
+                       if (err) {
+                               pr_err("%s evict_color(size=%u) failed for range [%x, %x]\n",
+                                      mode->name, n, range_start, range_end);
+                               goto out;
+                       }
+               }
+
+               for (n = 1; n < range_size; n <<= 1) {
+                       drm_random_reorder(order, total_size, &prng);
+                       err = evict_color(&mm, range_start, range_end,
+                                         nodes, order, total_size,
+                                         range_size/2, n, color++,
+                                         mode);
+                       if (err) {
+                               pr_err("%s evict_color(size=%u, alignment=%u) failed for range [%x, %x]\n",
+                                      mode->name, total_size/2, n, range_start, range_end);
+                               goto out;
+                       }
+               }
+
+               for_each_prime_number_from(n, 1, min(range_size, max_prime)) {
+                       unsigned int nsize = (range_size - n + 1) / 2;
+
+                       DRM_MM_BUG_ON(!nsize);
+
+                       drm_random_reorder(order, total_size, &prng);
+                       err = evict_color(&mm, range_start, range_end,
+                                         nodes, order, total_size,
+                                         nsize, n, color++,
+                                         mode);
+                       if (err) {
+                               pr_err("%s evict_color(size=%u, alignment=%u) failed for range [%x, %x]\n",
+                                      mode->name, nsize, n, range_start, range_end);
+                               goto out;
+                       }
+               }
+       }
+
+       ret = 0;
+out:
+       if (ret)
+               drm_mm_debug_table(&mm, __func__);
+       drm_mm_for_each_node_safe(node, next, &mm)
+               drm_mm_remove_node(node);
+       drm_mm_takedown(&mm);
+       kfree(order);
+err_nodes:
+       vfree(nodes);
+err:
+       return ret;
+}
+
 #include "drm_selftest.c"
 
 static int __init test_drm_mm_init(void)