Re: [PATCH] EDAC: core EDAC support code

[Tim Small]

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.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/