[PATCH i-g-t v2 1/2] tests/intel/xe_wedged: Ignore more dmesg warnings

[Cavitt, Jonathan]

-----Original Message-----
From: Kamil Konieczny <kamil.konieczny at linux.intel.com> 
Sent: Tuesday, October 29, 2024 11:23 AM
To: igt-dev at lists.freedesktop.org
Cc: Kamil Konieczny <kamil.konieczny at linux.intel.com>; Cavitt, Jonathan <jonathan.cavitt at intel.com>; Vivi, Rodrigo <rodrigo.vivi at intel.com>; Kempczynski, Zbigniew <zbigniew.kempczynski at intel.com>
Subject: [PATCH i-g-t v2 1/2] tests/intel/xe_wedged: Ignore more dmesg warnings
> 
> There are more warnings in dmesg which should be ignored,
> one comes from driver, for example:
> 
> xe 0000:00:02.0: [drm] *ERROR* GT0: reset failed (-ECANCELED)
> xe 0000:00:02.0: [drm] *ERROR* GT1: Failed to submit gsc proxy rq (-ETIME)
> 
> or from drm:
> 
> <4> [438.651441]  ? __pfx___drm_printfn_seq_file+0x10/0x10 [drm]
> <4> [438.651476]  ? __pfx___drm_puts_seq_file+0x10/0x10 [drm]
> 
> or from kernel itself:
> 
> <4> [226.597358] Modules linked in:
> 
> so extend regex with them.
> 
> v2: move '|' to begin of strings for better reading (Jonathan)
>   extend regex with two more, correct __pfx___ regex (Kamil)
>   added r-b and correcting name in Cc
> 
> Closes: https://gitlab.freedesktop.org/drm/xe/kernel/-/issues/2919
> Cc: Jonathan Cavitt <jonathan.cavitt at intel.com>
> Cc: Rodrigo Vivi <rodrigo.vivi at intel.com>
> Cc: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
> Signed-off-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> Reviewed-by: Jonathan Cavitt <jonathan.cavitt at intel.com>
> ---
>  tests/intel/xe_wedged.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/intel/xe_wedged.c b/tests/intel/xe_wedged.c
> index 88e5d47f2..123ce6396 100644
> --- a/tests/intel/xe_wedged.c
> +++ b/tests/intel/xe_wedged.c
> @@ -29,7 +29,11 @@
>  static void ignore_wedged_in_dmesg(void)
>  {
>  	/* this is needed for igt_runner so it will ignore it */
> -	igt_emit_ignore_dmesg_regex("CRITICAL: Xe has declared device [0-9A-Fa-f:.]* as wedged");
> +	igt_emit_ignore_dmesg_regex("CRITICAL: Xe has declared device [0-9A-Fa-f:.]* as wedged"
> +				    "|GT[0-9A-Fa-f]*: reset failed .-ECANCELED"
> +				    "|GT[0-9A-Fa-f]*: Failed to submit"
> +				    "|Modules linked in:"
> +				    "|__pfx___drm_.*drm");

It's starting to get a little bit silly how long this ignore list is getting, but if it's necessary,
it's necessary.

My Reviewed-by still stands.
-Jonathan Cavitt

>  }
>  
>  static void force_wedged(int fd)
> -- 
> 2.47.0
> 
>