Hi roopa, On Tue, Jul 28, 2015, at 21:28, roopa wrote: > On 7/28/15, 6:04 AM, Hannes Frederic Sowa wrote: > > Can't you simply use ipv6_stub_impl.ipv6_dst_lookup with sk=NULL to do > > that and don't have a run-time dependency on IPv6 at all (for the cost > > of a function pointer). > ipv6_stub_impl.ipv6_dst_lookup seems to require sk today. > But it only needs it to get 'net' in the beginning and sk is optional > afterwards. > I will submit a patch to add 'net' as an arg to ipv6_dst_lookup. > Users of ipv6_dst_lookup are few and that seems like an easy change and > helps my patch. > If you or others think otherwise, pls let me know.
No need to extend this function at any cost. Simply add your own function pointer to the struct if needed. Probably you have to move the ipv6_stub = &ipv6_stub_impl; initialization in inet6_init down so you don't expose the function pointer too early and thus it races with initialization (and error handling seems to be incorrect in this function, too). Thanks, Hannes -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
