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

[Kamil Konieczny]

Hi Jonathan,
On 2024-10-29 at 19:15:04 +0000, Cavitt, Jonathan wrote:
> -----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
> 

Hmm, I see there are some problems with verification for Xe driver
after this test, after test passes or skips all others starts skip,
for example:

https://intel-gfx-ci.01.org/tree/intel-xe/IGTPW_11997/bat-pvc-2/igt_runner0.txt

https://intel-gfx-ci.01.org/tree/intel-xe/IGTPW_11997/bat-pvc-2/igt@xe_wedged@wedged-at-any-timeout.html

This needs more work so please do not merge it.

Regards,
Kamil

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