Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH i-g-t] kms_flip_tiling: reduce exec time for sim env
@ 2025-10-08 15:25 Jeevan B
  2025-10-08 19:43 ` ✓ Xe.CI.BAT: success for " Patchwork
                   ` (4 more replies)
  0 siblings, 5 replies; 10+ messages in thread
From: Jeevan B @ 2025-10-08 15:25 UTC (permalink / raw)
  To: igt-dev; +Cc: swati2.sharma, pranay.samala, Jeevan B

For simulation environment, we can reduce execution time
by running test only with basic tiling combinations.

v2: Add helper to check basic tiling support.

Signed-off-by: Jeevan B <jeevan.b@intel.com>
---
 tests/intel/kms_flip_tiling.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/tests/intel/kms_flip_tiling.c b/tests/intel/kms_flip_tiling.c
index ff9ad1229..024421b58 100644
--- a/tests/intel/kms_flip_tiling.c
+++ b/tests/intel/kms_flip_tiling.c
@@ -84,6 +84,13 @@ static int try_commit(igt_display_t *display)
 				       COMMIT_ATOMIC : COMMIT_LEGACY);
 }
 
+static bool is_allowed_modifier(uint64_t mod)
+{
+	return mod == DRM_FORMAT_MOD_LINEAR ||
+	       mod == I915_FORMAT_MOD_4_TILED ||
+	       mod == I915_FORMAT_MOD_X_TILED;
+}
+
 static uint64_t pageflip_timeout_us(drmModeModeInfo *mode)
 {
 	uint64_t timeout_ns;
@@ -216,6 +223,7 @@ igt_main
 	igt_describe("Check pageflip between modifiers");
 	igt_subtest_with_dynamic("flip-change-tiling") {
 		enum pipe pipe;
+		bool run_in_simulation = igt_run_in_simulation();
 
 		for_each_pipe_with_valid_output(&data.display, pipe, output) {
 			igt_plane_t *plane;
@@ -242,6 +250,11 @@ igt_main
 					if (plane->formats[j] != data.testformat)
 						continue;
 
+					if (run_in_simulation &&
+					    (!is_allowed_modifier(plane->modifiers[i]) ||
+					     !is_allowed_modifier(plane->modifiers[j])))
+						continue;
+
 					igt_dynamic_f("pipe-%s-%s-%s-to-%s",
 						      kmstest_pipe_name(pipe),
 						      igt_output_name(output),
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 10+ messages in thread
* [PATCH i-g-t] kms_flip_tiling: reduce exec time for sim env
@ 2025-10-09  6:29 Jeevan B
  2025-10-09 16:25 ` Kamil Konieczny
  0 siblings, 1 reply; 10+ messages in thread
From: Jeevan B @ 2025-10-09  6:29 UTC (permalink / raw)
  To: igt-dev; +Cc: swati2.sharma, pranay.samala, Jeevan B

For simulation environment, we can reduce execution time
by running test only with basic tiling combinations.

v2: Add helper to check basic tiling support.
v3: Rename helper to is_basic_tiling_modifier() and add comment.

Signed-off-by: Jeevan B <jeevan.b@intel.com>
---
 tests/intel/kms_flip_tiling.c | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/tests/intel/kms_flip_tiling.c b/tests/intel/kms_flip_tiling.c
index ff9ad1229..d078653e9 100644
--- a/tests/intel/kms_flip_tiling.c
+++ b/tests/intel/kms_flip_tiling.c
@@ -84,6 +84,16 @@ static int try_commit(igt_display_t *display)
 				       COMMIT_ATOMIC : COMMIT_LEGACY);
 }
 
+/* This helper is used to restrict tests to commonly supported tiling modes
+ * across platforms for simulation.
+ */
+static bool is_basic_tiling_modifier(uint64_t mod)
+{
+	return mod == DRM_FORMAT_MOD_LINEAR ||
+	       mod == I915_FORMAT_MOD_4_TILED ||
+	       mod == I915_FORMAT_MOD_X_TILED;
+}
+
 static uint64_t pageflip_timeout_us(drmModeModeInfo *mode)
 {
 	uint64_t timeout_ns;
@@ -216,6 +226,7 @@ igt_main
 	igt_describe("Check pageflip between modifiers");
 	igt_subtest_with_dynamic("flip-change-tiling") {
 		enum pipe pipe;
+		bool run_in_simulation = igt_run_in_simulation();
 
 		for_each_pipe_with_valid_output(&data.display, pipe, output) {
 			igt_plane_t *plane;
@@ -242,6 +253,11 @@ igt_main
 					if (plane->formats[j] != data.testformat)
 						continue;
 
+					if (run_in_simulation &&
+					    (!is_basic_tiling_modifier(plane->modifiers[i]) ||
+					     !is_basic_tiling_modifier(plane->modifiers[j])))
+						continue;
+
 					igt_dynamic_f("pipe-%s-%s-%s-to-%s",
 						      kmstest_pipe_name(pipe),
 						      igt_output_name(output),
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 10+ messages in thread
end of thread, other threads:[~2025-10-14 19:29 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-10-08 15:25 [PATCH i-g-t] kms_flip_tiling: reduce exec time for sim env Jeevan B
2025-10-08 19:43 ` ✓ Xe.CI.BAT: success for " Patchwork
2025-10-08 19:57 ` [PATCH i-g-t] " Ville Syrjälä
2025-10-09  3:17   ` B, Jeevan
2025-10-08 20:03 ` ✓ i915.CI.BAT: success for " Patchwork
2025-10-08 22:16 ` ✗ Xe.CI.Full: failure " Patchwork
2025-10-09  9:48 ` ✓ i915.CI.Full: success " Patchwork
  -- strict thread matches above, loose matches on Subject: below --
2025-10-09  6:29 [PATCH i-g-t] " Jeevan B
2025-10-09 16:25 ` Kamil Konieczny
2025-10-14 19:29   ` Sharma, Swati2

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox