On Thu, Feb 14, 2013 at 3:18 AM, Johannes Berg <[email protected]> wrote: > On Wed, 2013-02-13 at 12:14 -0800, Thomas Pedersen wrote: >> Previously, the entire mesh beacon would be generated each >> time the beacon timer fired. Instead generate a beacon >> head and tail (so the TIM can easily be inserted when mesh >> power save is on) when starting a mesh or the MBSS >> parameters change. >> >> Also add a mutex for protecting beacon updates and >> preventing leaks. > > And I dropped the patch again due to a sparse warning (with RCU sparse > enabled in Kconfig):
That's handy, I'll keep it enabled in the future. > net/mac80211/mesh.c:836:9: warning: incorrect type in argument 1 (different > address spaces) > net/mac80211/mesh.c:836:9: expected struct callback_head *head > net/mac80211/mesh.c:836:9: got struct callback_head [noderef] > <asn:4>*<noident> > > (btw, should there be locking there in that function?) I think all timers, work, etc. should be stopped at that point, but I guess it couldn't hurt. -- Thomas _______________________________________________ Devel mailing list [email protected] http://lists.open80211s.org/cgi-bin/mailman/listinfo/devel
