On 10/25/07, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > This patch implements the cfg80211 hooks for configuring beaconing > on an access point interface in mac80211. While doing so, it fixes > a number of races that could badly crash the machine when the > beacon is changed while being requested by the driver. > > Signed-off-by: Johannes Berg <johannes@xxxxxxxxxxxxxxxx> > > --- > The dtim_count field should possibly also be part of the beacon > structure, but the possible race there doesn't really matter, > worst thing is that one beacon will be sent with a wrong dtim > count if (and only if) userspace changes the dtim period during > operation. > > net/mac80211/cfg.c | 108 +++++++++++++++++++++++++++++++++++++++++ > net/mac80211/debugfs_netdev.c | 27 ---------- > net/mac80211/ieee80211_i.h | 14 +++-- > net/mac80211/ieee80211_iface.c | 4 - > net/mac80211/tx.c | 63 ++++++++++++++--------- > 5 files changed, 156 insertions(+), 60 deletions(-) > > --- linux-2.6.orig/net/mac80211/cfg.c 2007-10-24 11:35:49.722690568 +0200 > +++ linux-2.6/net/mac80211/cfg.c 2007-10-25 10:46:56.070256998 +0200 > @@ -10,6 +10,7 @@ > #include <linux/nl80211.h> > #include <linux/rtnetlink.h> > #include <net/net_namespace.h> > +#include <linux/rcupdate.h> > #include <net/cfg80211.h> > #include "ieee80211_i.h" > #include "cfg.h" > @@ -269,6 +270,112 @@ static int ieee80211_config_default_key( > return 0; > } > > +static int ieee80211_set_beacon(struct wiphy *wiphy, struct net_device *dev, > + struct beacon_parameters *beacon) To be consistent how about s/beacon/params here for beacon_parameters. Luis - 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
