cvs commit: src/etc network.subr src/etc/rc.d netif src/etc/defaults rc.conf src/share/man/man5 rc.conf.5

Brooks Davis brooks at FreeBSD.org
Fri May 16 20:43:38 UTC 2008


On Fri, May 16, 2008 at 06:53:46PM +0300, Mike Makonnen wrote:
> Brooks Davis wrote:
>> brooks      2008-05-15 01:06:10 UTC
>> 
>>   FreeBSD src repository
>> 
>>   Modified files:
>>     etc                  network.subr     etc/rc.d             netif     
>> etc/defaults         rc.conf     share/man/man5       rc.conf.5   Log:
>>   Change the default value of synchronous_dhclient to NO.
>>     To preserve the existing behavior of etc/rc.d/netif, add code to wait
>>   up to if_up_delay seconds (30 seconds by default) for a default route to
>>   be configured if there are any dhcp interfaces.  This should be extended
>>   to test that the interface is actually up.
> 
> Can you clue us in to the reason for the change? From the log message it 
> seems that it preserves the current behavior, but in a more convoluted way. 

My long term goal is to reduce the degree to which configuring
interfaces at startup differs from the arrival of hot plugable or cloned
interfaces.  Also, once we figure out a way to easily query the link
state we'll be able to use basically the same code to pause until the
interface comes up which will work around some rather broken daemons
that can't handle not having a working resolver.

It turns out that I didn't quite get this version right.  I need to move
the test out of netif because devd isn't (and can't be) running at this
point.

> Also, if_up_delay seems to negate the need for synchronous_dhclient.

That's true.  I'm planning to remove it later, but I want to leave a
temporary fallback in case users need the old behavior or find problems
in the current behavior.

-- Brooks
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-all/attachments/20080516/13b089c1/attachment.pgp


More information about the cvs-all mailing list