On Fri, Feb 02, 2007 at 11:46:08AM -0800, David Miller wrote: > From: Neil Horman <[EMAIL PROTECTED]> > Date: Fri, 2 Feb 2007 14:06:34 -0500 > > > Ok, I'm still testing it, but heres a new patch for review. > > Significant changes include the addition of a > > CONFIG_IPV6_OPTIMISTIC_DAD option that is dependent on the inclusion > > of both IPPV6 and EXPERIMENTAL options, as well as a new method for > > redirecting packets from optimistic sources to incomplete neighbors > > by instead looking up a default router in ip6_dst_lookup_tail, as I > > described in my previous note. > > This looks largely fine to me, but I wonder about one bit: > > + > + /* > + * Optimistic nodes need to join the anycast address > + * right away > + */ > + if (ifp->flags & IFA_F_OPTIMISTIC) > + addrconf_join_anycast(ifp); > + > > If something can clear that bit during the DAD, we won't > call addrconf_leave_anycast() later. Can that happen? The only way the flag should get cleared once we've started DAD is when it completes or fails. In the failure case, we destroy the ifaddr structure, which I think should force a leave_anycast, while the completed condition calls addrconf_leave_anycast as part of the completion process before it clears the flags, so I think we should be ok. If anyone sees anything to the contrary, please let me know and I'll be sure to plug the hole.
Thanks & Regards Neil > - > To unsubscribe from this list: send the line "unsubscribe netdev" in > the body of a message to [EMAIL PROTECTED] > More majordomo info at http://vger.kernel.org/majordomo-info.html - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
