Re: [regression] e1000e broke e1000

From: Ingo Molnar
Date: Fri Apr 11 2008 - 05:00:27 EST



* Bill Davidsen <davidsen@xxxxxxx> wrote:

>> the solution is rather straightforward: if E1000 is built-in then
>> E1000E should be built-in as well or disabled (i.e. it should not be
>> possible to build it as a module in that case) - because the PCI ID
>> stealing trick now connects the two drivers unconditionally. [ If
>> e1000 is a module then e1000e can be a module (or disabled) - this
>> would be the most common configuration. ]
>
> And this would seem to break the most common means of testing a new
> driver for existing (and working!) hardware, which is to build both
> drivers as modules, install the new one, and if it appears to have
> problems either remove and insert the old driver by hand, or boot
> forcing the old driver.
>
> I can't be the only person who tests kernels on machines I wouldn't
> use to build a kernel, and uses modprobe.conf to test new driver
> functionality.

yes, but note that the breakage you are talking about is not caused my
patch, it is caused by the planned change to remove those PCI IDs from
e1000.

my suggested change only solves part of the more general problem you
touch upon. (and it does not make it worse in any way)

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