igt/gem_userptr_blits: Fix forked access test
[platform/upstream/intel-gpu-tools.git] / tests / gem_reset_stats.c
index a98e2ad..3f5a01f 100644 (file)
@@ -26,6 +26,7 @@
  */
 
 #define _GNU_SOURCE
+#include <stdbool.h>
 #include <unistd.h>
 #include <stdlib.h>
 #include <stdio.h>
 #include <time.h>
 #include <signal.h>
 
-#include "i915_drm.h"
-#include "intel_bufmgr.h"
-#include "intel_batchbuffer.h"
-#include "intel_gpu_tools.h"
-#include "rendercopy.h"
+#include "ioctl_wrappers.h"
+#include "drmtest.h"
 #include "igt_debugfs.h"
+#include "intel_chipset.h"
+#include "intel_io.h"
+#include "igt_aux.h"
 
 #define RS_NO_ERROR      0
 #define RS_BATCH_ACTIVE  (1 << 0)
 #define RS_BATCH_PENDING (1 << 1)
 #define RS_UNKNOWN       (1 << 2)
 
+static uint32_t devid;
+static bool hw_contexts;
+
 struct local_drm_i915_reset_stats {
        __u32 ctx_id;
        __u32 flags;
@@ -76,8 +80,6 @@ struct local_drm_i915_gem_context_destroy {
 #define CONTEXT_DESTROY_IOCTL DRM_IOWR(DRM_COMMAND_BASE + 0x2e, struct local_drm_i915_gem_context_destroy)
 #define GET_RESET_STATS_IOCTL DRM_IOWR(DRM_COMMAND_BASE + 0x32, struct local_drm_i915_reset_stats)
 
-static igt_debugfs_t dfs;
-
 #define LOCAL_I915_EXEC_VEBOX  (4 << 0)
 
 struct target_ring;
@@ -103,6 +105,9 @@ static const struct target_ring {
 
 static bool has_context(const struct target_ring *ring)
 {
+       if (!hw_contexts)
+               return false;
+
        if(ring->exec == I915_EXEC_RENDER)
                return true;
 
@@ -237,21 +242,10 @@ static int exec_valid(int fd, int ctx)
        return exec_valid_ring(fd, ctx, current_ring->exec);
 }
 
-static void stop_rings(void)
-{
-       int fd;
-
-       fd = igt_debugfs_open(&dfs, "i915_ring_stop", O_WRONLY);
-       igt_assert(fd >= 0);
-
-       igt_assert(write(fd, "0xff", 4) == 4);
-       close(fd);
-}
-
 #define BUFSIZE (4 * 1024)
 #define ITEMS   (BUFSIZE >> 2)
 
-static int inject_hang_ring(int fd, int ctx, int ring)
+static int inject_hang_ring(int fd, int ctx, int ring, bool ignore_ban_error)
 {
        struct drm_i915_gem_execbuffer2 execbuf;
        struct drm_i915_gem_exec_object2 exec;
@@ -259,10 +253,11 @@ static int inject_hang_ring(int fd, int ctx, int ring)
        uint32_t *buf;
        int roff, i;
        unsigned cmd_len = 2;
+       enum stop_ring_flags flags;
 
        srandom(time(NULL));
 
-       if (intel_gen(intel_get_drm_devid(fd)) >= 8)
+       if (intel_gen(devid) >= 8)
                cmd_len = 3;
 
        buf = malloc(BUFSIZE);
@@ -300,18 +295,18 @@ static int inject_hang_ring(int fd, int ctx, int ring)
        for (i = 0; i < ITEMS; i++)
                buf[i] = MI_NOOP;
 
-       roff = random() % (ITEMS - cmd_len);
+       roff = random() % (ITEMS - cmd_len - 1);
        buf[roff] = MI_BATCH_BUFFER_START | (cmd_len - 2);
        buf[roff + 1] = (gtt_off & 0xfffffffc) + (roff << 2);
        if (cmd_len == 3)
-               buf[roff + 2] = gtt_off & 0xffffffff00000000ull;
-
-#ifdef VERBOSE
-       printf("loop injected at 0x%lx (off 0x%x, bo_start 0x%lx, bo_end 0x%lx)\n",
-              (long unsigned int)((roff << 2) + gtt_off),
-              roff << 2, (long unsigned int)gtt_off,
-              (long unsigned int)(gtt_off + BUFSIZE - 1));
-#endif
+               buf[roff + 2] = (gtt_off & 0xffffffff00000000ull) >> 32;
+
+       buf[roff + cmd_len] = MI_BATCH_BUFFER_END;
+
+       igt_debug("loop injected at 0x%lx (off 0x%x, bo_start 0x%lx, bo_end 0x%lx)\n",
+                 (long unsigned int)((roff << 2) + gtt_off),
+                 roff << 2, (long unsigned int)gtt_off,
+                 (long unsigned int)(gtt_off + BUFSIZE - 1));
        gem_write(fd, exec.handle, 0, buf, BUFSIZE);
 
        exec.relocation_count = 0;
@@ -340,14 +335,26 @@ static int inject_hang_ring(int fd, int ctx, int ring)
 
        free(buf);
 
-       stop_rings();
+       flags = igt_to_stop_ring_flag(ring);
+
+       flags |= STOP_RING_ALLOW_BAN;
+
+       if (!ignore_ban_error)
+               flags |= STOP_RING_ALLOW_ERRORS;
+
+       igt_set_stop_rings(flags);
 
        return exec.handle;
 }
 
 static int inject_hang(int fd, int ctx)
 {
-       return inject_hang_ring(fd, ctx, current_ring->exec);
+       return inject_hang_ring(fd, ctx, current_ring->exec, false);
+}
+
+static int inject_hang_no_ban_error(int fd, int ctx)
+{
+       return inject_hang_ring(fd, ctx, current_ring->exec, true);
 }
 
 static int _assert_reset_status(int fd, int ctx, int status)
@@ -356,14 +363,14 @@ static int _assert_reset_status(int fd, int ctx, int status)
 
        rs = gem_reset_status(fd, ctx);
        if (rs < 0) {
-               printf("reset status for %d ctx %d returned %d\n",
-                      fd, ctx, rs);
+               igt_info("reset status for %d ctx %d returned %d\n",
+                        fd, ctx, rs);
                return rs;
        }
 
        if (rs != status) {
-               printf("%d:%d reset status %d differs from assumed %d\n",
-                      fd, ctx, rs, status);
+               igt_info("%d:%d reset status %d differs from assumed %d\n",
+                        fd, ctx, rs, status);
 
                return 1;
        }
@@ -529,7 +536,7 @@ static void test_ban(void)
        assert_reset_status(fd_bad, 0, RS_NO_ERROR);
        assert_reset_status(fd_good, 0, RS_NO_ERROR);
 
-       h2 = inject_hang(fd_bad, 0);
+       h2 = inject_hang_no_ban_error(fd_bad, 0);
        igt_assert(h2 >= 0);
        active_count++;
        /* Second hang will be pending for this */
@@ -539,7 +546,7 @@ static void test_ban(void)
        h7 = exec_valid(fd_good, 0);
 
         while (retry--) {
-                h3 = inject_hang(fd_bad, 0);
+                h3 = inject_hang_no_ban_error(fd_bad, 0);
                 igt_assert(h3 >= 0);
                 gem_sync(fd_bad, h3);
                active_count++;
@@ -559,7 +566,7 @@ static void test_ban(void)
                 gem_close(fd_bad, h3);
                 gem_close(fd_bad, h4);
 
-                printf("retrying for ban (%d)\n", retry);
+                igt_info("retrying for ban (%d)\n", retry);
         }
 
        igt_assert(h4 == -EIO);
@@ -621,7 +628,7 @@ static void test_ban_ctx(void)
        assert_reset_status(fd, ctx_good, RS_NO_ERROR);
        assert_reset_status(fd, ctx_bad, RS_NO_ERROR);
 
-       h2 = inject_hang(fd, ctx_bad);
+       h2 = inject_hang_no_ban_error(fd, ctx_bad);
        igt_assert(h2 >= 0);
        active_count++;
        /* Second hang will be pending for this */
@@ -631,7 +638,7 @@ static void test_ban_ctx(void)
        h7 = exec_valid(fd, ctx_good);
 
         while (retry--) {
-                h3 = inject_hang(fd, ctx_bad);
+                h3 = inject_hang_no_ban_error(fd, ctx_bad);
                 igt_assert(h3 >= 0);
                 gem_sync(fd, h3);
                active_count++;
@@ -651,7 +658,7 @@ static void test_ban_ctx(void)
                 gem_close(fd, h3);
                 gem_close(fd, h4);
 
-                printf("retrying for ban (%d)\n", retry);
+                igt_info("retrying for ban (%d)\n", retry);
         }
 
        igt_assert(h4 == -EIO);
@@ -874,17 +881,6 @@ static void test_close_pending_fork(const bool reverse)
        close(fd);
 }
 
-static void drop_root(void)
-{
-       igt_assert(getuid() == 0);
-
-       igt_assert(setgid(2) == 0);
-       igt_assert(setuid(2) == 0);
-
-       igt_assert(getgid() == 2);
-       igt_assert(getuid() == 2);
-}
-
 static void test_reset_count(const bool create_ctx)
 {
        int fd, h, ctx;
@@ -912,7 +908,7 @@ static void test_reset_count(const bool create_ctx)
        igt_assert(c2 == (c1 + 1));
 
        igt_fork(child, 1) {
-               drop_root();
+               igt_drop_root();
 
                c2 = get_reset_count(fd, ctx);
 
@@ -956,7 +952,7 @@ static int _test_params(int fd, int ctx, uint32_t flags, uint32_t pad)
 
 typedef enum { root = 0, user } cap_t;
 
-static void test_param_ctx(const int fd, const int ctx, const cap_t cap)
+static void _check_param_ctx(const int fd, const int ctx, const cap_t cap)
 {
        const uint32_t bad = rand() + 1;
 
@@ -977,8 +973,7 @@ static void check_params(const int fd, const int ctx, cap_t cap)
        igt_assert(ioctl(fd, GET_RESET_STATS_IOCTL, 0) == -1);
        igt_assert(_test_params(fd, 0xbadbad, 0, 0) == -ENOENT);
 
-       test_param_ctx(fd, 0, cap);
-       test_param_ctx(fd, ctx, cap);
+       _check_param_ctx(fd, ctx, cap);
 }
 
 static void _test_param(const int fd, const int ctx)
@@ -988,7 +983,7 @@ static void _test_param(const int fd, const int ctx)
        igt_fork(child, 1) {
                check_params(fd, ctx, root);
 
-               drop_root();
+               igt_drop_root();
 
                check_params(fd, ctx, user);
        }
@@ -998,7 +993,7 @@ static void _test_param(const int fd, const int ctx)
        igt_waitchildren();
 }
 
-static void test_params(void)
+static void test_params_ctx(void)
 {
        int fd, ctx;
 
@@ -1011,32 +1006,103 @@ static void test_params(void)
        close(fd);
 }
 
-#define RING_HAS_CONTEXTS current_ring->contexts(current_ring)
-#define RUN_CTX_TEST(...) do { igt_skip_on(RING_HAS_CONTEXTS == false); __VA_ARGS__; } while (0)
+static void test_params(void)
+{
+       int fd;
 
-igt_main
+       fd = drm_open_any();
+       igt_assert(fd >= 0);
+
+       _test_param(fd, 0);
+
+       close(fd);
+
+}
+
+static bool gem_has_hw_contexts(int fd)
 {
        struct local_drm_i915_gem_context_create create;
-       uint32_t devid;
        int ret;
 
+       memset(&create, 0, sizeof(create));
+       ret = drmIoctl(fd, CONTEXT_CREATE_IOCTL, &create);
+
+       if (ret == 0) {
+               drmIoctl(fd, CONTEXT_DESTROY_IOCTL, &create);
+               return true;
+       }
+
+       return false;
+}
+
+static bool gem_has_reset_stats(int fd)
+{
+       struct local_drm_i915_reset_stats rs;
+       int ret;
+
+       /* Carefully set flags and pad to zero, otherwise
+          we get -EINVAL
+       */
+       memset(&rs, 0, sizeof(rs));
+
+       ret = drmIoctl(fd, GET_RESET_STATS_IOCTL, &rs);
+       if (ret == 0)
+               return true;
+
+       /* If we get EPERM, we have support but did not
+          have CAP_SYSADM */
+       if (ret == -1 && errno == EPERM)
+               return true;
+
+       return false;
+}
+
+static void check_gpu_ok(void)
+{
+       int retry_count = 30;
+       enum stop_ring_flags flags;
+       int fd;
+
+       igt_debug("checking gpu state\n");
+
+       while (retry_count--) {
+               flags = igt_get_stop_rings();
+               if (flags == 0)
+                       break;
+
+               igt_debug("waiting previous hang to clear\n");
+               sleep(1);
+       }
+
+       igt_assert(flags == 0);
+
+       fd = drm_open_any();
+       gem_quiescent_gpu(fd);
+       close(fd);
+}
+
+#define RING_HAS_CONTEXTS (current_ring->contexts(current_ring))
+#define RUN_TEST(...) do { check_gpu_ok(); __VA_ARGS__; check_gpu_ok(); } while (0)
+#define RUN_CTX_TEST(...) do { igt_skip_on(RING_HAS_CONTEXTS == false); RUN_TEST(__VA_ARGS__); } while (0)
+
+igt_main
+{
        igt_skip_on_simulation();
 
        igt_fixture {
                int fd;
+
+               bool has_reset_stats;
                fd = drm_open_any();
                devid = intel_get_drm_devid(fd);
-               igt_require_f(intel_gen(devid) >= 4,
-                             "Architecture %d too old\n", intel_gen(devid));
-
-               ret = drmIoctl(fd, CONTEXT_CREATE_IOCTL, &create);
-               igt_skip_on_f(ret != 0 && (errno == ENODEV || errno == EINVAL),
-                             "Kernel is too old, or contexts not supported: %s\n",
-                             strerror(errno));
 
-               assert(igt_debugfs_init(&dfs) == 0);
+               hw_contexts = gem_has_hw_contexts(fd);
+               has_reset_stats = gem_has_reset_stats(fd);
 
                close(fd);
+
+               igt_require_f(has_reset_stats,
+                             "No reset stats ioctl support. Too old kernel?\n");
        }
 
        igt_subtest("params")
@@ -1044,28 +1110,37 @@ igt_main
 
        for (int i = 0; i < NUM_RINGS; i++) {
                const char *name;
-               int fd;
 
                current_ring = &rings[i];
                name = current_ring->name;
 
-               fd = drm_open_any();
-               gem_require_ring(fd, current_ring->exec);
+               igt_fixture {
+                       int fd = drm_open_any();
+                       gem_require_ring(fd, current_ring->exec);
+                       close(fd);
+               }
+
+               igt_fixture
+                       igt_require_f(intel_gen(devid) >= 4,
+                                     "gen %d doesn't support reset\n", intel_gen(devid));
+
+               igt_subtest_f("params-ctx-%s", name)
+                       RUN_CTX_TEST(test_params_ctx());
 
                igt_subtest_f("reset-stats-%s", name)
-                       test_rs(4, 1, 0);
+                       RUN_TEST(test_rs(4, 1, 0));
 
                igt_subtest_f("reset-stats-ctx-%s", name)
                        RUN_CTX_TEST(test_rs_ctx(4, 4, 1, 2));
 
                igt_subtest_f("ban-%s", name)
-                       test_ban();
+                       RUN_TEST(test_ban());
 
                igt_subtest_f("ban-ctx-%s", name)
                        RUN_CTX_TEST(test_ban_ctx());
 
                igt_subtest_f("reset-count-%s", name)
-                       test_reset_count(false);
+                       RUN_TEST(test_reset_count(false));
 
                igt_subtest_f("reset-count-ctx-%s", name)
                        RUN_CTX_TEST(test_reset_count(true));
@@ -1073,19 +1148,16 @@ igt_main
                igt_subtest_f("unrelated-ctx-%s", name)
                        RUN_CTX_TEST(test_unrelated_ctx());
 
-               igt_subtest_f("close-pending-%s", name) {
-                       test_close_pending();
-                       gem_quiescent_gpu(fd);
-               }
+               igt_subtest_f("close-pending-%s", name)
+                       RUN_TEST(test_close_pending());
 
-               igt_subtest_f("close-pending-ctx-%s", name) {
+               igt_subtest_f("close-pending-ctx-%s", name)
                        RUN_CTX_TEST(test_close_pending_ctx());
-                       gem_quiescent_gpu(fd);
-               }
 
-               igt_subtest_f("close-pending-fork-%s", name) {
-                       test_close_pending_fork(true);
-                       test_close_pending_fork(false);
-               }
+               igt_subtest_f("close-pending-fork-%s", name)
+                       RUN_TEST(test_close_pending_fork(false));
+
+               igt_subtest_f("close-pending-fork-reverse-%s", name)
+                       RUN_TEST(test_close_pending_fork(true));
        }
 }