Lets make wifi fast again!
 help / color / mirror / Atom feed
From: Dave Taht <dave.taht@gmail.com>
To: "Toke Høiland-Jørgensen" <toke@toke.dk>
Cc: make-wifi-fast@lists.bufferbloat.net
Subject: Re: [Make-wifi-fast] [PATCH] killing retries harder, perhaps
Date: Fri, 3 Feb 2017 11:30:31 -0800	[thread overview]
Message-ID: <CAA93jw57qqXFz2BMfqxMkE=OA=8RGdsT4R7ZFOf5xtDsg7Zz8w@mail.gmail.com> (raw)
In-Reply-To: <87poiz1kcg.fsf@toke.dk>

The net effect of
http://mirrors.bufferbloat.net/~kevindb/999-daves-special-patch.patch
of:

#define ATH_NON_AGGR_MIN_QDEPTH    2
#define ATH_MAX_SW_RETRIES      2
#define ATH_TXFIFO_DEPTH           3

was to disable ampdu generation entirely, and ship pure MPDUs instead,
which gives me a solid bound of about 15Mbits transfers in my noisy
tests (where I'd see anywhere from 4Mbit to 80mbit with ampdus).

http://pastebin.com/WU390GFL

I note: I have *always* wanted to measure these with aggregation
completely out of the loop, so this is kind of inadvertently awesome.

root@archer-atf:/sys/kernel/debug/ieee80211/phy1/ath9k# cat xmit
                            BE         BK        VI        VO

MPDUs Queued:                0          0         0      1234
MPDUs Completed:       6402806          0         0      1234
MPDUs XRetried:             19          0         0         0
Aggregates:                  0          0         0         0
AMPDUs Queued HW:            0          0         0         0
AMPDUs Completed:            0          0         0         0
AMPDUs Retried:              0          0         0         0
AMPDUs XRetried:             0          0         0         0
TXERR Filtered:            115          0         0         0
FIFO Underrun:               0          0         0         0
TXOP Exceeded:               0          0         0         0
TXTIMER Expiry:              0          0         0         0
DESC CFG Error:              0          0         0         0
DATA Underrun:               0          0         0         0
DELIM Underrun:              0          0         0         0
TX-Pkts-All:           6402825          0         0      1234
TX-Bytes-All:       1171739830          0         0    178778
HW-put-tx-buf:         3213134          0         0      1233
HW-tx-start:                 0          0         0         0
HW-tx-proc-desc:       6402825          0         0      1234
TX-Failed:                   0          0         0         0




http://pastebin.com/WU390GFL

  reply	other threads:[~2017-02-03 19:30 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-02-02 23:47 Dave Taht
2017-02-03  0:37 ` Dave Taht
2017-02-03 13:21 ` Toke Høiland-Jørgensen
2017-02-03 19:30   ` Dave Taht [this message]
2017-02-03 23:07     ` 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/make-wifi-fast.lists.bufferbloat.net/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAA93jw57qqXFz2BMfqxMkE=OA=8RGdsT4R7ZFOf5xtDsg7Zz8w@mail.gmail.com' \
    --to=dave.taht@gmail.com \
    --cc=make-wifi-fast@lists.bufferbloat.net \
    --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