Re: [patch] e1000=y && e1000e=m regression fix

From: Christoph Hellwig
Date: Sat Apr 12 2008 - 09:08:40 EST


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..
--
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/