[PATCH i-g-t 9/9] tests/i915/gen9_exec_parse: Expect sync_fence_status to succeed

Jason Ekstrand jason at jlekstrand.net
Tue Jul 13 18:02:47 UTC 2021


i915 is switching to synchronously parse command buffers and deliver the
error immediately instead of trying to propagate it to the fence.  We
can assume that, if there is an error, it's returned from execbuf.

Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
---
 tests/i915/gen9_exec_parse.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/tests/i915/gen9_exec_parse.c b/tests/i915/gen9_exec_parse.c
index b35f2cb43..512891873 100644
--- a/tests/i915/gen9_exec_parse.c
+++ b/tests/i915/gen9_exec_parse.c
@@ -72,10 +72,8 @@ __checked_execbuf(int i915, struct drm_i915_gem_execbuffer2 *eb)
 	fence = eb->rsvd2 >> 32;
 
 	sync_fence_wait(fence, -1);
-	err = sync_fence_status(fence);
+	igt_assert(sync_fence_status(fence) >= 0);
 	close(fence);
-	if (err < 0)
-		return err;
 
 	return 0;
 }
-- 
2.31.1



More information about the Intel-gfx-trybot mailing list