General list for discussing Bufferbloat
 help / color / mirror / Atom feed
From: "Toke Høiland-Jørgensen" <toke@toke.dk>
To: David Lang <david@lang.hm>, Jonathan Morton <chromatix99@gmail.com>
Cc: bloat <bloat@lists.bufferbloat.net>
Subject: Re: [Bloat] lwn.net's tcp small queues vs wifi aggregation solved
Date: Tue, 26 Jun 2018 13:16:54 +0200	[thread overview]
Message-ID: <87in65n6ft.fsf@toke.dk> (raw)
In-Reply-To: <alpine.DEB.2.02.1806251746010.10496@nftneq.ynat.uz>

David Lang <david@lang.hm> writes:

> On Tue, 26 Jun 2018, Jonathan Morton wrote:
>
>>> On 26 Jun, 2018, at 3:36 am, Simon Barber <simon@superduper.net> wrote:
>>>
>>> Most hardware needs the packet finalized before it starts to contend for the 
>>> medium (as far as I’m aware - let me know if you know differently). One issue 
>>> is that if RTS/CTS is in use, then the packet duration needs to be known in 
>>> advance (or at least mid point of the RTS transmission).
>>
>> This is a valid argument.  I think we could successfully argue for a delay of 
>> 1ms, if there isn't already enough data in the queue to fill an aggregate, 
>> after the oldest packet arrives until a request is issued.
>
> why does the length of the txop need to be known at the time that it's
> requested?

Because that's how the hardware is designed. There are really two
discussions here: (1) what could we do with a clean-slate(ish) design,
and (2) what can we retrofit into existing drivers such as the ath9k.

I think that the answer to (1) is probably 'quite a lot', but
unfortunately the answer to (2) is 'not that much'. We could probably do
a little bit better in ath9k, but for anything newer all bets are off,
as this functionality has moved into firmware.

Now, if there was a hardware vendor that was paying attention and could
do the right thing throughout the stack, that would be awesome of
course. But for Linux at least, sadly it seems that most hardware
vendors can barely figure out how to get *any* driver upstream... :/

Also, from a conceptual point of view, I really think ACK timing issues
are best solved at the TCP stack level. Which Eric is already working on
(SACK compression is already in 4.18, with normal ACK compression to
follow).

-Toke

  reply	other threads:[~2018-06-26 11:16 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-21  4:58 Dave Taht
2018-06-21  9:22 ` Toke Høiland-Jørgensen
2018-06-21 12:55   ` Eric Dumazet
2018-06-21 15:18     ` Dave Taht
2018-06-21 15:31       ` Caleb Cushing
2018-06-21 15:46         ` Stephen Hemminger
2018-06-21 17:41           ` Caleb Cushing
2018-06-21 15:50         ` Dave Taht
2018-06-21 16:29       ` David Collier-Brown
2018-06-21 16:54         ` Jonathan Morton
2018-06-21 16:43       ` Kathleen Nichols
2018-06-21 19:17         ` Dave Taht
2018-06-21 19:41           ` Sebastian Moeller
2018-06-21 19:51             ` Toke Høiland-Jørgensen
2018-06-21 19:54             ` Dave Taht
2018-06-21 20:11               ` Sebastian Moeller
2018-06-22 14:01           ` Kathleen Nichols
2018-06-22 14:12             ` Jonathan Morton
2018-06-22 14:49               ` Michael Richardson
2018-06-22 15:02                 ` Jonathan Morton
2018-06-22 21:55                   ` Michael Richardson
2018-06-25 10:38                     ` Toke Høiland-Jørgensen
2018-06-25 23:54                       ` Jim Gettys
2018-06-26  0:07                         ` Jonathan Morton
2018-06-26  0:21                           ` David Lang
2018-06-26  0:36                             ` Simon Barber
2018-06-26  0:44                               ` Jonathan Morton
2018-06-26  0:52                                 ` Jim Gettys
2018-06-26  0:56                                 ` David Lang
2018-06-26 11:16                                   ` Toke Høiland-Jørgensen [this message]
2018-06-26  1:27                                 ` Dave Taht
2018-06-26  3:30                                   ` Simon Barber

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=87in65n6ft.fsf@toke.dk \
    --to=toke@toke.dk \
    --cc=bloat@lists.bufferbloat.net \
    --cc=chromatix99@gmail.com \
    --cc=david@lang.hm \
    /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