From: Tvrtko Ursulin <tvrtko.ursu...@intel.com>

Test that VMAs associated with a context are cleaned up when
contexts are destroyed.

In practice this emulates the leak seen between fbcon and X server.
Every time the X server exits we gain one VMA on the fbcon frame
buffer object as externally visible via for example
/sys/kernel/debug/dri/0/i915_gem_gtt.

v2: Use igt_debugfs_open, getline and strstr instead of home-brewed
string matching. (Thomas Wood)

Signed-off-by: Tvrtko Ursulin <tvrtko.ursu...@intel.com>
Cc: Thomas Wood <thomas.w...@intel.com>
---
 tests/gem_ppgtt.c | 86 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 86 insertions(+)

diff --git a/tests/gem_ppgtt.c b/tests/gem_ppgtt.c
index 4f6df063214a..f3e45b0c74d1 100644
--- a/tests/gem_ppgtt.c
+++ b/tests/gem_ppgtt.c
@@ -37,6 +37,7 @@
 #include <drm.h>
 
 #include "intel_bufmgr.h"
+#include "igt_debugfs.h"
 
 #define WIDTH 512
 #define STRIDE (WIDTH*4)
@@ -265,6 +266,88 @@ static void flink_and_close(void)
        close(fd2);
 }
 
+static bool grep_name(const char *fname, const char *match)
+{
+       int fd;
+       FILE *fh;
+       size_t n = 0;
+       char *line = NULL;
+       char *matched = NULL;
+
+       fd = igt_debugfs_open(fname, O_RDONLY);
+       igt_assert(fd >= 0);
+
+       fh = fdopen(fd, "r");
+       igt_assert(fh);
+
+       while (getline(&line, &n, fh) >= 0) {
+               matched = strstr(line, match);
+               if (line) {
+                       free(line);
+                       line = NULL;
+               }
+               if (matched)
+                       break;
+       }
+
+       if (line)
+               free(line);
+       fclose(fh);
+
+       return matched != NULL;
+}
+
+static void flink_and_exit(void)
+{
+       uint32_t fd, fd2;
+       uint32_t bo, flinked_bo, name;
+       char match[100];
+       int to_match;
+       bool matched;
+       int retry = 0;
+       const int retries = 50;
+
+       fd = drm_open_any();
+       igt_require(uses_full_ppgtt(fd));
+
+       bo = gem_create(fd, 4096);
+       name = gem_flink(fd, bo);
+
+       to_match  = snprintf(match, sizeof(match), "(name: %u)", name);
+       igt_assert(to_match < sizeof(match));
+
+       fd2 = drm_open_any();
+
+       flinked_bo = gem_open(fd2, name);
+       exec_and_get_offset(fd2, flinked_bo);
+       gem_sync(fd2, flinked_bo);
+
+       /* Verify looking for string works OK. */
+       matched = grep_name("i915_gem_gtt", match);
+       igt_assert_eq(matched, true);
+
+       gem_close(fd2, flinked_bo);
+
+       /* Close the context. */
+       close(fd2);
+
+retry:
+       /* Give cleanup some time to run. */
+       usleep(100000);
+
+       /* The flinked bo VMA should have been cleared now, so list of VMAs
+        * in debugfs should not contain the one for the imported object.
+        */
+       matched = grep_name("i915_gem_gtt", match);
+       if (matched && retry++ < retries)
+               goto retry;
+
+       igt_assert_eq(matched, false);
+
+       gem_close(fd, bo);
+       close(fd);
+}
+
 #define N_CHILD 8
 int main(int argc, char **argv)
 {
@@ -297,5 +380,8 @@ int main(int argc, char **argv)
        igt_subtest("flink-and-close-vma-leak")
                flink_and_close();
 
+       igt_subtest("flink-and-exit-vma-leak")
+               flink_and_exit();
+
        igt_exit();
 }
-- 
2.5.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to