[PATCH 3/3] mac80211: recalc min_def chanctx even when chandef is identical

[Emmanuel Grumbach]

From: Arik Nemtsov 

The min_def chanctx is affected not only by the current chandef, but
sometimes also by other stations on the vif. There's a valid scenario
where a TDLS peer can widen its BW, thereby causing the min_def
to increase.

Signed-off-by: Arik Nemtsov 
Signed-off-by: Emmanuel Grumbach 
---
 net/mac80211/chan.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/net/mac80211/chan.c b/net/mac80211/chan.c
index 2839811..74142d0 100644
--- a/net/mac80211/chan.c
+++ b/net/mac80211/chan.c
@@ -343,8 +343,10 @@ static void ieee80211_change_chanctx(struct 
ieee80211_local *local,
 struct ieee80211_chanctx *ctx,
 const struct cfg80211_chan_def *chandef)
 {
-   if (cfg80211_chandef_identical(&ctx->conf.def, chandef))
+   if (cfg80211_chandef_identical(&ctx->conf.def, chandef)) {
+   ieee80211_recalc_chanctx_min_def(local, ctx);
return;
+   }
 
WARN_ON(!cfg80211_chandef_compatible(&ctx->conf.def, chandef));
 
-- 
2.5.0

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 3/3] mac80211: recalc min_def chanctx even when chandef is identical

[Johannes Berg]

All three applied, but I had to fix your Fixes tag commit ID, no idea
what that referred to :)

johannes
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 3/3] mac80211: recalc min_def chanctx even when chandef is identical

[Arik Nemtsov]

On Thu, Mar 3, 2016 at 5:40 PM, Johannes Berg  wrote:
> All three applied, but I had to fix your Fixes tag commit ID, no idea
> what that referred to :)

Yea I might have taken it from some internal tree :)

Arik
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 3/3] mac80211: recalc min_def chanctx even when chandef is identical

[Johannes Berg]

On Thu, 2016-03-03 at 17:41 +0200, Arik Nemtsov wrote:
> On Thu, Mar 3, 2016 at 5:40 PM, Johannes Berg  net> wrote:
> > All three applied, but I had to fix your Fixes tag commit ID, no
> > idea
> > what that referred to :)
> 
> Yea I might have taken it from some internal tree :)
> 

Oh, actually what prompted me to look at all was that it was too short
- should be 12 hex digits.

johannes
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html