Hi!
This patch fixes problem in drivers/net/ibm_ocp/ibm_ocp_enet.c.
Ring buffers initialization (init_rings) is called _after_ enabling MAL rx
channel.
If a packet arrives just after MAL was enabled but before init_rings
initialized buffers, you can see random crashes during EMAC open or receive
descriptor error interrupt.
This bug affects all systems that use drivers/net/ibm_ocp/ibm_ocp_enet.c
(405GP based, 440GP based, etc).
This patch is against current devel_2_4, but I think should apply without
any problems to devel_2_5 also.
Thanks.
===== drivers/net/ibm_ocp/ibm_ocp_enet.c 1.55 vs edited =====
--- 1.55/drivers/net/ibm_ocp/ibm_ocp_enet.c Thu Aug 22 11:29:32 2002
+++ edited/drivers/net/ibm_ocp/ibm_ocp_enet.c Wed Nov 20 21:34:30 2002
@@ -375,13 +375,15 @@
request_irq(BL_MAL_RXEOB,ppc405_eth_rxeob,0,"OCP EMAC RX
EOB",dev);
}
- /* enable all MAL transmit and receive channels */
+ /* init buffer descriptors rings */
+ init_rings(dev);
+ /* enable all MAL transmit and receive channels */
enable_mal_chan(fep);
/* set transmit and receive enable */
out_be32(&emacp->em0mr0, EMAC_M0_TXE | EMAC_M0_RXE);
- init_rings(dev);
+
netif_start_queue(dev);
printk(KERN_NOTICE "%s: IBM EMAC: open completed\n\n", dev->name);
Eugene Surovegin <mailto:ebs at innocent.com>
** Sent via the linuxppc-embedded mail list. See http://lists.linuxppc.org/
