Return-path: Received: from mail-ee0-f51.google.com ([74.125.83.51]:35920 "EHLO mail-ee0-f51.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965450AbaCUN6L (ORCPT ); Fri, 21 Mar 2014 09:58:11 -0400 Received: by mail-ee0-f51.google.com with SMTP id c13so1853118eek.10 for ; Fri, 21 Mar 2014 06:58:10 -0700 (PDT) From: Michal Kazior To: linux-wireless@vger.kernel.org Cc: johannes@sipsolutions.net, Michal Kazior Subject: [PATCH v2 2/7] mac80211: make check_combinations() aware of chanctx reservation Date: Fri, 21 Mar 2014 14:52:16 +0100 Message-Id: <1395409941-26303-3-git-send-email-michal.kazior@tieto.com> (sfid-20140321_145819_508174_BFF949B2) In-Reply-To: <1395409941-26303-1-git-send-email-michal.kazior@tieto.com> References: <1395150804-24090-1-git-send-email-michal.kazior@tieto.com> <1395409941-26303-1-git-send-email-michal.kazior@tieto.com> Sender: linux-wireless-owner@vger.kernel.org List-ID: The ieee80211_check_combinations() computes radar_detect accordingly depending on chanctx reservation status. This makes it possible to use the function for channel_switch validation. Signed-off-by: Michal Kazior --- net/mac80211/util.c | 37 +++++++++++++++++++++++++++++++++++-- 1 file changed, 35 insertions(+), 2 deletions(-) diff --git a/net/mac80211/util.c b/net/mac80211/util.c index 7233102..59041d4 100644 --- a/net/mac80211/util.c +++ b/net/mac80211/util.c @@ -2802,6 +2802,40 @@ void ieee80211_recalc_dtim(struct ieee80211_local *local, ps->dtim_count = dtim_count; } +static u8 ieee80211_chanctx_radar_detect(struct ieee80211_local *local, + struct ieee80211_chanctx *ctx) +{ + struct ieee80211_sub_if_data *sdata; + struct ieee80211_chanctx_conf *conf; + u8 radar_detect = 0; + bool in_place = false; + + lockdep_assert_held(&local->chanctx_mtx); + + list_for_each_entry(sdata, &ctx->reserved_vifs, reserved_chanctx_list) { + if (sdata->reserved_radar_required) + radar_detect |= BIT(sdata->reserved_chandef.width); + + conf = rcu_dereference_protected(sdata->vif.chanctx_conf, + lockdep_is_held(&local->chanctx_mtx)); + if (conf == &ctx->conf) + in_place = true; + } + + /* if chanctx has in-place reservation then consider only the future + * radar_detect. multi-vif reservation is deferred so ignore assigned + * vifs. it is impossible for new vifs to be bound to an in-place + * reserved chanctx so consistency is guranteed */ + if (in_place) + return radar_detect; + + list_for_each_entry(sdata, &ctx->assigned_vifs, assigned_chanctx_list) + if (sdata->radar_required) + radar_detect |= BIT(sdata->vif.bss_conf.chandef.width); + + return radar_detect; +} + int ieee80211_check_combinations(struct ieee80211_sub_if_data *sdata, const struct cfg80211_chan_def *chandef, enum ieee80211_chanctx_mode chanmode, @@ -2843,8 +2877,7 @@ int ieee80211_check_combinations(struct ieee80211_sub_if_data *sdata, num[iftype] = 1; list_for_each_entry(ctx, &local->chanctx_list, list) { - if (ctx->conf.radar_enabled) - radar_detect |= BIT(ctx->conf.def.width); + radar_detect |= ieee80211_chanctx_radar_detect(local, ctx); if (ctx->mode == IEEE80211_CHANCTX_EXCLUSIVE) { num_different_channels++; continue; -- 1.8.5.3