Re: [RFT] mv643xxx_eth_start_xmit oops

2006-11-10 Thread Stephen Hemminger
On Fri, 10 Nov 2006 22:30:43 +0100 Francois Romieu <[EMAIL PROTECTED]> wrote: > Stephen Hemminger <[EMAIL PROTECTED]> : > [...] > > skb_linearize is documented to free skb on failure. > > __skb_linearize > -> __pskb_pull_tail >-> pskb_expand_head > [...] > data = kmalloc(size +

Re: [RFT] mv643xxx_eth_start_xmit oops

2006-11-10 Thread Francois Romieu
Stephen Hemminger <[EMAIL PROTECTED]> : [...] > skb_linearize is documented to free skb on failure. __skb_linearize -> __pskb_pull_tail -> pskb_expand_head [...] data = kmalloc(size + sizeof(struct skb_shared_info), gfp_mask); if (!data) goto nodata;

Re: [RFT] mv643xxx_eth_start_xmit oops

2006-11-10 Thread Stephen Hemminger
On Fri, 10 Nov 2006 22:03:43 +0100 Francois Romieu <[EMAIL PROTECTED]> wrote: > Dale Farnsworth <[EMAIL PROTECTED]> : > [...] > > Can you provide more detail about the leak? > > + if (has_tiny_unaligned_frags(skb) && __skb_linearize(skb)) { > + stats->tx_dropped++; > +

Re: [RFT] mv643xxx_eth_start_xmit oops

2006-11-10 Thread Francois Romieu
Dale Farnsworth <[EMAIL PROTECTED]> : [...] > Can you provide more detail about the leak? + if (has_tiny_unaligned_frags(skb) && __skb_linearize(skb)) { + stats->tx_dropped++; + if (net_ratelimit()) + printk(KERN_DEBUG "%s: failed to lineariz

Re: [RFT] mv643xxx_eth_start_xmit oops

2006-11-10 Thread Dale Farnsworth
In article <[EMAIL PROTECTED]> you write: Ueimor <[EMAIL PROTECTED]> wrote: > Stephen Hemminger <[EMAIL PROTECTED]> : > [...] > > diff --git a/drivers/net/mv643xx_eth.c b/drivers/net/mv643xx_eth.c > [...] > > It seems to propagate a leak from the initial codebase. Can you provide more detail abou

Re: [RFT] mv643xxx_eth_start_xmit oops

2006-11-10 Thread Francois Romieu
Stephen Hemminger <[EMAIL PROTECTED]> : [...] > diff --git a/drivers/net/mv643xx_eth.c b/drivers/net/mv643xx_eth.c > index 9997081..4052bfe 100644 > --- a/drivers/net/mv643xx_eth.c > +++ b/drivers/net/mv643xx_eth.c > @@ -1191,25 +1191,23 @@ static int mv643xx_eth_start_xmit(struct > struct ne

[RFT] mv643xxx_eth_start_xmit oops

2006-11-10 Thread Stephen Hemminger
On Fri, 10 Nov 2006 12:17:45 -0700 Erik Andersen <[EMAIL PROTECTED]> wrote: > I have a Pegasos2 powerpc system acting as my home server. With > 2.6.16.x it was 100% stable and I had months of uptime, rebooting > only to periodically apply security updates to the kernel. > > With 2.6.17 and 2.6.1