Remove constant basereachable and retrans members from struct nd_ifinfo
authorkn <kn@openbsd.org>
Fri, 2 Dec 2022 15:35:35 +0000 (15:35 +0000)
committerkn <kn@openbsd.org>
Fri, 2 Dec 2022 15:35:35 +0000 (15:35 +0000)
commitabcde9bc4201fd03dd5b1a03a374ae00b9297e11
tree633d58f629a85c2a23bae5fd7da913cb51b40c99
parent24793c0c0b6797d16cc9a6f2e0462b7beced28ed
Remove constant basereachable and retrans members from struct nd_ifinfo

Both are initalised with compile-time constants and never written to.

They are part of the Neighbour Discovery machinery and only surface
through the single-user SIOCGIFINFO_IN6:
$ ndp -i lo0
basereachable=30s0ms, reachable=39s, retrans=1s0ms

These values are read-only since 2017
sys/netinet6/nd6.c r1.217
usr.sbin/ndp/ndp.c r1.85
    Remove knob and always do neighbor unreachable detection

Inline the macros (to keep meaningful names), shrink the per-interface
allocated struct nd_ifinfo to what is actually needed and inline
nd6_dad_starttimer()'s constant `msec' argument.

Nothing else in base, incl. regress, uses SIOCGIFINFO_IN6 or `ndp -i'.

OK bluhm
sys/netinet6/nd6.c
sys/netinet6/nd6.h
sys/netinet6/nd6_nbr.c
usr.sbin/ndp/ndp.c