Messages in this thread | | | Date | Sat, 12 Apr 2008 09:07:33 -0400 | From | Christoph Hellwig <> | Subject | Re: [patch] e1000=y && e1000e=m regression fix |
| |
On Fri, Apr 11, 2008 at 07:43:49PM -0400, Jeff Garzik wrote: > However, due to a mistake now released to the public -- a tiny few PCI-E > chips are supported by e1000 -- you have a widely disparate feature set: > > e1000, old chips: full support > > e1000, a few PCI-E chips: basic support > > e1000e, all PCI-E chips: full support > > Since e1000e is all new and fancy AND CLEAN, the code for the same chips is > different -- thus Intel must make every PCI-E fix _twice_. > > It also means WE HAVE TO KEEP TOUCHING E1000, while supporting PCI-E chips. > After this PCI-E issue is resolved, I want to let e1000 sit and be stable > and not be touched. > > For a temporary situation, this is fine. Give me transition suggestions, > please!
PCI-E support should be removed from the e1000 driver ASAP, that is .26. What we need is a way to have CONFIG_E1000 pull in the e1000e driver automatically to not confuse kernel developers that don't know what hardware they actually have..
| |