[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