On Sun, Mar 23, 2008 at 06:00:17PM +0100, Bernard Pidoux wrote:
...
quoted text
> Here is the result of your patch :
>
>
> WARNING: at net/core/sock.c:1522 sock_def_write_space()
> Pid: 3452, comm: mkiss Not tainted 2.6.24.3 #2
...

Thanks. Here is one more (to apply after the previous one).

Happy Holidays...
Jarek P.

---

diff -Nurp 2.6.24.3-with1/include/net/sock.h 2.6.24.3-with2/include/net/sock.h
--- 2.6.24.3-with1/include/net/sock.h	2008-01-24 23:58:37.000000000 +0100
+++ 2.6.24.3-with2/include/net/sock.h	2008-03-24 21:14:43.000000000 +0100
@@ -263,6 +263,7 @@ struct sock {
   	int			(*sk_backlog_rcv)(struct sock *sk,
 						  struct sk_buff *skb);  
 	void                    (*sk_destruct)(struct sock *sk);
+	int			sk_ax25_debug;
 };
 
 /*
diff -Nurp 2.6.24.3-with1/net/ax25/af_ax25.c 2.6.24.3-with2/net/ax25/af_ax25.c
--- 2.6.24.3-with1/net/ax25/af_ax25.c	2008-01-24 23:58:37.000000000 +0100
+++ 2.6.24.3-with2/net/ax25/af_ax25.c	2008-03-23 00:53:51.000000000 +0100
@@ -936,6 +936,7 @@ struct sock *ax25_make_new(struct sock *
 	}
 
 	sk->sk_protinfo = ax25;
+	sk->sk_ax25_debug = 1;
 	ax25->sk    = sk;
 
 	return sk;
diff -Nurp 2.6.24.3-with1/net/core/sock.c 2.6.24.3-with2/net/core/sock.c
--- 2.6.24.3-with1/net/core/sock.c	2008-03-22 12:10:10.000000000 +0100
+++ 2.6.24.3-with2/net/core/sock.c	2008-03-24 21:43:46.000000000 +0100
@@ -1519,6 +1519,9 @@ static void sock_def_write_space(struct 
 	if ((atomic_read(&sk->sk_wmem_alloc) << 1) <= sk->sk_sndbuf) {
 		if (sk->sk_sleep && waitqueue_active(sk->sk_sleep)) {
 			if (!sk->sk_socket || sk->sk_socket->sk != sk) {
+				if (net_ratelimit())
+					printk(KERN_INFO "sk_ax25_debug = %d, %p\n",
+						 sk->sk_ax25_debug, sk->sk_socket);
 				WARN_ON_ONCE(1);
 				sk->sk_socket = NULL;
 				sk->sk_sleep = NULL;
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html