diff options
| author | Adrian Chadd <adrian@FreeBSD.org> | 2012-03-09 22:58:34 +0000 |
|---|---|---|
| committer | Adrian Chadd <adrian@FreeBSD.org> | 2012-03-09 22:58:34 +0000 |
| commit | c5940c30a7fcb5f49c1c556d5bce22fb7261d083 (patch) | |
| tree | ecb768f5e034ab6edf171066534e233b28db45e1 | |
| parent | 91d92caece5ce014a944d5bfd95d8b26f08704ff (diff) | |
Notes
| -rw-r--r-- | sys/dev/ath/if_ath_tx.c | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/sys/dev/ath/if_ath_tx.c b/sys/dev/ath/if_ath_tx.c index f7305c37b308..931403470565 100644 --- a/sys/dev/ath/if_ath_tx.c +++ b/sys/dev/ath/if_ath_tx.c @@ -1162,6 +1162,11 @@ ath_tx_normal_setup(struct ath_softc *sc, struct ieee80211_node *ni, * (or) if there is some mcast data waiting on the mcast * queue (to prevent out of order delivery) multicast * frames must be buffered until after the beacon. + * + * XXX This likely means that if there's a station in power + * save mode, we won't be doing any kind of aggregation towards + * anyone. This is likely a very suboptimal way of dealing + * with things. */ if (ismcast && (vap->iv_ps_sta || avp->av_mcastq.axq_depth)) txq = &avp->av_mcastq; @@ -1409,6 +1414,12 @@ ath_tx_start(struct ath_softc *sc, struct ieee80211_node *ni, if (ismcast) txq = &avp->av_mcastq; + /* + * XXX This likely means that if there's a station in power + * save mode, we won't be doing any kind of aggregation towards + * anyone. This is likely a very suboptimal way of dealing + * with things. + */ if ((! is_ampdu) && (vap->iv_ps_sta || avp->av_mcastq.axq_depth)) txq = &avp->av_mcastq; |
