On 30 January 2015 at 14:10, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Fri, 2015-01-30 at 14:06 +0100, Michal Kazior wrote: > >> I'm a bit reluctant to freely use mac80211 owned structures out of >> mac80211_ops callback context, i.e. from a tasklet fearing >> inconsistencies. Or am I worrying unnecessarily? If so I guess I'll >> redo this and use mac80211's structures directly :-) > > Yeah that's a concern, we were just looking at something similar. > However, I think at least with channel contexts you should be fine since > mac80211 never really modifies them much, except for the width and # of > chains needed? With `width` you'll change `center_freq1` in many cases. Reading a pair of old-and-new of each may not necessarily be fatal but may introduce subtle bugs that are hard to debug. Even if the current code doesn't really use these values out of mac80211 callback context now it may become a problem later. I don't know. I'll think about this a bit more. You *do* have a point. > We've recently seen a similar issue with vif->bss_conf, and I'm > considering making that an RCU-protected pointer rather than the > embedded sub-struct, that way mac80211 can replace it atomically when > needed. I've yet to see if that's practical though. It'd be lovely if I could freely read mac80211 structures from whatever context I want without any worries but I think you'd have to introduce explicit locking between mac80211 and drivers or go ham with RCU :-) Michał -- 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
