diff options
| author | Stefan Sperling <stsp@cvs.openbsd.org> | 2017-04-26 07:53:18 +0000 |
|---|---|---|
| committer | Stefan Sperling <stsp@cvs.openbsd.org> | 2017-04-26 07:53:18 +0000 |
| commit | 7a10dab63a69a2484b8ca84ee55f85941943f738 (patch) | |
| tree | d96c519e3e48e95216a908f32d98fbdc148f1c78 /sys/dev/ic | |
| parent | 1b0bd69df455869b23bafc598a0d645e08fdc7bc (diff) | |
Tell MiRA about the actual number of retries instead of pretending that
a frame which was retried N times was retried just once.
This punishes retry-heavy data rates even more and should reduce latency.
It does not seem to have a noticable effect on net throughput in my testing.
If this change causes throughput problems for anyone, let me know.
ok tb@ mlarkin@
Diffstat (limited to 'sys/dev/ic')
| -rw-r--r-- | sys/dev/ic/ar5008.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/dev/ic/ar5008.c b/sys/dev/ic/ar5008.c index 805f4e0de5f..544a0e99321 100644 --- a/sys/dev/ic/ar5008.c +++ b/sys/dev/ic/ar5008.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ar5008.c,v 1.43 2017/03/08 12:02:41 mpi Exp $ */ +/* $OpenBSD: ar5008.c,v 1.44 2017/04/26 07:53:17 stsp Exp $ */ /*- * Copyright (c) 2009 Damien Bergamini <damien.bergamini@free.fr> @@ -1015,7 +1015,7 @@ ar5008_tx_process(struct athn_softc *sc, int qid) an->mn.ampdu_size = bf->bf_m->m_pkthdr.len + IEEE80211_CRC_LEN; an->mn.agglen = 1; /* XXX We do not yet support Tx agg. */ if (failcnt > 0) - an->mn.retries++; + an->mn.retries += failcnt; if (txfail) an->mn.txfail++; if (ic->ic_state == IEEE80211_S_RUN) { |