Lets make wifi fast again!
 help / color / mirror / Atom feed
From: Dave Taht <dave.taht@gmail.com>
To: make-wifi-fast@lists.bufferbloat.net
Subject: [Make-wifi-fast] [PATCH] killing retries harder, perhaps
Date: Thu, 2 Feb 2017 15:47:38 -0800	[thread overview]
Message-ID: <CAA93jw6+iP71PyR5=cHL-U_VaiMgMmb-wmPgKyE3cX+=s1P6=Q@mail.gmail.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 1177 bytes --]

ATH_TXMAXTRY is now unused.

ATH_MAX_SW_RETRIES is insanely high by default.

I finally have some good (well, horrible) data in a consistently noisy
environment and would like to try knocking this down a little in a
future test run. There's smarter ways to do it (like, if you are at
the lowest rate, retry once, higher rates, 4).

I have tried in the past values as low as 2 for soft retries.

diff --git a/drivers/net/wireless/ath/ath9k/ath9k.h
b/drivers/net/wireless/ath/ath9k/ath9k.h
index 331947b..304976d 100644
--- a/drivers/net/wireless/ath/ath9k/ath9k.h
+++ b/drivers/net/wireless/ath/ath9k/ath9k.h
@@ -91,8 +91,7 @@ int ath_descdma_setup(struct ath_softc *sc, struct
ath_descdma *dd,
 #define ATH_RXBUF               512
 #define ATH_TXBUF               512
 #define ATH_TXBUF_RESERVE       5
-#define ATH_TXMAXTRY            13
-#define ATH_MAX_SW_RETRIES      30
+#define ATH_MAX_SW_RETRIES      4

 #define TID_TO_WME_AC(_tid)                            \
        ((((_tid) == 0) || ((_tid) == 3)) ? IEEE80211_AC_BE :   \



-- 
Dave Täht
Let's go make home routers and wifi faster! With better software!
http://blog.cerowrt.org

[-- Attachment #2: reduce_retry_insanity.diff --]
[-- Type: text/plain, Size: 609 bytes --]

diff --git a/drivers/net/wireless/ath/ath9k/ath9k.h b/drivers/net/wireless/ath/ath9k/ath9k.h
index 331947b..304976d 100644
--- a/drivers/net/wireless/ath/ath9k/ath9k.h
+++ b/drivers/net/wireless/ath/ath9k/ath9k.h
@@ -91,8 +91,7 @@ int ath_descdma_setup(struct ath_softc *sc, struct ath_descdma *dd,
 #define ATH_RXBUF               512
 #define ATH_TXBUF               512
 #define ATH_TXBUF_RESERVE       5
-#define ATH_TXMAXTRY            13
-#define ATH_MAX_SW_RETRIES      30
+#define ATH_MAX_SW_RETRIES      4
 
 #define TID_TO_WME_AC(_tid)				\
 	((((_tid) == 0) || ((_tid) == 3)) ? IEEE80211_AC_BE :	\

             reply	other threads:[~2017-02-02 23:47 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-02-02 23:47 Dave Taht [this message]
2017-02-03  0:37 ` Dave Taht
2017-02-03 13:21 ` Toke Høiland-Jørgensen
2017-02-03 19:30   ` Dave Taht
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='CAA93jw6+iP71PyR5=cHL-U_VaiMgMmb-wmPgKyE3cX+=s1P6=Q@mail.gmail.com' \
    --to=dave.taht@gmail.com \
    --cc=make-wifi-fast@lists.bufferbloat.net \
    /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