Re: [PATCH i-g-t] i915/gem_exec_fence: Use default-context for syncobj-timeline-wait

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 07/08/2020 11:45, Chris Wilson wrote:
syncobj-timeline-wait is only exercising waits on the syncobj along a
single timeline, for which it can use the universally available default
context, and so not try and fail to create a HW context on very old
platforms.

Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/2308
Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
Cc: Lionel Landwerlin <lionel.g.landwerlin@xxxxxxxxx>


Hm... I thought maybe we didn't get all the engines on the default context.

If that's not the case :


Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@xxxxxxxxx>


Thanks!


-Lionel


---
  tests/i915/gem_exec_fence.c | 6 +-----
  1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/tests/i915/gem_exec_fence.c b/tests/i915/gem_exec_fence.c
index c9b95fe64..b240c30bf 100644
--- a/tests/i915/gem_exec_fence.c
+++ b/tests/i915/gem_exec_fence.c
@@ -1970,7 +1970,6 @@ static void test_syncobj_timeline_wait(int fd)
  		MI_BATCH_BUFFER_END,
  		MI_NOOP,
  	};
-	uint32_t gem_context = gem_context_clone_with_engines(fd, 0);
  	struct drm_i915_gem_exec_object2 obj;
  	struct drm_i915_gem_execbuffer2 execbuf;
  	struct drm_i915_gem_execbuffer_ext_timeline_fences timeline_fences;
@@ -1988,7 +1987,7 @@ static void test_syncobj_timeline_wait(int fd)
gem_quiescent_gpu(fd); - spin = igt_spin_new(fd, .ctx = gem_context, .engine = ALL_ENGINES);
+	spin = igt_spin_new(fd, .engine = ALL_ENGINES);
memset(&timeline_fences, 0, sizeof(timeline_fences));
  	timeline_fences.base.name = DRM_I915_GEM_EXECBUFFER_EXT_TIMELINE_FENCES;
@@ -2000,7 +1999,6 @@ static void test_syncobj_timeline_wait(int fd)
  	execbuf.buffers_ptr = to_user_pointer(&obj);
  	execbuf.buffer_count = 1;
  	execbuf.batch_len = sizeof(bbe);
-	execbuf.rsvd1 = gem_context;
memset(&obj, 0, sizeof(obj));
  	obj.handle = gem_create(fd, 4096);
@@ -2055,8 +2053,6 @@ static void test_syncobj_timeline_wait(int fd)
  		gem_sync(fd, handle[i]);
  		gem_close(fd, handle[i]);
  	}
-
-	gem_context_destroy(fd, gem_context);
  }
static const char *test_syncobj_timeline_export_desc =


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



[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux