General list for discussing Bufferbloat
 help / color / mirror / Atom feed
From: Neal Cardwell <ncardwell@google.com>
To: Eric Dumazet <eric.dumazet@gmail.com>
Cc: "Steinar H. Gunderson" <sgunderson@bigfoot.com>,
	Jonathan Morton <chromatix99@gmail.com>,
	 "aqm@ietf.org" <aqm@ietf.org>,
	bloat <bloat@lists.bufferbloat.net>
Subject: Re: [Bloat] TCP BBR paper is now generally available
Date: Sat, 3 Dec 2016 22:18:08 -0500	[thread overview]
Message-ID: <CADVnQynhHoD2-3TEgTFrQPZwejRvGkKO-uND0tEegzVNpPjZOg@mail.gmail.com> (raw)
In-Reply-To: <1480807468.18162.436.camel@edumazet-glaptop3.roam.corp.google.com>

[-- Attachment #1: Type: text/plain, Size: 1590 bytes --]

>    http://storage.sesse.net/bbr.pcap -- ssh+tar+gnupg

I agree with Eric that for the ssh+tar+gnupg case the ACK stream seems
like the culprit here. After about 1 second, the ACKs are suddenly
very stretched and very delayed (often more than 100ms). See the
attached screen shots.

I like Eric's theory that the ACKs might be going through fq.
Particularly since the uplink data starts having issues around the
same time as the ACKs for the downlink data.

neal

On Sat, Dec 3, 2016 at 6:24 PM, Eric Dumazet <eric.dumazet@gmail.com> wrote:
> On Sat, 2016-12-03 at 15:15 -0800, Eric Dumazet wrote:
>> On Sun, 2016-12-04 at 00:03 +0100, Steinar H. Gunderson wrote:
>> > On Sat, Dec 03, 2016 at 02:55:37PM -0800, Eric Dumazet wrote:
>> > > Note that starting from linux-4.4, e1000e gets gro_flush_timeout that
>> > > would help this precise workload
>> > >
>> > > https://git.kernel.org/cgit/linux/kernel/git/davem/net.git/commit/?id=32b3e08fff60494cd1d281a39b51583edfd2b18f
>> > >
>> > > Maybe you can redo the experiment in ~5 years when distro catches up ;)
>> >
>> > I can always find a backport, assuming the IPMI is still working. But it
>> > really wasn't like this earlier :-) Perhaps something changed on the path,
>> > unrelated to BBR.
>>
>> If the tcpdump is taken on receiver, how would you explain these gaps ?
>> Like the NIC was frozen for 40 ms  !
>
> Wait a minute. If you use fq on the receiver, then maybe your old debian
> kernel did not backport :
>
> https://git.kernel.org/cgit/linux/kernel/git/davem/net.git/commit/?id=9878196578286c5ed494778ada01da094377a686
>
>
>

[-- Attachment #2: bbr-hitting-ack-issue-port-4298-downlink.png --]
[-- Type: image/png, Size: 43540 bytes --]

[-- Attachment #3: bbr-hitting-ack-issue-port-4298-uplink.png --]
[-- Type: image/png, Size: 44206 bytes --]

  reply	other threads:[~2016-12-04  3:18 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-12-02 15:52 Dave Taht
2016-12-02 19:15 ` Aaron Wood
2016-12-02 20:32   ` Jonathan Morton
2016-12-02 22:22     ` Neal Cardwell
2016-12-02 22:40       ` Steinar H. Gunderson
2016-12-02 23:31         ` Eric Dumazet
2016-12-03 13:03           ` Neal Cardwell
2016-12-03 19:13             ` Steinar H. Gunderson
2016-12-03 20:20               ` Eric Dumazet
2016-12-03 20:26                 ` Jonathan Morton
2016-12-03 21:07                   ` Eric Dumazet
2016-12-03 21:34                     ` Steinar H. Gunderson
2016-12-03 21:50                       ` Eric Dumazet
2016-12-03 22:13                         ` Steinar H. Gunderson
2016-12-03 22:55                           ` Eric Dumazet
2016-12-03 23:02                             ` Eric Dumazet
2016-12-03 23:09                               ` Eric Dumazet
2016-12-03 23:03                             ` Steinar H. Gunderson
2016-12-03 23:15                               ` Eric Dumazet
2016-12-03 23:24                                 ` Eric Dumazet
2016-12-04  3:18                                   ` Neal Cardwell [this message]
2016-12-04  8:44                                   ` Steinar H. Gunderson
2016-12-04 17:13                                     ` Eric Dumazet
2016-12-04 17:38                                       ` Steinar H. Gunderson
2016-12-06 17:20                                   ` Steinar H. Gunderson
2016-12-06 21:31                                     ` Neal Cardwell
2016-12-03 21:38                     ` Jonathan Morton
2016-12-03 21:33                 ` Steinar H. Gunderson
2016-12-07 16:28               ` Alan Jenkins
2016-12-07 16:47                 ` Steinar H. Gunderson
2016-12-07 17:03                   ` Alan Jenkins
2016-12-08  8:24 ` Mikael Abrahamsson
2016-12-08 13:22   ` Dave Täht
2016-12-08 14:01     ` Mikael Abrahamsson
2016-12-08 21:29       ` Neal Cardwell
2016-12-08 22:31         ` Yuchung Cheng
2016-12-09 14:52           ` Klatsky, Carl

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

  List information: https://lists.bufferbloat.net/postorius/lists/bloat.lists.bufferbloat.net/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CADVnQynhHoD2-3TEgTFrQPZwejRvGkKO-uND0tEegzVNpPjZOg@mail.gmail.com \
    --to=ncardwell@google.com \
    --cc=aqm@ietf.org \
    --cc=bloat@lists.bufferbloat.net \
    --cc=chromatix99@gmail.com \
    --cc=eric.dumazet@gmail.com \
    --cc=sgunderson@bigfoot.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox