Jeff Garzik wrote:
Jason Lunz wrote:
What's the prognosis for the 7.5.5-series e1000 reorg going into
netdev-2.6?
As I have posted previously, I am not keen on merging basically an e1000
rewrite. That basically throws away all Internet-wide testing so far.
The rewrite was not done according to Linux kernel standards (read:
progression of changes), so any problems will bisect into The Big
Commit(tm) and go no further. A rewrite also means the complete driver
has to be reviewed from scratch, since such a large patch is basically
impossible to review. Finally, the rewrite doesn't do much to clean up
the e1000 driver.
I warned Intel for months not to do things this way, but I guess my
opinion counts for little :)
no, they do
However, I am stuck with a half rewritten e1000 codebase (the 7.4.x code base)
and on top of that all the changes that you requested from us. This effort has
taken me about two months of time.
Taking apart that work change for every change seems very wasteful to me: it
mucks up the current e1000 driver potentially, might break it and that is
exactly what we want to prevent.
That's why we want to introduce a second e1000 driver (named differently, pick
any name) that contains the new code base, side-by-side into the kernel with the
current e1000.
This would allow everyone to fallback and compare the new and old code
instantly, for several kernel releases at least. After that period, we can
retire the old codebase.
Given the size of the changes and the fact that this is an interal API change
that gigantically changes how e1000 internally works, there is *no* way that we
can introduce this change in any other way in my opinion.
Looking at that, I think that bringing a second and new e1000 driver into the
kernel is the best thing to do, and that is what I have been working towards.
This new e1000 codebase goes miles and miles beyond what I posted in april/march
and what is in -mm. For instance: the new codebase no longer has mac_type checks
in *any* part of e1000_main.c.
I will try to post it as soon as I can. Please bear with me until I get that out
to everyone.
Auke
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
