On Wed, Feb 02, 2011 at 08:49:17PM +0100, Marek Lindner wrote: > On Wednesday 02 February 2011 18:37:18 Linus Lüssing wrote: > > So after some more discussions with Marek and Sven, it looks like we > > have to use the rcu protected macros rcu_dereference() and > > rcu_assign_pointer() for the bat_priv->curr_gw and curr_gw->orig_node. > > > > Changes here also include moving the kref_get() from unicast_send_skb() > > into gw_get_selected(). The orig_node could have been freed already at > > the time the kref_get() was called in unicast_send_skb(). > > I'd suggest you make a standalone patch because the patches address different > problems. > > Thanks, > Marek >
Oki doki, will do that (wasn't ment to be a clean patch yet anyway - I wanted to know first if the usage of rcu_dereference/rcu_assign_pointer() was going in the right direction). Cheers, Linus
