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