d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Properly propagate whether the channel is HT40 or not when calculating

Browse Source 
packet duration for the ath_rate_sample module.

This doesn't affect the packet TX at all; only how much time the
sample rate module attributes to a completed TX.
This commit is contained in:
Adrian Chadd 2011-02-17 05:16:59 +00:00
parent 39e692013a
commit e09c8c4cd4
2 changed files with 14 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
sys/dev/ath/ath_rate/sample/sample.c
View File

@ -426,18 +426,19 @@ update_stats(struct ath_softc *sc, struct ath_node *an,
const int size_bin = size_to_bin(frame_size);
const int size = bin_to_size(size_bin);
int tt, tries_so_far;
int is_ht40 = (an->an_node.ni_htcap & IEEE80211_HTCAP_CHWIDTH40);
if (!IS_RATE_DEFINED(sn, rix0))
return;
tt = calc_usecs_unicast_packet(sc, size, rix0, short_tries,
MIN(tries0, tries) - 1);
MIN(tries0, tries) - 1, is_ht40);
tries_so_far = tries0;
if (tries1 && tries_so_far < tries) {
if (!IS_RATE_DEFINED(sn, rix1))
return;
tt += calc_usecs_unicast_packet(sc, size, rix1, short_tries,
MIN(tries1 + tries_so_far, tries) - tries_so_far - 1);
MIN(tries1 + tries_so_far, tries) - tries_so_far - 1, is_ht40);
tries_so_far += tries1;
}
@ -445,7 +446,7 @@ update_stats(struct ath_softc *sc, struct ath_node *an,
if (!IS_RATE_DEFINED(sn, rix2))
return;
tt += calc_usecs_unicast_packet(sc, size, rix2, short_tries,
MIN(tries2 + tries_so_far, tries) - tries_so_far - 1);
MIN(tries2 + tries_so_far, tries) - tries_so_far - 1, is_ht40);
tries_so_far += tries2;
}
@ -453,7 +454,7 @@ update_stats(struct ath_softc *sc, struct ath_node *an,
if (!IS_RATE_DEFINED(sn, rix3))
return;
tt += calc_usecs_unicast_packet(sc, size, rix3, short_tries,
MIN(tries3 + tries_so_far, tries) - tries_so_far - 1);
MIN(tries3 + tries_so_far, tries) - tries_so_far - 1, is_ht40);
}
if (sn->stats[size_bin][rix0].total_packets < ssc->smoothing_minpackets) {
@ -765,7 +766,8 @@ ath_rate_ctl_reset(struct ath_softc *sc, struct ieee80211_node *ni)
if ((mask & 1) == 0)
continue;
printf(" %d/%d", dot11rate(rt, rix),
calc_usecs_unicast_packet(sc, 1600, rix, 0,0));
calc_usecs_unicast_packet(sc, 1600, rix, 0,0,
(ni->ni_htcap & IEEE80211_HTCAP_CHWIDTH40)));
}
printf("\n");
}
@ -794,7 +796,8 @@ ath_rate_ctl_reset(struct ath_softc *sc, struct ieee80211_node *ni)
sn->stats[y][rix].last_tx = 0;
sn->stats[y][rix].perfect_tx_time =
calc_usecs_unicast_packet(sc, size, rix, 0, 0);
calc_usecs_unicast_packet(sc, size, rix, 0, 0,
(ni->ni_htcap & IEEE80211_HTCAP_CHWIDTH40));
sn->stats[y][rix].average_tx_time =
sn->stats[y][rix].perfect_tx_time;
}

8
sys/dev/ath/ath_rate/sample/sample.h
View File

@ -115,7 +115,9 @@ struct sample_node {
*/
static unsigned calc_usecs_unicast_packet(struct ath_softc *sc,
int length,
int rix, int short_retries, int long_retries) {
int rix, int short_retries,
int long_retries, int is_ht40)
{
const HAL_RATE_TABLE *rt = sc->sc_currates;
struct ifnet *ifp = sc->sc_ifp;
struct ieee80211com *ic = ifp->if_l2com;
@ -198,7 +200,7 @@ static unsigned calc_usecs_unicast_packet(struct ath_softc *sc,
/* XXX assumes short preamble */
/* XXX assumes HT/20; the node info isn't yet available here */
ctsduration += ath_hal_pkt_txtime(sc->sc_ah, rt, length, rix, 0, AH_TRUE);
ctsduration += ath_hal_pkt_txtime(sc->sc_ah, rt, length, rix, 0, is_ht40);
if (cts) /* SIFS + ACK */
ctsduration += rt->info[cix].spAckDuration;
@ -209,7 +211,7 @@ static unsigned calc_usecs_unicast_packet(struct ath_softc *sc,
/* XXX assumes short preamble */
/* XXX assumes HT/20; the node info isn't yet available here */
tt += (long_retries+1)*ath_hal_pkt_txtime(sc->sc_ah, rt, length, rix, 0, AH_TRUE);
tt += (long_retries+1)*ath_hal_pkt_txtime(sc->sc_ah, rt, length, rix, 0, is_ht40);
tt += (long_retries+1)*(t_sifs + rt->info[rix].spAckDuration);
for (x = 0; x <= short_retries + long_retries; x++) {