General list for discussing Bufferbloat
 help / color / mirror / Atom feed
From: Bob McMahon <bob.mcmahon@broadcom.com>
To: "Toke Høiland-Jørgensen" <toke@toke.dk>
Cc: Neal Cardwell <ncardwell@google.com>,
	 Make-Wifi-fast <make-wifi-fast@lists.bufferbloat.net>,
	 BBR Development <bbr-dev@googlegroups.com>,
	bloat <bloat@lists.bufferbloat.net>
Subject: Re: [Bloat] [bbr-dev] Aggregating without bloating - hard times for tcp and wifi
Date: Tue, 22 Nov 2022 13:00:14 -0800	[thread overview]
Message-ID: <CAHb6Lvp5kKz-WijP=kurKLR=Z9WeAC0BERpPPjkGrF8ExjbYmQ@mail.gmail.com> (raw)
In-Reply-To: <87r0xuvhgp.fsf@toke.dk>


[-- Attachment #1.1: Type: text/plain, Size: 2816 bytes --]

Does the TSQ code honor no-aggregation per voice access class or
TCP_NODELAY where the app making the socket write calls knows that the WiFi
aggregation isn't likely helpful? Sorry, my Linux stack expertise is quite
limited.

Bob

On Tue, Nov 22, 2022 at 12:53 PM Toke Høiland-Jørgensen <toke@toke.dk>
wrote:

> Neal Cardwell via Bloat <bloat@lists.bufferbloat.net> writes:
>
> > On Tue, Nov 22, 2022 at 2:43 PM 'Bob McMahon' via BBR Development <
> > bbr-dev@googlegroups.com> wrote:
> >
> >> Thanks for sharing this. Curious about how the xTSQ value can be set?
> Can
> >> it be done with sysctl?
> >>
> >> *We continue our analysis by using the ms-version of TSQ patch, which
> >> enables the tune of the TSQ size allowing each TCP variant to enqueue
> more
> >> than 1 ms of data at the current TCP rate. In particular, we allow to
> >> enqueue the equivalent of x ms of data, naming each test xTSQ, with x
> being
> >> an integer value. It is important to notice that this patch has been
> >> included in the Linux kernel mainline, and each Wi-Fi driver can now set
> >> the desired xTSQ value**.*
> >>
> >
> > I believe they are setting the xTSQ value using the sk_pacing_shift
> field,
> > which was added here:
> >
> >
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=3a9b76fd0db9f0d426533f96a68a62a58753a51e
> >
> > AFAIK the intent is only for drivers to set that, and there's no sysctl
> for
> > that, but of course you could add a sysctl for testing if you wanted.
> > :-)
>
> Yup, indeed this is what mac80211 fiddles with:
> https://elixir.bootlin.com/linux/latest/source/net/mac80211/main.c#L739
> https://elixir.bootlin.com/linux/latest/source/net/mac80211/tx.c#L4156
>
> AFAICT, no in-tree drivers override the value set by mac80211.
>
> I believe the tests in that paper were conducted with this series
> applied:
>
> https://lore.kernel.org/all/20180105113256.14835-1-natale.patriciello@gmail.com/
>
> -Toke
>

-- 
This electronic communication and the information and any files transmitted 
with it, or attached to it, are confidential and are intended solely for 
the use of the individual or entity to whom it is addressed and may contain 
information that is confidential, legally privileged, protected by privacy 
laws, or otherwise restricted from disclosure to anyone else. If you are 
not the intended recipient or the person responsible for delivering the 
e-mail to the intended recipient, you are hereby notified that any use, 
copying, distributing, dissemination, forwarding, printing, or copying of 
this e-mail is strictly prohibited. If you received this e-mail in error, 
please return the e-mail to the sender, delete it from your computer, and 
destroy any printed copy of it.

[-- Attachment #1.2: Type: text/html, Size: 3984 bytes --]

[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 4206 bytes --]

  reply	other threads:[~2022-11-22 21:00 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-11-22  6:04 [Bloat] " Dave Taht
2022-11-22 19:42 ` [Bloat] [bbr-dev] " Bob McMahon
2022-11-22 20:03   ` [Bloat] [Make-wifi-fast] " David Lang
2022-11-22 20:13     ` Bob McMahon
2022-11-22 20:16       ` David Lang
2022-11-22 20:28         ` Bob McMahon
2022-11-22 20:48           ` Bob McMahon
2022-11-22 20:10   ` [Bloat] " Neal Cardwell
2022-11-22 20:53     ` Toke Høiland-Jørgensen
2022-11-22 21:00       ` Bob McMahon [this message]
2022-11-23 13:50         ` Toke Høiland-Jørgensen
2022-11-23 20:36           ` Bob McMahon
     [not found]     ` <003d01d8ffc5$2ace1a20$806a4e60$@umt.edu.pk>
2022-11-24 16:23       ` Dave Taht

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='CAHb6Lvp5kKz-WijP=kurKLR=Z9WeAC0BERpPPjkGrF8ExjbYmQ@mail.gmail.com' \
    --to=bob.mcmahon@broadcom.com \
    --cc=bbr-dev@googlegroups.com \
    --cc=bloat@lists.bufferbloat.net \
    --cc=make-wifi-fast@lists.bufferbloat.net \
    --cc=ncardwell@google.com \
    --cc=toke@toke.dk \
    /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