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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxxxxx>

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@xxxxxxxxxxxxxxx>

Thanks Chris,
Martin
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux