From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751368AbcGMLPz (ORCPT ); Wed, 13 Jul 2016 07:15:55 -0400 Received: from arroyo.ext.ti.com ([198.47.19.12]:60537 "EHLO arroyo.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751098AbcGMLPp (ORCPT ); Wed, 13 Jul 2016 07:15:45 -0400 From: "Machani, Yaniv" To: Johannes Berg , "linux-kernel@vger.kernel.org" , "David S . Miller" , "linux-wireless@vger.kernel.org" , "netdev@vger.kernel.org" CC: "Kama, Meirav" Subject: RE: [PATCH 3/4] mac80211: mesh: fixed HT ies in beacon template Thread-Topic: [PATCH 3/4] mac80211: mesh: fixed HT ies in beacon template Thread-Index: AQHR0dZPssrI1RK+CkS9EOpOJzadKaAWOjFA Date: Wed, 13 Jul 2016 11:15:35 +0000 Message-ID: References: <20160628111307.8784-1-yanivma@ti.com> <20160628111307.8784-2-yanivma@ti.com> <20160628111307.8784-3-yanivma@ti.com> <20160628111307.8784-4-yanivma@ti.com> (sfid-20160628_131116_981554_64C22D92) <1467184655.2461.2.camel@sipsolutions.net> In-Reply-To: <1467184655.2461.2.camel@sipsolutions.net> Accept-Language: en-US, he-IL Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.167.188.42] x-exclaimer-md-config: f9c360f5-3d1e-4c3c-8703-f45bf52eff6b Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id u6DBGLkI002882 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