Dead since introduction in 2001 with icmp6.c r1.31: implement upper limit to icmp6 redirects (experimental, turned off) negative value to {mtudisc,redirect}_{hi,lo}wat will turn off the limitation. sync with kame. icmp6_redirect_lowat was always -1 and never hit the empty conditional.
icmp6_redirect_hiwat never existed. icmp6_mtudisc_{hi,lo}wat are exposed as net.inet6.icmp6.mtudisc_{hi,lo}wat sysctl(2)s, so don't touch those for now. Feedback? OK? diff --git a/sys/netinet6/icmp6.c b/sys/netinet6/icmp6.c index 94612a560c3..7a64861d15f 100644 --- a/sys/netinet6/icmp6.c +++ b/sys/netinet6/icmp6.c @@ -129,9 +129,6 @@ static int icmp6_mtudisc_lowat = 256; */ struct rttimer_queue icmp6_redirect_timeout_q; -/* XXX experimental, turned off */ -static int icmp6_redirect_lowat = -1; - void icmp6_errcount(int, int); int icmp6_ratelimit(const struct in6_addr *, const int, const int); const char *icmp6_redirect_diag(struct in6_addr *, struct in6_addr *, @@ -1386,12 +1383,6 @@ icmp6_redirect_input(struct mbuf *m, int off) rtcount = rt_timer_queue_count(&icmp6_redirect_timeout_q); if (0 <= ip6_maxdynroutes && rtcount >= ip6_maxdynroutes) goto freeit; - else if (0 <= icmp6_redirect_lowat && - rtcount > icmp6_redirect_lowat) { - /* - * XXX nuke a victim, install the new one. - */ - } bzero(&sdst, sizeof(sdst)); bzero(&sgw, sizeof(sgw));