Re: BUG in tcp.c ?
Date: Fri, 6 Oct 2000 17:45:47 -0200 (BRST) From: Marcelo Tosatti <[EMAIL PROTECTED]> 2.2.17 is broken too. I've fixed it in my 2.2.x sources as well. Thanks. Later, David S. Miller [EMAIL PROTECTED] - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] Please read the FAQ at http://www.tux.org/lkml/
Re: BUG in tcp.c ?
On Fri, 6 Oct 2000, David S. Miller wrote: >Date: Fri, 6 Oct 2000 19:25:38 -0300 (BRST) >From: Rik van Riel <[EMAIL PROTECTED]> > >Is this an actual bug, or am I overlooking something? > > It is a bug and I'll change TCP's sendmsg to use sk->allocation as it > should. Thanks for pointing this out. 2.2.17 is broken too. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] Please read the FAQ at http://www.tux.org/lkml/
Re: BUG in tcp.c ?
Date: Fri, 6 Oct 2000 19:25:38 -0300 (BRST) From: Rik van Riel <[EMAIL PROTECTED]> Is this an actual bug, or am I overlooking something? It is a bug and I'll change TCP's sendmsg to use sk->allocation as it should. Thanks for pointing this out. Later, David S. Miller [EMAIL PROTECTED] - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] Please read the FAQ at http://www.tux.org/lkml/
BUG in tcp.c ?
Hi, I found the following lines in tcp.c, after trying to identify and track down what I thought to be an nbd bug... net/ipv4/tcp.c: 1028 if (tcp_memory_free(sk)) 1029 skb = tcp_alloc_skb(sk, tmp, GFP_KERNEL) While this looks ok at first glance, it rather conflicts with the following lines of code from net/core/sock.c: 785 skb = alloc_skb(try_size, sk->allocation); And from drivers/block/nbd.c: 109 sock->sk->allocation = GFP_ATOMIC; .. 121 if (send) 122 result = sock_sendmsg(sock, &msg, size); Here we see that the nbd driver sets the allocation type to GFP_ATOMIC (and for a good reason), sock.c honours this field, but for some reason tcp.c doesn't ... Is this an actual bug, or am I overlooking something? regards, Rik -- "What you're running that piece of shit Gnome?!?!" -- Miguel de Icaza, UKUUG 2000 http://www.conectiva.com/ http://www.surriel.com/ - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] Please read the FAQ at http://www.tux.org/lkml/