On Mon, 2011-02-28 at 17:40 +0100, Bernhard Schmidt wrote: > Implemented so far is the complete state machine, handling CAC and NOL > as well as starting appropriate action in hostapd. Note, this is based > on top of the 'dfs region support' work posted by Luis. I'm not quite sure about all of this. First of all, you don't really explain the global state vs. local state but you did say that you no longer had global state, which clearly isn't true. Also, what's the point in "struct radar" if there's only a single instance of that, which is global? I'd rather have multiple global variables I think -- but that might just be a matter of personal taste or choice. Finally, I think your code split is a little hard to understand. Is there really any point in adding the code bit by bit into both cfg80211 and mac80211 at the same time? I'd rather see a few patches to make cfg80211 complete and then implement it all in mac80211. Which brings me to another point -- there's no way to detect from userspace whether or not this is all available. Finally, such details like not needing a timer -- a delayed work should be sufficient, etc. 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
