On 17.08.2016 12:28, Mike Manning wrote: > +static void dev_disable_change(struct inet6_dev *idev); > > /* > * Configured unicast address hash table > @@ -1945,6 +1946,12 @@ lock_errdad: > > pr_info("%s: IPv6 being disabled!\n", > ifp->idev->dev->name); > + spin_unlock_bh(&ifp->lock); > + addrconf_dad_stop(ifp, 1); > + rtnl_lock(); > + dev_disable_change(idev); > + rtnl_unlock(); > + return; > } > }
You can't take rtnl_lock at that point but must postpone the actions and do that in addrconf_dad_work. Probably the whole ... else if (idev->cnf.accept_dad > 1 && ...) needs to move there. Bye, Hannes