Re: Poss. bug in tulip driver since 2.4.7

From: Jeff Garzik
Date: Wed Nov 12 2003 - 23:35:43 EST


Prasanna Meda wrote:
No, you need to bring the for loop outside the loop.
- Otherwise we need to reset the setup_frame to
tp->setup_frame after every loop.
- You do not need to set the setup_frm for every
mc address, we can set once after the complete
has_table is ready.
And also the 2.4 code missed a bit in tx_flags.

We did the following change that makes it identical
to 2.2 tulip driver, and it worked.

--- Linux/drivers/net/tulip/tulip_core.c Fri Oct 10 20:22:29 2003
+++ linux/drivers/net/tulip/tulip_core.c Fri Oct 10 20:28:19 2003


Yeah, that looks better... I'll give it a quick test locally, then push it to Linus.

Jeff



-
To unsubscribe from this list: send the line "unsubscribe linux-net" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html