Re: [PATCH] tcp: Avoid infinite loop on recvmsg bug

2012-12-10 Thread David Miller
I've tossed these two patches under the carpet, so you'll need to repost whichever one you want me to consider. Basically, discussing old patches is pretty useless without a resend to get it back into the fore-front of the patchwork queue. So please don't reference old stale patches without an a

Re: [PATCH] tcp: Avoid infinite loop on recvmsg bug

2012-12-10 Thread Julius Werner
Hi Dave, Have you thought about picking up one of the patches to tcp_recvmsg I proposed in this thread? We consider the underlying bug in Chromium OS that led mere here to be fixed now, but I bet this will not be the last time someone hits this code path and has to deal with the bad error handling

Re: [PATCH] tcp: Avoid infinite loop on recvmsg bug

2012-11-08 Thread Eric Dumazet
On Wed, 2012-11-07 at 18:25 -0800, Julius Werner wrote: > > So you probably are fighting a bug we already fixed in upstream kernel. > > > > (commit c8628155ece363 "tcp: reduce out_of_order memory use" did not > > played well with cloned skbs.) > > > > This issue was already discussed on netdev in t

Re: [PATCH] tcp: Avoid infinite loop on recvmsg bug

2012-11-07 Thread Julius Werner
> So you probably are fighting a bug we already fixed in upstream kernel. > > (commit c8628155ece363 "tcp: reduce out_of_order memory use" did not > played well with cloned skbs.) > > This issue was already discussed on netdev in the past. Thanks for the hint. Unfortunately, we have not pulled c86

Re: [PATCH] tcp: Avoid infinite loop on recvmsg bug

2012-11-07 Thread Eric Dumazet
On Wed, 2012-11-07 at 15:33 -0800, Eric Dumazet wrote: > So you probably are fighting a bug we already fixed in upstream kernel. > > (commit c8628155ece363 "tcp: reduce out_of_order memory use" did not > played well with cloned skbs.) > > This issue was already discussed on netdev in the past.

Re: [PATCH] tcp: Avoid infinite loop on recvmsg bug

2012-11-07 Thread Eric Dumazet
On Wed, 2012-11-07 at 13:14 -0800, Julius Werner wrote: > > What I find very sad in all this is that you didnt mention the driver > > that was triggering this bug. > > Sorry, I was just trying to keep this thread focussed on one patch. > The bug report that led me to this is publicly accessible at

Re: [PATCH] tcp: Avoid infinite loop on recvmsg bug

2012-11-07 Thread Julius Werner
> What I find very sad in all this is that you didnt mention the driver > that was triggering this bug. Sorry, I was just trying to keep this thread focussed on one patch. The bug report that led me to this is publicly accessible at http://crosbug.com/35827. We have encountered the problem only on

Re: [PATCH] tcp: Avoid infinite loop on recvmsg bug

2012-11-07 Thread Eric Dumazet
On Wed, 2012-11-07 at 11:33 -0800, Julius Werner wrote: > tcp_recvmsg contains a sanity check that WARNs when there is a gap > between the socket's copied_seq and the first buffer in the > sk_receive_queue. In theory, the TCP stack makes sure that This Should > Never Happen (TM)... however, practic

[PATCH] tcp: Avoid infinite loop on recvmsg bug

2012-11-07 Thread Julius Werner
tcp_recvmsg contains a sanity check that WARNs when there is a gap between the socket's copied_seq and the first buffer in the sk_receive_queue. In theory, the TCP stack makes sure that This Should Never Happen (TM)... however, practice shows that there are still a few bug reports from it out there