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 --]
next prev parent 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