Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v1] tests/kms_flip: Assert with twice of frame time while waiting fro vblank
@ 2024-10-11  6:21 Mitul Golani
  2024-10-11  9:56 ` Ville Syrjälä
                   ` (4 more replies)
  0 siblings, 5 replies; 7+ messages in thread
From: Mitul Golani @ 2024-10-11  6:21 UTC (permalink / raw)
  To: igt-dev; +Cc: mitulkumar.ajitkumar.golani, bhanuprakash.modem, ankit.k.nautiyal

Instead of hardcoding to 500us, wait for twice of mode frame time while
waiting for vblank. Motivation of this change is to remove hard coding
of 500us, which comes out to be more stringent and ambiguous vs changing
modeline.

Signed-off-by: Mitul Golani <mitulkumar.ajitkumar.golani@intel.com>
---
 tests/kms_flip.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/kms_flip.c b/tests/kms_flip.c
index cbabbe74f..decffa961 100755
--- a/tests/kms_flip.c
+++ b/tests/kms_flip.c
@@ -921,7 +921,7 @@ static bool run_test_step(struct test_output *o, unsigned int *events)
 					    0, &reply));
 		end = gettime_us();
 		igt_debug("Vblank took %luus\n", end - start);
-		igt_assert(end - start < 500);
+		igt_assert(end - start < 2 * mode_frame_time(o));
 		if (reply.sequence != exp_seq) {
 			igt_debug("unexpected vblank seq %u, should be %u\n",
 				  reply.sequence, exp_seq);
-- 
2.46.0


^ permalink raw reply related	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2024-10-14  5:41 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-11  6:21 [PATCH v1] tests/kms_flip: Assert with twice of frame time while waiting fro vblank Mitul Golani
2024-10-11  9:56 ` Ville Syrjälä
2024-10-14  5:41   ` Golani, Mitulkumar Ajitkumar
2024-10-11 15:03 ` ✓ Fi.CI.BAT: success for " Patchwork
2024-10-11 15:44 ` ✓ CI.xeBAT: " Patchwork
2024-10-11 19:14 ` ✗ CI.xeFULL: failure " Patchwork
2024-10-12  8:18 ` ✗ Fi.CI.IGT: " Patchwork

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