Re: [igt-dev] [PATCH] [i-g-t] tests/amdgpu/amd_hotplug: add delay after igt_system_suspend_autoresume

[Rodrigo Siqueira Jordao <Rodrigo.Siqueira@amd.com>]

On 4/24/23 07:48, Hersen Wu wrote:
> at call back of igt_system_suspend_autoresume, linux kernel resume
> finish and in d0 state. but user mode components may not be fully
> out of sleep mode. add delay wait for system stable before issuing
> debugfs_hotplug_trigger.
> 
> Signed-off-by: Hersen Wu <hersenxs.wu@amd.com>
> ---
>   tests/amdgpu/amd_hotplug.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/tests/amdgpu/amd_hotplug.c b/tests/amdgpu/amd_hotplug.c
> index 736040a2..1c946ff8 100644
> --- a/tests/amdgpu/amd_hotplug.c
> +++ b/tests/amdgpu/amd_hotplug.c
> @@ -133,6 +133,7 @@ static void test_hotplug_basic(data_t *data, bool suspend)
>   	if (suspend) {
>   		igt_system_suspend_autoresume(SUSPEND_STATE_MEM,
>   					      SUSPEND_TEST_NONE);
> +		sleep(30);

Do we need to wait all of this time? If so, why?

>   	}
>   
>   	/* Trigger hotplug and confirm reference image is the same. */