Thread (5 messages) 5 messages, 2 authors, 2003-12-25

[PATCH] fix outstanding ref's preventing ether driver unload

flat view
STALE8348d

From: Stephen Hemminger <hidden>
Date: 2003-12-23 22:29:31

Patch against 2.6.0 to fix the problem of being unable to load the ethernet
driver because of reference's still being held.  The problem reference's
are from IPV6 network discovery packets that get captured by the af_packet protocol
and queued onto a socket queue (which may never drain).  The route dst entries
in the skbuff get clone'd and won't be freed until the socket read.

diff -Nru a/net/packet/af_packet.c b/net/packet/af_packet.c
--- a/net/packet/af_packet.c	Tue Dec 23 14:25:10 2003
+++ b/net/packet/af_packet.c	Tue Dec 23 14:25:10 2003
@@ -246,6 +246,10 @@
 	if ((skb = skb_share_check(skb, GFP_ATOMIC)) == NULL)
 		goto oom;
 
+	/* drop any routing info */
+	dst_release(skb->dst);
+	skb->dst = NULL;
+
 	spkt = (struct sockaddr_pkt*)skb->cb;
 
 	skb_push(skb, skb->data-skb->mac.raw);
@@ -486,6 +490,9 @@
 
 	skb_set_owner_r(skb, sk);
 	skb->dev = NULL;
+	dst_release(skb->dst);
+	skb->dst = NULL;
+
 	spin_lock(&sk->sk_receive_queue.lock);
 	po->stats.tp_packets++;
 	__skb_queue_tail(&sk->sk_receive_queue, skb);
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help