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

Krzysztof Karas krzysztof.karas at intel.com
Fri Mar 21 12:36:45 UTC 2025


Hi Mikołaj,

> +	/*
> +	 * 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.
polutes -> pollutes
I am also not sure if we should mention CI at all in kernel
code. You could rephrase the second sentence a bit to something
like this:
"The bug was found and fixed in later generations, but it
might pollute testing on older hardware, so disable it."

Treat the rephrasing as a suggestion though.

Also, as Kamil already pointed out in one of his replies, you
might want to submit this to a kernel mailing list. After that:

Reviewed-by: Krzysztof Karas <krzysztof.karas at intel.com>

Best Regards,
Krzysztof



More information about the igt-dev mailing list