From: Lawrence Brakmo <brakmo@fb.com>
To: Kenneth Klette Jonassen <kennetkl@ifi.uio.no>
Cc: netdev <netdev@vger.kernel.org>, Kernel Team <Kernel-team@fb.com>,
"Neal Cardwell" <ncardwell@google.com>,
Eric Dumazet <eric.dumazet@gmail.com>,
Yuchung Cheng <ycheng@google.com>,
Stephen Hemminger <stephen@networkplumber.org>
Subject: Re: [RFC PATCH v5 net-next 4/4] tcp: add NV congestion control
Date: Wed, 26 Aug 2015 00:05:43 +0000 [thread overview]
Message-ID: <D20249D6.71D2%brakmo@fb.com> (raw)
In-Reply-To: <CA++eYdvYopM+TSiWa+gt4Z0RkjAM2Kyxvga8XodSm4w=kSmxVw@mail.gmail.com>
On 8/5/15, 5:51 PM, "knneth@gmail.com on behalf of Kenneth Klette
Jonassen" <knneth@gmail.com on behalf of kennetkl@ifi.uio.no> wrote:
>On Wed, Aug 5, 2015 at 3:39 AM, Lawrence Brakmo <brakmo@fb.com> wrote:
>> This is a request for comments.
>
>Nice to see more development on delay-based congestion control.
>
>It would be good to see how NV stacks up against CDG. Any chance of
>adding cdg as a congestion control parameter to your experiments?
Done. I¹m updating the TCP-NV page with the updated results, should be
done by tomorrow (8/26).
>Experiments on NV without its temporary cwnd reductions would also be
>of interest -- to get a reference of how effective this mechanism is.
Done. It turns out that it only improves fairness a little but hurts
P99 latencies more significantly. So it is now off by default.
Thanks for making me re-examine this feature.
prev parent reply other threads:[~2015-08-26 0:05 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-05 1:39 [RFC PATCH v5 net-next 0/4] tcp: add NV congestion control Lawrence Brakmo
2015-08-05 1:39 ` [RFC PATCH v5 net-next 1/4] tcp: replace cnt & rtt with struct in pkts_acked() Lawrence Brakmo
2015-08-05 1:39 ` [RFC PATCH v5 net-next 2/4] tcp: refactor struct tcp_skb_cb Lawrence Brakmo
2015-08-05 1:39 ` [RFC PATCH v5 net-next 3/4] tcp: add in_flight to tcp_skb_cb Lawrence Brakmo
2015-08-05 1:39 ` [RFC PATCH v5 net-next 4/4] tcp: add NV congestion control Lawrence Brakmo
2015-08-06 0:51 ` Kenneth Klette Jonassen
2015-08-21 21:29 ` Lawrence Brakmo
2015-08-26 0:05 ` Lawrence Brakmo [this message]
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=D20249D6.71D2%brakmo@fb.com \
--to=brakmo@fb.com \
--cc=Kernel-team@fb.com \
--cc=eric.dumazet@gmail.com \
--cc=kennetkl@ifi.uio.no \
--cc=ncardwell@google.com \
--cc=netdev@vger.kernel.org \
--cc=stephen@networkplumber.org \
--cc=ycheng@google.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 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.