On Thu, Sep 11, 2008 at 06:22:31AM -0700, Johannes Berg wrote: > On Thu, 2008-09-11 at 13:59 +0530, Sujith wrote: > > > > I'd been talking with Sujith about this one: > > > > > > * mac80211: share STA information with driver > > > > > > and decided to just do it; as it stands it's not very useful > > > but I expect it to be used soon, if not we can remove the > > > EXPORT_SYMBOL again. > > > > > > > This would be very useful for ath9k. Making use of the > > private area for each STA can enable removal of the node list > > that is maintained currently. > > I had actually tried > (http://johannes.sipsolutions.net/patches/kernel/ath9k-sta-node.patch) > but you're using refcounting for the nodes while mac80211 RCU-protects > them, so I didn't get very far. Good thing I read RCU docs yesterday. We should be able to remove ref counts and simply use RCU and sychronize_rcu() before deletion. > It needs a bit more effort to make sure > you don't have node pointers stick around in some tx descriptor after > mac80211 decides to remove a node (which may very well happen while > frames are queued) We can call_rcu() here no, and let the driver deal with what it needs to prior to deletion? 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
