On Fri, 2012-11-23 at 12:18 -0800, Marco Porsch wrote:
> Move the default mesh beacon interval and DTIM period to cfg80211 and
> make it accessible to nl80211. This enables setting both values when joining
> an MBSS.
Trivial notes for a couple of patches
> diff --git a/include/net/cfg80211.h b/include/net/cfg80211.h
[]
> @@ -937,6 +939,8 @@ struct mesh_setup {
> bool is_authenticated;
> bool is_secure;
> int mcast_rate[IEEE80211_NUM_BANDS];
> + u16 beacon_interval;
> + u8 dtim_period;
It might be better to move the new u8 after the last bool
to avoid unnecessary padding.
bool is_secure;
u8 dtim_period;
u16 beacon_interval;
int mcast_rate[IEEE80211_NUM_BANDS];
and for patch 3/13:
diff --git a/net/mac80211/mesh_ps.c b/net/mac80211/mesh_ps.c
[]
> +void ieee80211_mesh_local_ps_update(struct ieee80211_sub_if_data *sdata)
> +{
[]
> + list_for_each_entry_rcu(sta, &sdata->local->sta_list, list) {
[]
> + case NL80211_PLINK_ESTAB:
> + if (sta->local_ps_mode ==
> + NL80211_MESH_POWER_LIGHT_SLEEP)
> + light_sleep_cnt++;
> + if (sta->local_ps_mode ==
> + NL80211_MESH_POWER_DEEP_SLEEP)
else if
> + deep_sleep_cnt++;
> + break;
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
