Re: [PATCH] e1000e: test MSI interrupts

[Jeff Garzik <jeff@garzik.org>]

Brandeburg, Jesse wrote:
> I get your point, but this seems a maintainance problem due to not being
> able to "future proof" the solution.  I know what is (IDs) available
> now, but I don't know how many systems in the future IBM will release
> with a similar bridge but a different device ID that causes the same
> issue.

Future-proofing in that way is a pipe dream.

You hope to predict what _errata_, what out-of-spec behavior future 
hardware will have.  Trying to code for such N^M possible futures will 
lead to code bloat, depression, and eventually madness.


> Should we take on the maintenance of continually having to add
> every new bridge device that has this issue to our driver?  Users just
> want this stuff to work when they plug it in.

As David noted, we touch quirks.c all the time for various platform 
eccentricities.  Adding a new id is easy and takes two seconds.  The 
same ease of change applies to any driver-local list of ids, too.

Anyway, I think a better question to ask is:  should we bloat up every 
driver testing for platform quirks found on a minority of platforms?

Moreover, "it doesn't work" type errata is typically fixed in future 
chip generations -- making any such generic test /less/ valuable, 
because of the lower likelihood that IBM will continue to release this 
buggy hardware for decades.

We have an existing "this bridge and MSI don't get along" list.  Adding 
an id is a one-line patch.

	Jeff




-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace