[igt-dev] [PATCH i-g-t] i915/gem_busy: Prevent context ban with right ctx id

Chuansheng Liu chuansheng.liu at intel.com
Mon Jan 24 02:38:33 UTC 2022


When running multiple hang tests, the code is intended to
disallow context ban at first, but it is always with fix
ctx id 0. It will cause below test failures when running
the test gem_busy:

--
tarting dynamic subtest: vcs0
(gem_busy:18571) ioctl_wrappers-CRITICAL: Test assertion failure
function gem_execbuf_wr, file ../lib/ioctl_wrappers.c:706:
(gem_busy:18571) ioctl_wrappers-CRITICAL: Failed assertion:
__gem_execbuf_wr(fd, execbuf) == 0
(gem_busy:18571) ioctl_wrappers-CRITICAL: error: -5 != 0
--

-5 (EIO) means the context is banned and can not be used for
execution.

This patch corrects the ctx id accordindly. Furthermore,
there are other code places with similar mistakes, will
try to send out the fixes further.

Signed-off-by: Chuansheng Liu <chuansheng.liu at intel.com>
---
 tests/i915/gem_busy.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/i915/gem_busy.c b/tests/i915/gem_busy.c
index b4757736..b8a61084 100644
--- a/tests/i915/gem_busy.c
+++ b/tests/i915/gem_busy.c
@@ -517,7 +517,7 @@ igt_main
 		igt_hang_t hang;
 
 		igt_fixture {
-			hang = igt_allow_hang(fd, 0, 0);
+			hang = igt_allow_hang(fd, ctx->id, 0);
 		}
 
 		test_each_engine("hang", fd, ctx, e) {
-- 
2.25.0.rc2



More information about the igt-dev mailing list