RE: [PATCH 3/4] mac80211: mesh: fixed HT ies in beacon template

From: Machani, Yaniv
Date: Wed Jul 13 2016 - 07:16:03 EST


On Wed, Jun 29, 2016 at 10:17:35, Johannes Berg wrote:
> Cc: Kama, Meirav
> Subject: Re: [PATCH 3/4] mac80211: mesh: fixed HT ies in beacon
> template
>
> On Tue, 2016-06-28 at 14:13 +0300, Yaniv Machani wrote:
> >
> > Ânet/mac80211/mesh.c | 33 ++++++++++++++++++++++++++++++++-
> > Ânet/mac80211/util.c |ÂÂ3 ---
> > Ânet/wireless/mesh.c |ÂÂ2 +-
>
> That's not a good patch - one change is mac80211 and the other cfg80211.
>
> > - .ht_opmode =
> IEEE80211_HT_OP_MODE_PROTECTION_NONHT_MIXED,
> > + .ht_opmode = IEEE80211_HT_OP_MODE_PROTECTION_NONE,
> >
> How are you planning to comply with 802.11 now?

Good point, this changed should be removed.
The reason for this change was that we've noticed a difference between mesh beacon (built by the mac80211) and mesh actions (built by the supplicant) in the HT information IE.
In beacons the HT operational mode is Mixed Mode (0x11) while in actions it is None (0x00).
After a second look, it seems that it's the Supplicant that doesn't set the default value correctly.

We'll send an updated patch for it.
Thanks,
Yaniv

>
> The HT Protection field in a mesh STA may be set to no protection mode
> only if â All STAs detected in the primary or the secondary channel
> are HT
> Â STAs, and
> â All mesh STA members of this MBSS that are one-hop neighbors of the
> Â transmitting mesh STA are either:
> Â â 20/40 MHz HT mesh STAs in a 20/40 MHz MBSS, or
> Â â 20 MHz HT mesh STAs in a 20 MHz MBSS.
>
> johannes