[PATCH v2] i915/gt/selftest_lrc: Disable timestamp test

[Kamil Konieczny]

Hi Mikolaj,
On 2025-03-07 at 11:21:25 +0100, Mikolaj Wasiak wrote:

you sent this to i-g-t only, I guess you wanted to send this for
drm-i915 mailing list.

Regards,
Kamil

> This test was supposed to show bug in tigerlake and dg2 hardware.
> The bug was found and fixed in newer generations.
> Since we won't support any new hardware with this driver, the test
> should now be turned off in the CI to not polute it with random failures
> on previous hardware.
> 
> Signed-off-by: Mikolaj Wasiak <mikolaj.wasiak at intel.com>
> ---
> 
> Fixes: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13697
> 
> v1 -> v2: Disabled the test instead of removing it,
> 					Change name from 'remove' to 'disable'
> 
>  drivers/gpu/drm/i915/gt/selftest_lrc.c | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/gt/selftest_lrc.c b/drivers/gpu/drm/i915/gt/selftest_lrc.c
> index 22e750108c5f..419a7cd61b65 100644
> --- a/drivers/gpu/drm/i915/gt/selftest_lrc.c
> +++ b/drivers/gpu/drm/i915/gt/selftest_lrc.c
> @@ -3,6 +3,7 @@
>   * Copyright © 2018 Intel Corporation
>   */
>  
> +#include "linux/kconfig.h"
>  #include <linux/prime_numbers.h>
>  
>  #include "gem/i915_gem_internal.h"
> @@ -858,6 +859,16 @@ static int live_lrc_timestamp(void *arg)
>  		U32_MAX,
>  	};
>  
> +	/*
> +	 * This test was created to show existence of hardware bug.
> +	 * The bug was found and fixed in further generations but
> +	 * now this test polutes our CI on previous generations.
> +	 *
> +	 * https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13697
> +	 */
> +	if (!IS_ENABLED(CONFIG_DRM_I915_SELFTEST_BROKEN))
> +		return 0;
> +
>  	/*
>  	 * We want to verify that the timestamp is saved and restore across
>  	 * context switches and is monotonic.
> -- 
> 2.48.1
>