[Intel-gfx] [PATCH igt] Revert "kms_cursor_legacy: Add a burner thread to make basic-busy-* pass."

Martin Peres martin.peres at linux.intel.com
Wed Jun 7 11:05:14 UTC 2017


On 07/06/17 13:34, Chris Wilson wrote:
> This reverts commit 7c8703fb02b248c2bcf9756bba8812bcfe7ed5d3.
> 
> If we expect it to fail until we find a solution, let the hw fail and
> continue to track the known failure in CI/bugs.
> 
> Cc: Martin Peres <martin.peres at linux.intel.com>

I agree, the kernel/HW is broken.

However, this fix is not necessarily wrong, as in, the real kernel 
fix/workaround may actually be worse than the problem it is trying to 
fix, because it would increase the power usage while only fixing a use 
case that has not been seen in the wild (or has it?).

We thus need to prove that this is the case and document this issue 
extensively in both IGT and the kernel. Until then, let's merge the 
revert to keep it in the task list:

Acked-by: Martin Peres <martin.peres at linux.intel.com>

Thanks Chris,
Martin


More information about the Intel-gfx mailing list