[PATCH i-g-t] tests/intel/xe_fault_injection: Ignore enable activity stats error

[Cavitt, Jonathan]

-----Original Message-----
From: K V P, Satyanarayana <satyanarayana.k.v.p at intel.com> 
Sent: Wednesday, May 28, 2025 6:31 AM
To: igt-dev at lists.freedesktop.org
Cc: K V P, Satyanarayana <satyanarayana.k.v.p at intel.com>; Cavitt, Jonathan <jonathan.cavitt at intel.com>; Harrison, John C <john.c.harrison at intel.com>
Subject: [PATCH i-g-t] tests/intel/xe_fault_injection: Ignore enable activity stats error
> 
> Add some more GuC fault messages in the dmesg ignore list.

I think it would be prudent to explain why we're adding this message to the
dmesg ignore list.  Perhaps:

"""
Enable activity stats errors are expected to occur during GuC fault injection
tests, so add them to the dmesg ignore list.
"""

I won't block on changing it, though, so:
Reviewed-by: Jonathan Cavitt <jonathan.cavitt at intel.com>
-Jonathan Cavitt

> 
> Signed-off-by: Satyanarayana K V P <satyanarayana.k.v.p at intel.com>
> ---
> Cc: Jonathan Cavitt <jonathan.cavitt at intel.com>
> Cc: John Harrison <John.C.Harrison at Intel.com>
> ---
>  tests/intel/xe_fault_injection.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/tests/intel/xe_fault_injection.c b/tests/intel/xe_fault_injection.c
> index f9bd5c761..4ca8a34bf 100644
> --- a/tests/intel/xe_fault_injection.c
> +++ b/tests/intel/xe_fault_injection.c
> @@ -85,6 +85,7 @@ static void ignore_faults_in_dmesg(const char function_name[])
>  		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");
>  		strcat(regex, "|GT[0-9a-fA-F]*: GuC PC query task state failed:	-ENOMEM");
> +		strcat(regex, "|GT[0-9a-fA-F]*: failed to enable activity stats-[0-9]*");
>  	}
>  
>  	igt_emit_ignore_dmesg_regex(regex);
> -- 
> 2.43.0
> 
>