Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] tests/kms_vrr: Avoid race between freeing display reseources
@ 2024-09-10  5:29 Jeevan B
  2024-09-12  1:02 ` ✓ CI.xeBAT: success for tests/kms_vrr: Avoid race between freeing display reseources (rev2) Patchwork
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Jeevan B @ 2024-09-10  5:29 UTC (permalink / raw)
  To: igt-dev; +Cc: mitulkumar.ajitkumar.golani, bhanuprakash.modem, Jeevan B

While freeing display resources, that schedules driver to execute
panel power off, which races while scheduling next immediate test.
giving time after freeing resources allows driver to avoid race
between next non blocking atomic commit and panel power off from
previous test.

Signed-off-by: Jeevan B <jeevan.b@intel.com>
---
 tests/kms_vrr.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/kms_vrr.c b/tests/kms_vrr.c
index 36a22eebe..3c6e06a7f 100644
--- a/tests/kms_vrr.c
+++ b/tests/kms_vrr.c
@@ -410,7 +410,7 @@ do_flip(data_t *data, igt_fb_t *fb)
 {
 	int ret;
 
-	igt_set_timeout(1, "Scheduling page flip\n");
+	igt_set_timeout(2, "Scheduling page flip\n");
 
 	igt_plane_set_fb(data->primary, fb);
 
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 6+ messages in thread
* [PATCH] tests/kms_vrr: Avoid race between freeing display reseources
@ 2024-08-31  5:58 Mitul Golani
  2024-08-31  8:39 ` Maarten Lankhorst
  0 siblings, 1 reply; 6+ messages in thread
From: Mitul Golani @ 2024-08-31  5:58 UTC (permalink / raw)
  To: igt-dev; +Cc: bhanuprakash.modem

While freeing display resources, that schedules driver to execute
panel power off, which races while scheduling next immediate test.
Giving time after freeing resources allows driver to avoid race
between next non blocking atomic commit and panel power off from
previous test.
---
 tests/kms_vrr.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tests/kms_vrr.c b/tests/kms_vrr.c
index 36a22eebe..481a4253f 100644
--- a/tests/kms_vrr.c
+++ b/tests/kms_vrr.c
@@ -1110,5 +1110,7 @@ igt_main_args("drs:", long_opts, help_str, opt_handler, &data)
 	igt_fixture {
 		igt_display_fini(&data.display);
 		drm_close_driver(data.drm_fd);
+		//Wait for 2 seconds to free-up acquired display resources
+		sleep(2);
 	}
 }
-- 
2.46.0


^ permalink raw reply related	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2024-09-12  3:52 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-09-10  5:29 [PATCH] tests/kms_vrr: Avoid race between freeing display reseources Jeevan B
2024-09-12  1:02 ` ✓ CI.xeBAT: success for tests/kms_vrr: Avoid race between freeing display reseources (rev2) Patchwork
2024-09-12  1:18 ` ✗ Fi.CI.BAT: failure " Patchwork
2024-09-12  3:52 ` ✗ CI.xeFULL: " Patchwork
  -- strict thread matches above, loose matches on Subject: below --
2024-08-31  5:58 [PATCH] tests/kms_vrr: Avoid race between freeing display reseources Mitul Golani
2024-08-31  8:39 ` Maarten Lankhorst

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