On Fri, Jul 22, 2016 at 12:19:54AM +0200, Sven Eckelmann wrote: > On Donnerstag, 21. Juli 2016 23:59:30 CEST Sven Eckelmann wrote: > > On Donnerstag, 21. Juli 2016 23:38:32 CEST Linus Lüssing wrote: > > > - struct hlist_head mla_list; > > > + struct hlist_head mla_list; /* protected by mcast.work */ > > > > Protected by a workqueue item? How? > > Looks like you wanted to say here that it is only modified inside the worker > queue (via the mcast.work delayed workqueue item) - unless it is not done > this > way like in batadv_mcast_free after cancel_delayed_work_sync.
Exactly :). I tried to "summarize" that a little to make it fit as an inline comment ;). (although there are some further kerneldoc remarks on the functions using it) Is it ok now or do you think I should add something like your explanation somewhere else?
