On Mon, 25 Jul 2005, Russell Johnson wrote:
I just upgraded my Dell 2800 from kernel 2.6.7 to 2.6.11 and noticed that
the e1000 driver doesn't work when I change the MTU from 1500 to something
larger.  I traced the bug down to the following item in
e1000_main.c::e1000_up().  In a past life I recall PHY's needing to
explicitly have the 'power down' bit rewritten as a reset didn't clear that
state.  I changed the code as follows to fix the bug on my systems.  I
haven't seen this fix elsewhere so maybe somebody in the know about
e1000_main.c could verify this and apply it as a patch.

Thanks for the patch, your problem is interesting, although why reseting the phy has anything to do with mtu > 1500 not working I don't know.

when you do patches, please do then with diff -up, and specifiy the driver version you're patching, because I can't tell where in your driver you made this change.

Also, what adapter (lspci -n) what are steps to reproduce, etc.

If you could, please try the latest driver off of sourceforge at http://sf.net/projects/e1000, which will get us working off of the same source base.

Thanks,
  Jesse
-
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

Reply via email to