[PATCH] mac80211: Always check rates and capabilities in mesh mode

[PATCH] mac80211: Always check rates and capabilities in mesh mode

[Alexis Green]

In mesh mode there is a race between establishing links and processing rates
and capabilities in beacons. This is very noticeable with slow beacons (e.g.
beacon intervals of 1s) and manifested for us as stations using minstrel when
minstrel_ht should be used. Fixed by changing mesh_sta_info_init so that it
always checks rates and such if it has not already done so.

Signed-off-by: Alexis Green <agreen@cococorp.com>
CC: Jesse Jones <jjones@cococorp.com>

---

diff --git a/net/mac80211/mesh_plink.c b/net/mac80211/mesh_plink.c
index ac843fc..4f9f23c 100644
--- a/net/mac80211/mesh_plink.c
+++ b/net/mac80211/mesh_plink.c
@@ -394,8 +394,9 @@ static void mesh_sta_info_init(struct
ieee80211_sub_if_data *sdata,
        sta->last_rx = jiffies;

        /* rates and capabilities don't change during peering */
-       if (sta->plink_state == NL80211_PLINK_ESTAB)
+       if (sta->plink_state == NL80211_PLINK_ESTAB && sta->processedBeacon)
                goto out;
+       sta->processedBeacon = true;

        if (sta->sta.supp_rates[band] != rates)
                changed |= IEEE80211_RC_SUPP_RATES_CHANGED;
diff --git a/net/mac80211/sta_info.h b/net/mac80211/sta_info.h
index 9bd1e97..e3ae2b0 100644
--- a/net/mac80211/sta_info.h
+++ b/net/mac80211/sta_info.h
@@ -369,6 +369,8 @@ struct ieee80211_fast_tx {
  * @rx_msdu: MSDUs received from this station, using IEEE80211_NUM_TID
  *     entry for non-QoS frames
  * @fast_tx: TX fastpath information
+ * @processedBeacon: set to true after peer rates and capabilities are
+ *     processed
  */
 struct sta_info {
        /* General information, mostly static */
@@ -473,6 +475,7 @@ struct sta_info {
        enum nl80211_mesh_power_mode local_pm;
        enum nl80211_mesh_power_mode peer_pm;
        enum nl80211_mesh_power_mode nonpeer_pm;
+       bool processedBeacon;
 #endif

 #ifdef CONFIG_MAC80211_DEBUGFS

Re: [PATCH] mac80211: Always check rates and capabilities in mesh mode

[Johannes Berg]

On Fri, 2015-05-29 at 15:24 -0700, Alexis Green wrote:
> In mesh mode there is a race between establishing links and processing rates
> and capabilities in beacons. This is very noticeable with slow beacons (e.g.
> beacon intervals of 1s) and manifested for us as stations using minstrel when
> minstrel_ht should be used. Fixed by changing mesh_sta_info_init so that it
> always checks rates and such if it has not already done so.
> 
> Signed-off-by: Alexis Green <agreen@cococorp.com>
> CC: Jesse Jones <jjones@cococorp.com>

This patch doesn't apply, it's line-broken and whitespace damaged.

johannes

Re: [PATCH] mac80211: Always check rates and capabilities in mesh mode

[Alexis Green]

Will resend using a better mail client. Just found out that web-gmail is no-go.

On Tue, Jun 9, 2015 at 12:44 PM, Johannes Berg
<johannes@sipsolutions.net> wrote:
> On Fri, 2015-05-29 at 15:24 -0700, Alexis Green wrote:
>> In mesh mode there is a race between establishing links and processing rates
>> and capabilities in beacons. This is very noticeable with slow beacons (e.g.
>> beacon intervals of 1s) and manifested for us as stations using minstrel when
>> minstrel_ht should be used. Fixed by changing mesh_sta_info_init so that it
>> always checks rates and such if it has not already done so.
>>
>> Signed-off-by: Alexis Green <agreen@cococorp.com>
>> CC: Jesse Jones <jjones@cococorp.com>
>
> This patch doesn't apply, it's line-broken and whitespace damaged.
>
> johannes
>

Re: [PATCH] mac80211: Always check rates and capabilities in mesh mode

[Kalle Valo]

Alexis Green <agreen@cococorp.com> writes:

> Will resend using a better mail client. Just found out that web-gmail is no-go.

Just use git send-email, anything else is waste of time IMHO.

-- 
Kalle Valo