drm/i915/selftests: Use a full emulation of a user ppgtt context
authorChris Wilson <chris@chris-wilson.co.uk>
Thu, 19 Jul 2018 19:47:45 +0000 (20:47 +0100)
committerChris Wilson <chris@chris-wilson.co.uk>
Thu, 26 Jul 2018 12:32:07 +0000 (13:32 +0100)
To test eviction from a ppgtt, we just want a ppgtt i.e. something other
than the Global GTT which is shared and used by the kernel for HW
features like fencing and scanout. However, we also need it to pass
!i915_is_ggtt() and the simplest way is to emulate a full user context
rather than the internal kernel context that is used for the GGTT.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Matthew Auld <matthew.auld@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20180719194746.19111-1-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/selftests/intel_hangcheck.c

index 65d66cdedd26c6027be1881f380e27578d07518a..b2d6d15f025a24b426a06a4abe10da30433cb694 100644 (file)
@@ -1144,19 +1144,27 @@ static int igt_reset_evict_ppgtt(void *arg)
 {
        struct drm_i915_private *i915 = arg;
        struct i915_gem_context *ctx;
+       struct drm_file *file;
        int err;
 
+       file = mock_file(i915);
+       if (IS_ERR(file))
+               return PTR_ERR(file);
+
        mutex_lock(&i915->drm.struct_mutex);
-       ctx = kernel_context(i915);
+       ctx = live_context(i915, file);
        mutex_unlock(&i915->drm.struct_mutex);
-       if (IS_ERR(ctx))
-               return PTR_ERR(ctx);
+       if (IS_ERR(ctx)) {
+               err = PTR_ERR(ctx);
+               goto out;
+       }
 
        err = 0;
        if (ctx->ppgtt) /* aliasing == global gtt locking, covered above */
                err = __igt_reset_evict_vma(i915, &ctx->ppgtt->vm);
 
-       kernel_context_close(ctx);
+out:
+       mock_file_free(i915, file);
        return err;
 }