RE: [PATCH i-g-t] tests/intel/xe_exec_threads: ignore expected hang timeout

"Dandamudi, Priyanka" <[email protected]> Thu, 23 Jul 2026 04:04:37 +0000
Newsgroups org.freedesktop.lists.igt-dev
Message-ID <CH3PR11MB84342F8F30C8B9BF556DCCDF8DC02@CH3PR11MB8434.namprd11.prod.outlook.com>

> -----Original Message-----
> From: Girotra, Himanshu <[email protected]>
> Sent: 23 July 2026 08:03 AM
> To: [email protected]
> Cc: Gandi, Ramadevi <[email protected]>; Dandamudi, Priyanka
> <[email protected]>; Konieczny, Kamil
> <[email protected]>
> Subject: [PATCH i-g-t] tests/intel/xe_exec_threads: ignore expected hang
> timeout
> 
> From: Himanshu Girotra <[email protected]>
> 
> The hang subtests hang a queue and reset to recover. Jobs that were queued
> but never started are logged by the KMD at warn level as "Check job timeout:
> ... not started".
> 
> This warn is expected, but CI flags warn-level dmesg lines and fails the subtest
> even though it passes. Whitelist it for the hang subtests.
> 
> v2: Rework description and comment to be generic (Priyanka,Kamil)
> 
> Signed-off-by: Himanshu Girotra <[email protected]>
> ---
>  tests/intel/xe_exec_threads.c | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/tests/intel/xe_exec_threads.c b/tests/intel/xe_exec_threads.c index
> 0480df779..2974fc312 100644
> --- a/tests/intel/xe_exec_threads.c
> +++ b/tests/intel/xe_exec_threads.c
> @@ -1468,6 +1468,16 @@ static void threads(int fd, int flags)
>  	pthread_barrier_destroy(&barrier);
>  }
> 
> +/*
> + * Hang subtests reset the GPU to recover, so jobs that never started
> +are
> + * logged as "Check job timeout: ... not started". This warn is
> +expected;
> + * whitelist it to avoid spurious dmesg-warn failures in CI.
> + */
> +static void ignore_multi_queue_hang_warnings_in_dmesg(void)
> +{
> +	igt_emit_ignore_dmesg_regex("Check job timeout: .*not started"); }
> +
>  int igt_main()
>  {
>  	const struct section {
> @@ -1735,6 +1745,9 @@ int igt_main()
>  				igt_assert(!(s->flags & BALANCER));
>  			}
> 
> +			if ((s->flags & MULTI_QUEUE) && (s->flags & HANG))
> +
> 	ignore_multi_queue_hang_warnings_in_dmesg();
> +
>  			threads(fd, s->flags);
LGTM,
Reviewed-by: Priyanka Dandamudi <[email protected]>
>  		}
>  	}
> --
> 2.50.1