[igt-dev] [PATCH i-g-t] intel/xe_exec_store.c: Fix the check for num of placements in store_all.

sai.gowtham.ch at intel.com sai.gowtham.ch at intel.com
Wed Nov 29 10:26:28 UTC 2023


From: Sai Gowtham Ch <sai.gowtham.ch at intel.com>

Adjust the store_all execution with dynamic subtest, so that test will
skip the engines with no instances and executes on the engines with instances.

Signed-off-by: Sai Gowtham Ch <sai.gowtham.ch at intel.com>
---
 tests/intel/xe_exec_store.c | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/tests/intel/xe_exec_store.c b/tests/intel/xe_exec_store.c
index 9c14bfd14..95c678bf7 100644
--- a/tests/intel/xe_exec_store.c
+++ b/tests/intel/xe_exec_store.c
@@ -245,7 +245,8 @@ static void store_all(int fd, int gt, int class)
 		eci[num_placements++] = *hwe;
 	}
 
-	igt_require(num_placements);
+	igt_skip_on_f(!num_placements, "Engine class:%d gt:%d not enabled on this platform\n",
+			class, gt);
 
 	for (i = 0; i < num_placements; i++) {
 		struct drm_xe_exec_queue_create create = {
@@ -304,10 +305,13 @@ igt_main
 	igt_subtest("basic-store")
 		store(fd);
 
-	igt_subtest("basic-all") {
+	igt_subtest_with_dynamic("basic-all") {
 		xe_for_each_gt(fd, gt)
-			xe_for_each_hw_engine_class(class)
-				store_all(fd, gt, class);
+			xe_for_each_hw_engine_class(class) {
+				igt_dynamic_f("class-%d", class) {
+					store_all(fd, gt, class);
+				}
+			}
 	}
 
 	igt_subtest("cachelines")
-- 
2.39.1



More information about the igt-dev mailing list