[igt-dev] [PATCH i-g-t 2/2] tests/drm_fdinfo: Drop active busyness tests for GuC backend

[Umesh Nerlige Ramappa]

With GuC scheduling backends, the context busyness for an active context is
obtained from the GuC FW. The support for this is WIP, so disable tests that
check TEST_BUSY until the feature is implemented.

Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa at intel.com>
---
 tests/i915/drm_fdinfo.c | 148 +++++++++++++++++++++-------------------
 1 file changed, 79 insertions(+), 69 deletions(-)

diff --git a/tests/i915/drm_fdinfo.c b/tests/i915/drm_fdinfo.c
index 65b8b830..5df33933 100644
--- a/tests/i915/drm_fdinfo.c
+++ b/tests/i915/drm_fdinfo.c
@@ -761,83 +761,93 @@ igt_main
 	igt_subtest("virtual-idle")
 		virtual(i915, &ctx->cfg, 0);
 
-	/**
-	 * Test that a single engine reports load correctly.
-	 */
-	test_each_engine("busy", i915, ctx, e)
-		single(i915, ctx, e, TEST_BUSY);
-
-	igt_subtest("virtual-busy")
-		virtual(i915, &ctx->cfg, TEST_BUSY);
-
-	test_each_engine("busy-idle", i915, ctx, e)
-		single(i915, ctx, e, TEST_BUSY | TEST_TRAILING_IDLE);
-
-	igt_subtest("virtual-busy-idle")
-		virtual(i915, &ctx->cfg, TEST_BUSY | TEST_TRAILING_IDLE);
-
-	test_each_engine("busy-hang", i915, ctx, e) {
-		igt_hang_t hang = igt_allow_hang(i915, ctx->id, 0);
-
-		single(i915, ctx, e, TEST_BUSY | FLAG_HANG);
-
-		igt_disallow_hang(i915, hang);
-	}
-
-	igt_subtest("virtual-busy-hang")
-		virtual(i915, &ctx->cfg, TEST_BUSY | FLAG_HANG);
-
-	/**
-	 * Test that when one engine is loaded other report no
-	 * load.
-	 */
-	test_each_engine("busy-check-all", i915, ctx, e)
-		busy_check_all(i915, ctx, e, num_engines, classes, num_classes,
-			       TEST_BUSY);
+	igt_fixture {
+		/**
+		 * With GuC scheduling backends, the context busyness for a
+		 * active context is obtained from the GUC FW. The support for
+		 * this is WIP, so disable tests that check TEST_BUSY until the
+		 * feature is implemented.
+		 */
+		igt_require(!gem_using_guc_submission(i915));
 
-	test_each_engine("busy-idle-check-all", i915, ctx, e)
-		busy_check_all(i915, ctx, e, num_engines, classes, num_classes,
-			       TEST_BUSY | TEST_TRAILING_IDLE);
+		/**
+		 * Test that a single engine reports load correctly.
+		 */
+		test_each_engine("busy", i915, ctx, e)
+			single(i915, ctx, e, TEST_BUSY);
 
-	/**
-	 * Test that when all except one engine are loaded all
-	 * loads are correctly reported.
-	 */
-	test_each_engine("most-busy-check-all", i915, ctx, e)
-		most_busy_check_all(i915, ctx, e, num_engines,
-				    classes, num_classes,
-				    TEST_BUSY);
+		igt_subtest("virtual-busy")
+			virtual(i915, &ctx->cfg, TEST_BUSY);
 
-	test_each_engine("most-busy-idle-check-all", i915, ctx, e)
-		most_busy_check_all(i915, ctx, e, num_engines,
-				    classes, num_classes,
-				    TEST_BUSY | TEST_TRAILING_IDLE);
+		test_each_engine("busy-idle", i915, ctx, e)
+			single(i915, ctx, e, TEST_BUSY | TEST_TRAILING_IDLE);
 
-	/**
-	 * Test that when all engines are loaded all loads are
-	 * correctly reported.
-	 */
-	igt_subtest("all-busy-check-all")
-		all_busy_check_all(i915, ctx, num_engines, classes, num_classes,
-				   TEST_BUSY);
+		igt_subtest("virtual-busy-idle")
+			virtual(i915, &ctx->cfg, TEST_BUSY | TEST_TRAILING_IDLE);
 
-	igt_subtest("all-busy-idle-check-all")
-		all_busy_check_all(i915, ctx, num_engines, classes, num_classes,
-				   TEST_BUSY | TEST_TRAILING_IDLE);
+		test_each_engine("busy-hang", i915, ctx, e) {
+			igt_hang_t hang = igt_allow_hang(i915, ctx->id, 0);
 
-	igt_subtest("virtual-busy-all")
-		virtual_all(i915, &ctx->cfg, TEST_BUSY);
+			single(i915, ctx, e, TEST_BUSY | FLAG_HANG);
 
-	igt_subtest("virtual-busy-idle-all")
-		virtual_all(i915, &ctx->cfg, TEST_BUSY | TEST_TRAILING_IDLE);
+			igt_disallow_hang(i915, hang);
+		}
 
-	igt_subtest("virtual-busy-hang-all")
-		virtual_all(i915, &ctx->cfg, TEST_BUSY | FLAG_HANG);
-	/**
-	 * Test for no cross-client contamination.
-	 */
-	test_each_engine("isolation", i915, ctx, e)
-		single(i915, ctx, e, TEST_BUSY | TEST_ISOLATION);
+		igt_subtest("virtual-busy-hang")
+			virtual(i915, &ctx->cfg, TEST_BUSY | FLAG_HANG);
+
+		/**
+		 * Test that when one engine is loaded other report no
+		 * load.
+		 */
+		test_each_engine("busy-check-all", i915, ctx, e)
+			busy_check_all(i915, ctx, e, num_engines, classes, num_classes,
+					TEST_BUSY);
+
+		test_each_engine("busy-idle-check-all", i915, ctx, e)
+			busy_check_all(i915, ctx, e, num_engines, classes, num_classes,
+					TEST_BUSY | TEST_TRAILING_IDLE);
+
+		/**
+		 * Test that when all except one engine are loaded all
+		 * loads are correctly reported.
+		 */
+		test_each_engine("most-busy-check-all", i915, ctx, e)
+			most_busy_check_all(i915, ctx, e, num_engines,
+					classes, num_classes,
+					TEST_BUSY);
+
+		test_each_engine("most-busy-idle-check-all", i915, ctx, e)
+			most_busy_check_all(i915, ctx, e, num_engines,
+					classes, num_classes,
+					TEST_BUSY | TEST_TRAILING_IDLE);
+
+		/**
+		 * Test that when all engines are loaded all loads are
+		 * correctly reported.
+		 */
+		igt_subtest("all-busy-check-all")
+			all_busy_check_all(i915, ctx, num_engines, classes, num_classes,
+					TEST_BUSY);
+
+		igt_subtest("all-busy-idle-check-all")
+			all_busy_check_all(i915, ctx, num_engines, classes, num_classes,
+					TEST_BUSY | TEST_TRAILING_IDLE);
+
+		igt_subtest("virtual-busy-all")
+			virtual_all(i915, &ctx->cfg, TEST_BUSY);
+
+		igt_subtest("virtual-busy-idle-all")
+			virtual_all(i915, &ctx->cfg, TEST_BUSY | TEST_TRAILING_IDLE);
+
+		igt_subtest("virtual-busy-hang-all")
+			virtual_all(i915, &ctx->cfg, TEST_BUSY | FLAG_HANG);
+		/**
+		 * Test for no cross-client contamination.
+		 */
+		test_each_engine("isolation", i915, ctx, e)
+			single(i915, ctx, e, TEST_BUSY | TEST_ISOLATION);
+	}
 
 	igt_fixture {
 		intel_ctx_destroy(i915, ctx);
-- 
2.34.1