From: Chris Snook <chris.snook@gmail.com>
To: Hagen Paul Pfeifer <hagen@jauu.net>
Cc: David Miller <davem@davemloft.net>,
shemminger@vyatta.com, netdev@vger.kernel.org,
eric.dumazet@gmail.com, ilpo.jarvinen@helsinki.fi,
therbert@google.com
Subject: Re: [PATCH] tcp: make TCP quick ACK behavior modifiable
Date: Mon, 23 Aug 2010 18:04:19 -0400 [thread overview]
Message-ID: <AANLkTi=aMjjz0dFqQg0ZKjEVjSbbvq49Yp7RBRV0SDri@mail.gmail.com> (raw)
In-Reply-To: <20100823215115.GA2745@nuttenaction>
On Mon, Aug 23, 2010 at 5:51 PM, Hagen Paul Pfeifer <hagen@jauu.net> wrote:
> * David Miller | 2010-08-23 14:21:14 [-0700]:
>
>>> At least google inc is too busy to disable tcp quick ack's. And I
>>> think they will save a lot of packets! Who make a bet? ;)
>>
>>I think assuming that turning off quick ACKs will be a net
>>positive for google's traffic is a bit presumptuous, don't
>>you?
>
> bet? ;) The patch provides a switch to disable quick acks. If google or
> someone else is using this switch or not is up to the audience! ;)
>
> I don't know how many packets the current average HTTP session includes, but
> assuming ~20 packets the saving of one packet is a benefit.
The user experience with HTTP is very sensitive to latency. Since the
payload packets in HTTP are large, HTTP workloads generally aren't
generating enough packets that packet count is something that needs to
be optimized. I'm sure there are exceptions, like web servers
delivering small AJAX or JSON objects, but they're the exception, not
the norm.
That said, I still like the idea of this feature, but I think it
should be a per-route tunable, for consistency with the delayed ack
patch.
-- Chris
next prev parent reply other threads:[~2010-08-23 22:04 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-08-23 19:00 [PATCH] tcp: make TCP quick ACK behavior modifiable Hagen Paul Pfeifer
2010-08-23 19:14 ` Stephen Hemminger
2010-08-23 19:57 ` Hagen Paul Pfeifer
2010-08-23 20:08 ` Hagen Paul Pfeifer
2010-08-23 21:21 ` David Miller
2010-08-23 21:51 ` Hagen Paul Pfeifer
2010-08-23 22:04 ` Chris Snook [this message]
2010-08-23 22:16 ` David Miller
2010-08-23 20:44 ` Eric Dumazet
2010-08-23 20:49 ` Hagen Paul Pfeifer
2010-08-23 21:10 ` Chris Snook
2010-08-23 22:01 ` Hagen Paul Pfeifer
2010-08-23 22:19 ` Chris Snook
2010-08-23 22:23 ` David Miller
2010-08-23 22:26 ` Hagen Paul Pfeifer
2010-08-23 23:17 ` Arnaldo Carvalho de Melo
2010-08-23 23:18 ` Arnaldo Carvalho de Melo
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='AANLkTi=aMjjz0dFqQg0ZKjEVjSbbvq49Yp7RBRV0SDri@mail.gmail.com' \
--to=chris.snook@gmail.com \
--cc=davem@davemloft.net \
--cc=eric.dumazet@gmail.com \
--cc=hagen@jauu.net \
--cc=ilpo.jarvinen@helsinki.fi \
--cc=netdev@vger.kernel.org \
--cc=shemminger@vyatta.com \
--cc=therbert@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 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).