[PATCH V2 i-g-t] tests/xe_exec_threads: Make hang tests reset domain aware
Tejas Upadhyay
tejas.upadhyay at intel.com
Tue Apr 2 12:22:23 UTC 2024
RCS/CCS are dependent engines as they are sharing reset
domain. Whenever there is reset from CCS, all the exec queues
running on RCS are victimised mainly on Lunarlake.
Lets skip parallel execution on CCS with RCS.
It helps in fixing following errors:
1. Test assertion failure function test_legacy_mode, file, Failed assertion: data[i].data == 0xc0ffee
2.Test assertion failure function xe_exec, file ../lib/xe/xe_ioctl.c, Failed assertion: __xe_exec(fd, exec) == 0, error: -125 != 0
Signed-off-by: Tejas Upadhyay <tejas.upadhyay at intel.com>
---
tests/intel/xe_exec_threads.c | 26 +++++++++++++++++++++++++-
1 file changed, 25 insertions(+), 1 deletion(-)
diff --git a/tests/intel/xe_exec_threads.c b/tests/intel/xe_exec_threads.c
index 8083980f9..31af61dc9 100644
--- a/tests/intel/xe_exec_threads.c
+++ b/tests/intel/xe_exec_threads.c
@@ -710,6 +710,17 @@ static void *thread(void *data)
return NULL;
}
+static bool is_engine_contexts_victimized(int fd, unsigned int flags)
+{
+ if (!IS_LUNARLAKE(intel_get_drm_devid(fd)))
+ return false;
+
+ if (flags & HANG)
+ return true;
+
+ return false;
+}
+
/**
* SUBTEST: threads-%s
* Description: Run threads %arg[1] test with multi threads
@@ -955,9 +966,13 @@ static void threads(int fd, int flags)
bool go = false;
int n_threads = 0;
int gt;
+ bool has_rcs = false;
- xe_for_each_engine(fd, hwe)
+ xe_for_each_engine(fd, hwe) {
+ if (hwe->engine_class == DRM_XE_ENGINE_CLASS_RENDER)
+ has_rcs = true;
++n_engines;
+ }
if (flags & BALANCER) {
xe_for_each_gt(fd, gt)
@@ -990,6 +1005,15 @@ static void threads(int fd, int flags)
}
xe_for_each_engine(fd, hwe) {
+ /* RCS/CCS sharing reset domain hence dependent engines.
+ * When CCS is doing reset, all the contexts of RCS are
+ * victimized, so skip the compute engine avoiding
+ * parallel execution with RCS
+ */
+ if (has_rcs && hwe->engine_class == DRM_XE_ENGINE_CLASS_COMPUTE &&
+ is_engine_contexts_victimized(fd, flags))
+ continue;
+
threads_data[i].mutex = &mutex;
threads_data[i].cond = &cond;
#define ADDRESS_SHIFT 39
--
2.25.1
More information about the igt-dev
mailing list