[PATCH i-g-t 2/2] tests/intel/xe_fault_injection: Ignore faults for *guc*

[K V P, Satyanarayana]

> -----Original Message-----
> From: igt-dev <igt-dev-bounces at lists.freedesktop.org> On Behalf Of Dong,
> Zhanjun
> Sent: Monday, May 5, 2025 8:29 PM
> To: igt-dev at lists.freedesktop.org
> Subject: Re: [PATCH i-g-t 2/2] tests/intel/xe_fault_injection: Ignore faults for
> *guc*
> 
> On 2025-04-30 3:38 a.m., Satyanarayana K V P wrote:
> > Add some more guc fault messages in the dmesg ignore list.
> >
> > Signed-off-by: Satyanarayana K V P <satyanarayana.k.v.p at intel.com>
> > ---
> > Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>
> > ---
> >   tests/intel/xe_fault_injection.c | 2 ++
> >   1 file changed, 2 insertions(+)
> >
> > diff --git a/tests/intel/xe_fault_injection.c b/tests/intel/xe_fault_injection.c
> > index 82d730a7c..f293ac8af 100644
> > --- a/tests/intel/xe_fault_injection.c
> > +++ b/tests/intel/xe_fault_injection.c
> > @@ -83,6 +83,8 @@ static void ignore_faults_in_dmesg(const char
> function_name[])
> >   	if (function_is_part_of_guc(function_name)) {
> >   		strcat(regex, "|GT[0-9a-fA-F]*: GuC init failed with -
> ENOMEM");
> >   		strcat(regex, "|GT[0-9a-fA-F]*: Failed to initialize uC .-
> ENOMEM");
> > +		strcat(regex, "|GT[0-9a-fA-F]*: Failed to enable GuC CT	(-
> ENOMEM)");
> '(' and ')' is special character for regular expression. The similar
> situation applies to one line above, so it could be changed to:
> strcat(regex, "|GT[0-9a-fA-F]*: Failed to enable GuC CT	.-ENOMEM");
> 
Fixed and sent new patch.
> And, for two lines above, is this test might trigger these 2 messages?
> If not, we might want to remove it from ignore list.
> 
Yes. The test injects faults at various levels and it is possible to get all the errors listed
here.
-Satya.
> Regards,
> Zhanjun Dong> +		strcat(regex, "|GT[0-9a-fA-F]*: GuC PC query
> task state
> failed:	-ENOMEM");
> >   	}
> >
> >   	igt_emit_ignore_dmesg_regex(regex);