Re: [PATCH] drm/i915: Remove support for legacy debugfs crc interface

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

 



Op 02-11-17 om 17:11 schreef Ville Syrjälä:
> On Thu, Nov 02, 2017 at 05:19:07PM +0200, Jani Nikula wrote:
>> On Thu, 02 Nov 2017, Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> wrote:
>>> On Thu, Nov 02, 2017 at 02:56:51PM +0100, Maarten Lankhorst wrote:
>>>> This interface is deprecated, and has been replaced by the upstream
>>>> drm crc interface.
>>> Before we nuke this I would like to see an option in the new interface
>>> to not filter out the "bad" CRCs. When analyzing how the hardware
>>> behaves seeing every CRC can be valuable. And I'm not at all convinced
>>> we should be dropping as many CRCs as we are currently.
>> I'm not against it, but do you have a concrete proposal on how that
>> option would look like?
> Some kind of of filter_bad_crcs file with a bool value perhaps?

You can set sources, might as well add a nofilter option.. But I don't see what it has to do
with this patch? This problem existed since before the api was introduced.. Only difference
is kernel eats possibly corrupt CRCs now instead of IGT.


~Maarten
_______________________________________________
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