From: Frederic Leroy <fredo@starox.org>
To: "Ilpo Järvinen" <ilpo.jarvinen@helsinki.fi>
Cc: Damian Lukowski <damian@tvk.rwth-aachen.de>,
Netdev <netdev@vger.kernel.org>, Asdo <asdo@shiftmail.org>,
David Miller <davem@davemloft.net>,
Eric Dumazet <eric.dumazet@gmail.com>,
Herbert Xu <herbert@gondor.apana.org.au>,
Greg KH <gregkh@suse.de>
Subject: Re: scp stalls mysteriously
Date: Thu, 3 Dec 2009 13:11:27 +0100 [thread overview]
Message-ID: <20091203131127.131e9122@houba> (raw)
In-Reply-To: <alpine.DEB.2.00.0912031121190.7024@wel-95.cs.helsinki.fi>
Le Thu, 3 Dec 2009 12:29:39 +0200 (EET),
"Ilpo Järvinen" <ilpo.jarvinen@helsinki.fi> a écrit :
> Opinions, Dave?, Greg?
>
> Now back to the issue...
>
> You said in the other mail that "All further test are on linus-stable
> tree.", which has this contradiction that Linus does not maintain
> stable trees. Which exactly was the tree used for the .9. test
Sorry I'm confused and so confuse you.
For .9 .10 and now I'm only using :
git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux-2.6.git
> Linus' tree or the 2.6.31 stable tree? I suppose the former since the
> revert wouldn't apply to 2.6.31 so I just want to confirm.
I didn't keep the source of the old 2.6.31 kernel I have.
So it's either
git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux-2.6.git
or
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-2.6-stable.git
> Nice thinking indeed Damian, thanks. ...But but, where exactly did
> you print? ...There are multiple returns and the return false branch
> is expected to have a zero retrans_stamp in a typical case but that
> is not a problem because we never use the value.
Here is the code :
http://www.starox.org/pub/scp_stall/printk_retrans_stamp.patch
> ...Anyway, if I'm wrong with my suspicion and it still holds that we
> have zero retrans_stamp in the substraction too, it could have
> something to do with this snippet:
>
> static void tcp_try_to_open(struct sock *sk, int flag)
> {
> struct tcp_sock *tp = tcp_sk(sk);
>
> tcp_verify_left_out(tp);
>
> if (!tp->frto_counter && tp->retrans_out == 0)
> tp->retrans_stamp = 0;
>
> ...It bit me last time when FRTO was enabled after very small
> modification (without running a full verification after the trivial
> looking modification). ...So I've worked around this clearing for
> FRTO as you can see :-).
:)
> Also, we have the another mystery to be solved, the fast
> retransmission is not triggered for some reason (or alternatively not
> captured in to a log), even in the working .9. case. It would be easy
> to see whether it works at all from TCP point of view by looking into
> mibs once you have have some transfers in a working configuration:
>
> grep -A1 TCP /proc/net/netstat
I will try this evening. I can do test only outside office hours.
--
Frédéric Leroy
next prev parent reply other threads:[~2009-12-03 12:11 UTC|newest]
Thread overview: 63+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-11-27 21:35 scp stalls mysteriously Frederic Leroy
2009-11-27 22:12 ` Ilpo Järvinen
[not found] ` <20091128010156.1219012a@houba>
2009-11-28 11:31 ` Ilpo Järvinen
2009-11-29 22:13 ` Ilpo Järvinen
2009-11-30 18:50 ` Frederic Leroy
2009-11-30 19:10 ` Ilpo Järvinen
2009-11-30 20:18 ` Ilpo Järvinen
2009-11-30 20:37 ` Frederic Leroy
2009-11-30 21:37 ` Ilpo Järvinen
2009-11-30 22:19 ` Frederic Leroy
2009-12-01 20:19 ` Frederic Leroy
2009-12-01 20:27 ` Ilpo Järvinen
2009-12-02 7:59 ` Frederic Leroy
2009-12-02 12:59 ` Ilpo Järvinen
2009-12-02 15:44 ` Frederic Leroy
2009-12-02 16:05 ` Ilpo Järvinen
2009-12-02 17:34 ` Frederic Leroy
2009-12-02 19:17 ` Damian Lukowski
2009-12-03 8:59 ` Frederic Leroy
2009-12-03 10:29 ` Ilpo Järvinen
2009-12-03 10:34 ` David Miller
2009-12-03 10:48 ` Ilpo Järvinen
2009-12-03 12:19 ` Asdo
2009-12-03 11:57 ` Damian Lukowski
2009-12-03 12:19 ` Damian Lukowski
2009-12-03 12:49 ` Ilpo Järvinen
2009-12-03 14:10 ` Damian Lukowski
2009-12-03 19:23 ` Frederic Leroy
2009-12-03 20:34 ` Damian Lukowski
2009-12-03 22:03 ` Frederic Leroy
2009-12-04 10:41 ` Ilpo Järvinen
2009-12-04 9:36 ` Frederic Leroy
2009-12-04 11:14 ` Ilpo Järvinen
2009-12-04 13:58 ` Frederic Leroy
2009-12-04 15:09 ` Ilpo Järvinen
2009-12-03 20:36 ` Ilpo Järvinen
2009-12-03 21:37 ` Frederic Leroy
2009-12-05 22:32 ` Damian Lukowski
2009-12-06 10:29 ` Ilpo Järvinen
2009-12-06 17:48 ` Ilpo Järvinen
2009-12-06 22:44 ` Damian Lukowski
2009-12-06 23:09 ` Ilpo Järvinen
2009-12-06 20:32 ` Frederic Leroy
2009-12-07 14:01 ` Ilpo Järvinen
2009-12-07 22:18 ` Frederic Leroy
2009-12-07 22:38 ` Ilpo Järvinen
2009-12-09 4:54 ` David Miller
2009-12-03 12:11 ` Frederic Leroy [this message]
2009-12-03 12:44 ` Ilpo Järvinen
2009-12-03 13:37 ` Arnd Hannemann
2009-12-03 14:27 ` Arnd Hannemann
2009-12-03 14:36 ` Ilpo Järvinen
2009-12-03 15:34 ` Arnd Hannemann
2009-12-03 15:48 ` Ilpo Järvinen
2009-12-03 18:32 ` Greg KH
2009-12-03 21:37 ` David Miller
2009-12-03 8:56 ` Frederic Leroy
2009-11-30 21:24 ` Frederic Leroy
2009-11-30 21:26 ` Ilpo Järvinen
2009-11-30 21:54 ` Frederic Leroy
2009-11-27 22:22 ` Frederic Leroy
2009-11-27 22:28 ` Ilpo Järvinen
2009-11-27 22:30 ` Ilpo Järvinen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20091203131127.131e9122@houba \
--to=fredo@starox.org \
--cc=asdo@shiftmail.org \
--cc=damian@tvk.rwth-aachen.de \
--cc=davem@davemloft.net \
--cc=eric.dumazet@gmail.com \
--cc=gregkh@suse.de \
--cc=herbert@gondor.apana.org.au \
--cc=ilpo.jarvinen@helsinki.fi \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.