* [PATCH] spidernet : fix memory leak in spider_net_stop
@ 2007-02-01 11:07 Jens Osterkamp
2007-02-01 17:36 ` Linas Vepstas
2007-02-02 13:34 ` Jeff Garzik
0 siblings, 2 replies; 9+ messages in thread
From: Jens Osterkamp @ 2007-02-01 11:07 UTC (permalink / raw)
To: Linas Vepstas, James K Lewis, linuxppc-dev, netdev, jgarzik,
cbe-oss-dev, Ishizaki Kou
We forget to call spider_net_free_rx_chain_contents which does the
actual dev_kfree_skb. New skbs are allocated from skbuff_head_cache
on each "ifconfig up" letting the cache grow infinitely.
This patch fixes it.
Signed-off-by: Jens Osterkamp <jens@de.ibm.com>
Index: linux-2.6.20-rc6/drivers/net/spider_net.c
===================================================================
--- linux-2.6.20-rc6.orig/drivers/net/spider_net.c
+++ linux-2.6.20-rc6/drivers/net/spider_net.c
@@ -1925,6 +1925,8 @@ spider_net_stop(struct net_device *netde
/* release chains */
spider_net_release_tx_chain(card, 1);
+ spider_net_free_rx_chain_contents(card);
+
spider_net_free_chain(card, &card->tx_chain);
spider_net_free_chain(card, &card->rx_chain);
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] spidernet : fix memory leak in spider_net_stop
2007-02-01 11:07 [PATCH] spidernet : fix memory leak in spider_net_stop Jens Osterkamp
@ 2007-02-01 17:36 ` Linas Vepstas
2007-02-02 13:34 ` Jeff Garzik
1 sibling, 0 replies; 9+ messages in thread
From: Linas Vepstas @ 2007-02-01 17:36 UTC (permalink / raw)
To: Jens Osterkamp
Cc: James K Lewis, linuxppc-dev, netdev, jgarzik, cbe-oss-dev, Ishizaki Kou
A variant of this patch is queued up for 2.6.21,
as a part of the patch
http://www.mail-archive.com/netdev@vger.kernel.org/msg28213.html
I beleive its in the -mm tree, I have not checked.
--linas
On Thu, Feb 01, 2007 at 12:07:47PM +0100, Jens Osterkamp wrote:
>
> We forget to call spider_net_free_rx_chain_contents which does the
> actual dev_kfree_skb. New skbs are allocated from skbuff_head_cache
> on each "ifconfig up" letting the cache grow infinitely.
>
> This patch fixes it.
>
> Signed-off-by: Jens Osterkamp <jens@de.ibm.com>
>
> Index: linux-2.6.20-rc6/drivers/net/spider_net.c
> ===================================================================
> --- linux-2.6.20-rc6.orig/drivers/net/spider_net.c
> +++ linux-2.6.20-rc6/drivers/net/spider_net.c
> @@ -1925,6 +1925,8 @@ spider_net_stop(struct net_device *netde
> /* release chains */
> spider_net_release_tx_chain(card, 1);
>
> + spider_net_free_rx_chain_contents(card);
> +
> spider_net_free_chain(card, &card->tx_chain);
> spider_net_free_chain(card, &card->rx_chain);
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] spidernet : fix memory leak in spider_net_stop
@ 2007-02-01 17:36 ` Linas Vepstas
0 siblings, 0 replies; 9+ messages in thread
From: Linas Vepstas @ 2007-02-01 17:36 UTC (permalink / raw)
To: Jens Osterkamp; +Cc: netdev, cbe-oss-dev, linuxppc-dev, jgarzik, James K Lewis
A variant of this patch is queued up for 2.6.21,
as a part of the patch
http://www.mail-archive.com/netdev@vger.kernel.org/msg28213.html
I beleive its in the -mm tree, I have not checked.
--linas
On Thu, Feb 01, 2007 at 12:07:47PM +0100, Jens Osterkamp wrote:
>
> We forget to call spider_net_free_rx_chain_contents which does the
> actual dev_kfree_skb. New skbs are allocated from skbuff_head_cache
> on each "ifconfig up" letting the cache grow infinitely.
>
> This patch fixes it.
>
> Signed-off-by: Jens Osterkamp <jens@de.ibm.com>
>
> Index: linux-2.6.20-rc6/drivers/net/spider_net.c
> ===================================================================
> --- linux-2.6.20-rc6.orig/drivers/net/spider_net.c
> +++ linux-2.6.20-rc6/drivers/net/spider_net.c
> @@ -1925,6 +1925,8 @@ spider_net_stop(struct net_device *netde
> /* release chains */
> spider_net_release_tx_chain(card, 1);
>
> + spider_net_free_rx_chain_contents(card);
> +
> spider_net_free_chain(card, &card->tx_chain);
> spider_net_free_chain(card, &card->rx_chain);
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] spidernet : fix memory leak in spider_net_stop
2007-02-01 17:36 ` Linas Vepstas
@ 2007-02-01 20:40 ` Jens Osterkamp
-1 siblings, 0 replies; 9+ messages in thread
From: Jens Osterkamp @ 2007-02-01 20:40 UTC (permalink / raw)
To: Linas Vepstas
Cc: James K Lewis, linuxppc-dev, netdev, jgarzik, cbe-oss-dev, Ishizaki Kou
On Thursday 01 February 2007 6:36 pm, Linas Vepstas wrote:
>
> A variant of this patch is queued up for 2.6.21,
> as a part of the patch
> http://www.mail-archive.com/netdev@vger.kernel.org/msg28213.html
>
> I beleive its in the -mm tree, I have not checked.
Ah, good ! I knew that you had several patches in the queue but wasnt aware
of the detailed content.
Jens
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] spidernet : fix memory leak in spider_net_stop
@ 2007-02-01 20:40 ` Jens Osterkamp
0 siblings, 0 replies; 9+ messages in thread
From: Jens Osterkamp @ 2007-02-01 20:40 UTC (permalink / raw)
To: Linas Vepstas; +Cc: netdev, cbe-oss-dev, linuxppc-dev, jgarzik, James K Lewis
On Thursday 01 February 2007 6:36 pm, Linas Vepstas wrote:
>
> A variant of this patch is queued up for 2.6.21,
> as a part of the patch
> http://www.mail-archive.com/netdev@vger.kernel.org/msg28213.html
>
> I beleive its in the -mm tree, I have not checked.
Ah, good ! I knew that you had several patches in the queue but wasnt aware
of the detailed content.
Jens
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] spidernet : fix memory leak in spider_net_stop
2007-02-01 11:07 [PATCH] spidernet : fix memory leak in spider_net_stop Jens Osterkamp
@ 2007-02-02 13:34 ` Jeff Garzik
2007-02-02 13:34 ` Jeff Garzik
1 sibling, 0 replies; 9+ messages in thread
From: Jeff Garzik @ 2007-02-02 13:34 UTC (permalink / raw)
To: Jens Osterkamp
Cc: Linas Vepstas, James K Lewis, linuxppc-dev, netdev, cbe-oss-dev,
Ishizaki Kou
Jens Osterkamp wrote:
> We forget to call spider_net_free_rx_chain_contents which does the
> actual dev_kfree_skb. New skbs are allocated from skbuff_head_cache
> on each "ifconfig up" letting the cache grow infinitely.
>
> This patch fixes it.
>
> Signed-off-by: Jens Osterkamp <jens@de.ibm.com>
applied
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] spidernet : fix memory leak in spider_net_stop
@ 2007-02-02 13:34 ` Jeff Garzik
0 siblings, 0 replies; 9+ messages in thread
From: Jeff Garzik @ 2007-02-02 13:34 UTC (permalink / raw)
To: Jens Osterkamp; +Cc: netdev, cbe-oss-dev, linuxppc-dev, James K Lewis
Jens Osterkamp wrote:
> We forget to call spider_net_free_rx_chain_contents which does the
> actual dev_kfree_skb. New skbs are allocated from skbuff_head_cache
> on each "ifconfig up" letting the cache grow infinitely.
>
> This patch fixes it.
>
> Signed-off-by: Jens Osterkamp <jens@de.ibm.com>
applied
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] spidernet : fix memory leak in spider_net_stop
2007-02-02 13:34 ` Jeff Garzik
@ 2007-02-08 4:15 ` Ishizaki Kou
-1 siblings, 0 replies; 9+ messages in thread
From: Ishizaki Kou @ 2007-02-08 4:15 UTC (permalink / raw)
To: jeff; +Cc: jens, linas, netdev, linuxppc-dev, jim
Jeff-san,
> Jens Osterkamp wrote:
> > We forget to call spider_net_free_rx_chain_contents which does the
> > actual dev_kfree_skb. New skbs are allocated from skbuff_head_cache
> > on each "ifconfig up" letting the cache grow infinitely.
> >
> > This patch fixes it.
> >
> > Signed-off-by: Jens Osterkamp <jens@de.ibm.com>
>
> applied
This fix was applied twice to netdev-2.6.git#upstream.
Because Linas-san's patch has been applied to netdev-2.6.git#upstream.
Best regards,
Kou Ishizaki
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] spidernet : fix memory leak in spider_net_stop
@ 2007-02-08 4:15 ` Ishizaki Kou
0 siblings, 0 replies; 9+ messages in thread
From: Ishizaki Kou @ 2007-02-08 4:15 UTC (permalink / raw)
To: jeff; +Cc: linuxppc-dev, jim, netdev
Jeff-san,
> Jens Osterkamp wrote:
> > We forget to call spider_net_free_rx_chain_contents which does the
> > actual dev_kfree_skb. New skbs are allocated from skbuff_head_cache
> > on each "ifconfig up" letting the cache grow infinitely.
> >
> > This patch fixes it.
> >
> > Signed-off-by: Jens Osterkamp <jens@de.ibm.com>
>
> applied
This fix was applied twice to netdev-2.6.git#upstream.
Because Linas-san's patch has been applied to netdev-2.6.git#upstream.
Best regards,
Kou Ishizaki
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2007-02-08 4:16 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-02-01 11:07 [PATCH] spidernet : fix memory leak in spider_net_stop Jens Osterkamp
2007-02-01 17:36 ` Linas Vepstas
2007-02-01 17:36 ` Linas Vepstas
2007-02-01 20:40 ` Jens Osterkamp
2007-02-01 20:40 ` Jens Osterkamp
2007-02-02 13:34 ` Jeff Garzik
2007-02-02 13:34 ` Jeff Garzik
2007-02-08 4:15 ` Ishizaki Kou
2007-02-08 4:15 ` Ishizaki Kou
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.