Re: Linux 2.4.19-rc1

From: Simon Kirby (sim@netnation.com)
Date: Tue Jun 25 2002 - 18:06:07 EST


Hello,

This fix did not yet make it in to the tg3 driver as of -rc1. This patch
fixes an occasional crash problem with several of our boxes and the tg3
driver (fix from Jes).

--

Hi Dave

Here's another one, tg3_recycle_rx() needs to declare dest_idx_unmasked as unsigned int or it will do funny stuff on modular division once we go above 31 bit values.

I don't think this is the bug that Scott was hitting, but it is of course possible. It takes about 120 minutes for me with a packet generator, spewing out 64 byte packets, to trigger it.

Cheers, Jes

--- ../orig/drivers/net/tg3.c Tue May 14 07:30:52 2002 +++ drivers/net/tg3.c Fri Jun 14 11:20:48 2002 @@ -1689,7 +1669,7 @@ * tg3_alloc_rx_skb for full details. */ static void tg3_recycle_rx(struct tg3 *tp, u32 opaque_key, - int src_idx, int dest_idx_unmasked) + int src_idx, u32 dest_idx_unmasked) { struct tg3_rx_buffer_desc *src_desc, *dest_desc; struct ring_info *src_map, *dest_map;

--

Simon-

[ Stormix Technologies Inc. ][ NetNation Communications Inc. ] [ sim@stormix.com ][ sim@netnation.com ] [ Opinions expressed are not necessarily those of my employers. ] - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Sun Jun 30 2002 - 22:00:10 EST