CVS commit: [netbsd-8] src/sys

["Martin Husemann" <martin%netbsd.org@localhost>]

Module Name:    src
Committed By:   martin
Date:           Mon Sep 30 15:48:45 UTC 2019

Modified Files:
        src/sys/netinet [netbsd-8]: if_arp.c
        src/sys/netinet6 [netbsd-8]: nd6.c nd6.h nd6_nbr.c

Log Message:
Pull up following revision(s) (requested by ozaki-r in ticket #1396):

        sys/netinet6/nd6.h: revision 1.88
        sys/netinet6/nd6_nbr.c: revision 1.174
        sys/netinet6/nd6.c: revision 1.264
        sys/netinet/if_arp.c: revision 1.288 (patch)

Initialize DAD components properly

The original code initialized each component in non-init functions such as
arp_dad_start and nd6_dad_find, conditionally based on a global flag for each.
However, it was racy because the flag and the code around it were not
protected by a lock and could cause a kernel panic at worst.

Fix the issue by initializing the components in bootup as usual.


To generate a diff of this commit:
cvs rdiff -u -r1.250.2.9 -r1.250.2.10 src/sys/netinet/if_arp.c
cvs rdiff -u -r1.232.2.12 -r1.232.2.13 src/sys/netinet6/nd6.c
cvs rdiff -u -r1.83.6.1 -r1.83.6.2 src/sys/netinet6/nd6.h
cvs rdiff -u -r1.138.6.8 -r1.138.6.9 src/sys/netinet6/nd6_nbr.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.