On Thu, Nov 18, 2010 at 05:21:24PM -0800, Johannes Berg wrote:
> On Thu, 18 Nov 2010 17:18:54 -0800, "Luis R. Rodriguez"
> <lrodriguez@xxxxxxxxxxx> wrote:
> > This should help with latency issues which can happen when
> > using aggregation.
> >
> > Cc: Matt Smith <matt.smith@xxxxxxxxxxx>
> > Cc: Senthil Balasubramanian <senthilkumar@xxxxxxxxxxx>
> > Signed-off-by: Luis R. Rodriguez <lrodriguez@xxxxxxxxxxx>
> > ---
> >
> > We discussed this a while ago but never to around to it.
> >
> > net/mac80211/tx.c | 3 ++-
> > 1 files changed, 2 insertions(+), 1 deletions(-)
> >
> > diff --git a/net/mac80211/tx.c b/net/mac80211/tx.c
> > index e694836..405ddc9 100644
> > --- a/net/mac80211/tx.c
> > +++ b/net/mac80211/tx.c
> > @@ -1225,7 +1225,8 @@ ieee80211_tx_prepare(struct ieee80211_sub_if_data
> > *sdata,
> > tx->sta = sta_info_get(sdata, hdr->addr1);
> >
> > if (tx->sta && ieee80211_is_data_qos(hdr->frame_control) &&
> > - (local->hw.flags & IEEE80211_HW_AMPDU_AGGREGATION)) {
> > + (local->hw.flags & IEEE80211_HW_AMPDU_AGGREGATION) &&
> > + (ieee802_1d_to_ac[skb->priority] != IEEE80211_AC_VO)) {
>
> wtf? total NACK. You're setting up an aggregation session and then not
> using it? That's totally deluded ... just don't set up the session!!
:) thanks for the kind review.
Luis
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
