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

From: Christoph Hellwig
Date: Fri Apr 11 2008 - 07:37:29 EST


On Fri, Apr 11, 2008 at 01:26:53PM +0200, Ingo Molnar wrote:
> really, just because you do exceedingly good things to Linux does not
> give you a free card to do something bad to Linux in exchange. The two
> do not cancel out each other - because the bad things _add up_ and drive
> away users, irreversibly. To you e1000 is the center of the universe so
> you feel the price is worth paying. For others it is not. We want the
> good things from you and we'll say no thanks to the bad ideas. Kernel
> developers, especially old-timers, regularly forget about that.

Hey, hey calm down. The device moving over to e1000e shouldn never
have been added to e1000. They're totally differnet and the only reason
they got added in thefirst time was because soemone talked intel into
it.

We discussed this a long time and came to a wide agreement it should
move out. Now the actual transition could and should have been handled
better, but with all the pci-e hardware in a separate driver we're all
off better in the long term.

And this is not really comparable to the libata transition at all,
there's no user-visible changed. For every distro kernel that just
builds both driver it's a completely seamless transition, and for
people who build their own kernel we should find some Kconfig trickery
to make the transition easier. For example we could just built e1000e
when CONFIG_E1000 is set and spill a warning that starting from 1.1.2009
you will have to have CONFIG_E1000E set aswell.

>
> Ingo
> --
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
---end quoted text---
--
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/