Re: [PATCH] EDAC: core EDAC support code

Tim Small
2006-03-10T11:06:36Z

Arjan van de Ven wrote:

> It depends on how many PCI devices in your machine you wish to
>
>>blacklist or whitelist. The motivation for this feature is that
>>certain known badly-designed devices report an endless stream of
>>spurious PCI bus parity errors. We want to skip such devices when
>>checking for PCI bus parity errors.
>>
>>
>
>ok so this is actually a per pci device property!
>I would suggest moving this property to the pci device itself, not doing
>it inside an edac directory.
>
>
Yes, this seems more sensible to me. For one thing, I suspect that just
keying on vendor:device is probably too blunt for this and that
blacklisting a particular PCI device revision is a likely requirement,
as well as subsystem vendor/subsystem device.

Tim.