On Mon, Jan 28, 2008 at 05:19:37PM +0100, Johannes Berg wrote: > This patch removes the 802.1X port acess control enable flag > since it is not required. Instead, set the authorized flag for > each station that we normally communicate with (WDS peers, IBSS > peers and APs we're associated to) and require hostapd to set > the authorized flag for all stations when port control is not > enabled. OK, that sounds reasonable. > Also, since I was working in that area, this documents station > flags and removes the unused "permanent" one. And changes the flags.. These used to be shared with hostapd. Has that been abstracted so that there is some kind of mapping taking care of the differences between hostapd and mac80211? If not, this change does not look safe.. > -#define WLAN_STA_AUTHORIZED BIT(5) /* If 802.1X is used, this flag is > - * controlling whether STA is authorized to > - * send and receive non-IEEE 802.1X frames > - */ > -#define WLAN_STA_SHORT_PREAMBLE BIT(7) > -/* whether this is an AP that we are associated with as a client */ > -#define WLAN_STA_ASSOC_AP BIT(8) > -#define WLAN_STA_WME BIT(9) > -#define WLAN_STA_WDS BIT(27) > + WLAN_STA_AUTHORIZED = 1<<4, > + WLAN_STA_SHORT_PREAMBLE = 1<<5, > + WLAN_STA_ASSOC_AP = 1<<6, > + WLAN_STA_WME = 1<<7, > + WLAN_STA_WDS = 1<<8, hostapd uses at least authorized, short_preamble, and WME flags and this change would make these not match with kernel side anymore.. It may be desirable to break this direct link between the bitfield, but set_flags operation needs to be mapped properly in that case and I don't know whether that has already been taken care of in mac80211. -- Jouni Malinen PGP id EFC895FA - 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
