From: Daniel Borkmann <daniel@iogearbox.net>
To: Eric Dumazet <eric.dumazet@gmail.com>
Cc: davem@davemloft.net, netdev@vger.kernel.org,
"Florian Westphal" <fw@strlen.de>,
"Mirja Kühlewind" <mirja.kuehlewind@tik.ee.ethz.ch>,
"Brian Trammell" <trammell@tik.ee.ethz.ch>,
"Eric Dumazet" <edumazet@google.com>,
"Dave That" <dave.taht@gmail.com>
Subject: Re: [PATCH net-next] tcp: add rfc3168, section 6.1.1.1. fallback
Date: Mon, 18 May 2015 14:31:29 +0200 [thread overview]
Message-ID: <5559DBA1.3060204@iogearbox.net> (raw)
In-Reply-To: <1431910796.621.7.camel@edumazet-glaptop2.roam.corp.google.com>
On 05/18/2015 02:59 AM, Eric Dumazet wrote:
...
> If first SYNACK ECE was delayed (our SYN rtx was spurious), maybe we can
> still keep ECN enabled ?
>
> So maybe do not clear tp->ecn_flags here, as it should be ultimately
> cleared in tcp_ecn_rcv_synack().
Thanks a lot for the feedback, Eric!
We'll be looking into this.
Best,
Daniel
prev parent reply other threads:[~2015-05-18 13:04 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-17 20:11 [PATCH net-next] tcp: add rfc3168, section 6.1.1.1. fallback Daniel Borkmann
2015-05-18 0:59 ` Eric Dumazet
2015-05-18 12:31 ` Daniel Borkmann [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=5559DBA1.3060204@iogearbox.net \
--to=daniel@iogearbox.net \
--cc=dave.taht@gmail.com \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=eric.dumazet@gmail.com \
--cc=fw@strlen.de \
--cc=mirja.kuehlewind@tik.ee.ethz.ch \
--cc=netdev@vger.kernel.org \
--cc=trammell@tik.ee.ethz.ch \
/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;
as well as URLs for NNTP newsgroup(s).