On 11 May 2015 at 14:01, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote:
> On Mon, 2015-05-11 at 13:52 +0200, Janusz Dziedzic wrote:
>> On 11 May 2015 at 11:53, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote:
>> > On Mon, 2015-05-11 at 11:52 +0200, Johannes Berg wrote:
>> >> On Mon, 2015-05-11 at 11:39 +0200, Janusz Dziedzic wrote:
>> >> > This is required to handle sta.wme correctly
>> >> > after update rates (eg. when using HT/VHT) when
>> >> > driver using IEEE80211_HW_SUPPORT_FAST_XMIT.
>> >>
>> >> Wouldn't you want to check, so it can be re-assigned?
>> >
>> > i.e. something like
>> >
>> > diff --git a/net/mac80211/ibss.c b/net/mac80211/ibss.c
>> > index bfef1b215050..7e62183525aa 100644
>> > --- a/net/mac80211/ibss.c
>> > +++ b/net/mac80211/ibss.c
>> > @@ -1031,8 +1031,11 @@ static void ieee80211_update_sta_info(struct ieee80211_sub_if_data *sdata,
>> > }
>> > }
>> >
>> > - if (sta && elems->wmm_info && local->hw.queues >= IEEE80211_NUM_ACS)
>> > + if (sta && !sta->sta.wme && elems->wmm_info &&
>> > + local->hw.queues >= IEEE80211_NUM_ACS) {
>> > sta->sta.wme = true;
>> > + ieee80211_check_fast_xmit(sta);
>> > + }
>> >
>> > if (sta && elems->ht_operation && elems->ht_cap_elem &&
>> > sdata->u.ibss.chandef.width != NL80211_CHAN_WIDTH_20_NOHT &&
>> >
>> Your version also works fine.
>> Should I send new patch or you already have this in your tree?
>
> I didn't put it in, but can do that.
>
Yes, please do that.
BR
Janusz
--
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
