netdev
[Top] [All Lists]

Re: possible bug in tcp_input.c

To: Tomas Szepe <szepe@xxxxxxxxxxxxxxx>
Subject: Re: possible bug in tcp_input.c
From: Andi Kleen <ak@xxxxxxx>
Date: Tue, 18 Nov 2003 15:01:22 +0100
Cc: linux-kernel@xxxxxxxxxxxxxxx, netdev@xxxxxxxxxxx, grof@xxxxxxxxx, davem@xxxxxxxxxx
In-reply-to: <20031118135805.GA9705@louise.pinerecords.com>
References: <20031024162959.GB11154@louise.pinerecords.com.suse.lists.linux.kernel> <p73ptgma58b.fsf@oldwotan.suse.de> <20031118135805.GA9705@louise.pinerecords.com>
Sender: netdev-bounce@xxxxxxxxxxx
On Tue, 18 Nov 2003 14:58:05 +0100
Tomas Szepe <szepe@xxxxxxxxxxxxxxx> wrote:

> On Oct-24 2003, Fri, 19:57 +0200
> Andi Kleen <ak@xxxxxxx> wrote:
> 
> > > /* tcp_input.c, line 1138 */
> > > static inline int tcp_head_timedout(struct sock *sk, struct tcp_opt *tp)
> > > {
> > >   return tp->packets_out && tcp_skb_timedout(tp, 
> > > skb_peek(&sk->write_queue));
> > > }
> > 
> > tp->packets_out > 0 implies that there is at least one packet in the write 
> > queue (it counts the number of unacked packets in flight, which are kept
> > in the write queue). When that's not the case something else is wrong.
> 
> Yes, that's exactly what davem said.  The corruption is happening somewhere
> in netsched/imq code that's not even part of the official kernel tree (and
> I'm told there's nobody to maintain the patch at present).

Ignore the mail. It was some machine flushing out an old mail queue
(with some very old mails from me that never made it out) 

I actually wrote it before DaveM if you check the dates ;-)

-Andi

<Prev in Thread] Current Thread [Next in Thread>