svn commit: r206408 - in head: etc etc/defaults etc/rc.d share/man/man5

Bjoern A. Zeeb bz at FreeBSD.org
Fri Apr 16 22:30:07 UTC 2010


On Fri, 9 Apr 2010, Doug Barton wrote:

Hi,

first off all it would have been easier to figure a few things out, if
the several different things had been individual commits or if the
commit hadn't happed in the middle of a discussion that died with
this.  The comments below are on the first four things I could figure
out easily.

> Author: dougb
> Date: Fri Apr  9 01:35:09 2010
> New Revision: 206408
> URL: http://svn.freebsd.org/changeset/base/206408
>
> Log:
...
>  6. Add support for the [NO]RTADV options in ifconfig_getargs() and
>  ipv6_autoconfif(). In the latter, include support for the explicit
>  addition of [-]accept_rtadv in ifconfig_<interface>_ipv6 as is done
>  in the current code.

So usually we seem to use the upper case pseudo arguments like DHCP,
SYNCDHCP, WPA, .. in combination with an actual command to start apart
from ifconfig.  Now RTADV does not do that but it passes accept_rtadv or
-accept_rtadv to ifconfig. So if you need a command alias for that it
should probably be in ifconfig and discussed separately.
Please revert this part.


>  Switch ipv6_prefer to YES. If ipv6_enable is not set this will have
>  no effect.

This changed the default.  I am pretty sure this is going to bite
massively.  Why do we need opt-out rather than the previous opt-in
here?


I have no idea where in the commit message it was but I must have
missed it.  Anyway, with this change the link-local addresses are
there by default (even though luckily still disabled -- though I not
entirely sure that this would hold for all possible combinations)
even if there is no IPv6 configured.
We didn't have this on say stable/7 unless explicitly configured
(globally) and I am not sure for how many releases.
This should be reverted.
It should be added that I think these defaults will hell confuse people
even more as things are there even though not configured and are not
working because they are disabled.  The learning curve to figure that
out is way higher than simply "nothing there -> put and ifconfig_IF_ipv6
line into rc.conf".


>  5. In rc.d/netoptions, add code for an ipv6_privacy option to use
>  RFC 4193 style pseudo-random addresses
..
>  Add a default for ipv6_privacy (NO).

I think this is the only option I can say I feel ok with so far.

/bz

-- 
Bjoern A. Zeeb         It will not break if you know what you are doing.


More information about the svn-src-head mailing list