On Fri, May 11, 2012 at 10:31:49AM -0700, Seth Forshee wrote: > > > I've sent RFC patches that strip out the vast majority of this code in > > > favor of better integration with the protocol-level regulatory support. > > > The piece that's under discussion here is a behavior I maintained in my > > > changes, which mutes tx on passive scan channels until data is received > > > on a given channel. But in my opinion, if this sort of behavior is > > > desired it ought to be implemented at the protocol level instead of in > > > the driver, so I'd really prefer to see it removed from brcmsmac. > > > > This isn't implemented in mac80211, and since some HW (e.g. ours) > > implements it in lower levels, I don't think you can just generally say > > it has to be in mac80211 now. > > I guess I need to familiarize myself with the situation for other > hardware then, which I will do next week when I'm not at a conference. > It just seems to me that if we want to enforce consistent regulatory > behavior for all HW then mac80211 is the logical place to implement it. I had a look at iwlwifi, and I see how passive channels are being handled. I don't understand though how having this functionality in mac80211 would hurt iwlwifi, but perhaps we can find an acceptible solution. Actually the support already present for regulatory beacon hints would probably suit this purpose fairly well, and in fact it already does seem to work for some channels. But for reasons I don't understand the beacon hints are only processed for channels with IEEE80211_CHAN_RADAR cleared, making it ineffective for most of the 5 GHz band. If this was changed to at least give the driver some indication that a beacon has been seen on the channel (even just setting chan->beacon_found) then drivers should be able to use it to know when tx can be enabled on passive channels. Cheers, Seth -- 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
