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

[Ingo Molnar <mingo@elte.hu>]

* Kok, Auke <auke-jan.h.kok@intel.com> wrote:

> Again - this has happened before, I remember many of my boxes not 
> booting because SATA Kconfig options changed and all my boxes failed 
> to move the proper Kconfig symbols over when I ran `make oldconfig` 
> myself. Somewhere around 2.6.20 or so.

that's an insane argument ... because we messed up in the past and have 
hurt users (and probably lost users) you feel like it gives you a free 
card to mess up again???

The IDE -> SATA migration, while i like the new SATA code and find it 
excellent and well-maintained (many kudos for that to Tejun, Jeff, Alan 
& co), caused a lot of trouble for users in one specific area, for no 
good reasons other than stupid personality conflicts: /dev/hda worked 
just as well as /dev/sda, the _name_ of the device should never have 
been changed.

So if you use _that_ aspect of the (otherwise cool) SATA/PATA code as a 
blueprint for the e1000 -> e1000e migration then you are on the worst 
possible track in terms of picking a role model ;-) It's as if you 
adored Sylvester Stallone for his vivid mimics, Jean-Claude Van Damne 
for his excellent acting skills and Paris Hilton for her brillant brain.

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.

	Ingo

-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone