Re: [PATCH] tests/kms_vrr: Avoid race between freeing display reseources

[Maarten Lankhorst <maarten.lankhorst@linux.intel.com>]

Hey,

The sleep seems a workaround.

Can't you just do a blocking commit to disable everything instead?
igt_display_reset()
igt_display_commit2() before igt_display_fini

Den 2024-08-31 kl. 07:58, skrev Mitul Golani:
> 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);
>  	}
>  }