On Tue, 2011-12-13 at 16:29 -0800, Thomas Pedersen wrote: > On Tue, Dec 13, 2011 at 12:07 PM, Johannes Berg > <johannes@xxxxxxxxxxxxxxxx> wrote: > > From: Johannes Berg <johannes.berg@xxxxxxxxx> > > > > Station entries can have various states, the most > > important ones being auth, assoc and authorized. > > This patch prepares us for telling the driver about > > these states, we don't want to confuse drivers with > > strange transitions, so with this we enforce that > > they move in the right order between them (back and > > forth); some transitions might happen before the > > driver even knows about the station, but at least > > runtime transitions will be ordered correctly. > > > > As a consequence, IBSS and MESH stations will now > > have the ASSOC flag set (so they can transition to > > AUTHORIZED), and we can get rid of a special case > > in TX processing. > > > > TBD: WLAN_STA_TDLS_PEER_AUTH? > > TBD: secure mesh AUTH transitions? > > This patch makes it impossible to add a new unauthenticated station, > AFAICT. Maybe this is not even desirable from mac80211's POV? The > reason the secure mesh daemon adds a new unauthenticated peer is to > stop NL80211_NEW_PEER_CANDIDATE notifications while userspace is > handling the authentication process, and we need a station entry for > ieee80211_mgmt_tx(). Ah, yes. I had to sleep over this -- now I understand why I was confused. I'll amend the patch series to properly manage more of this in cfg80211. I have a feeling we should also allow userspace to manage the ASSOC state, but can we require it from the secure mesh userspace daemon? It would break backward compatibility if we do ... I'll try to find a workaround :) johannes -- 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
