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

Hi Himanshu,
On 2026-07-23 at 08:02:51 +0530, [email protected] wrote:
> 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.

s/Whitelist/Ignore/

> 
> 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.

s/whitelist/ignore/

> + */
> +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))

Why not simpler:
			if (s->flags & (MULTI_QUEUE | HANG))



Regards,
Kamil

> +				ignore_multi_queue_hang_warnings_in_dmesg();
> +
>  			threads(fd, s->flags);
>  		}
>  	}
> -- 
> 2.50.1
>