Hi, Florian, On 2/4/20 15:08, Florian Obser wrote: [...]
After talking to Fernando I came up with this much simpler patch:
Will come back with comments on your patch, but wanted to note a couple of things bellow:
[...]
Other aspects of draft-gont-6man-slaac-renum are still being discussed in the 6man WG, these numbers don't seem to be too contentious though.
Exactly. In fact, I believe The default Valid Lifetime shouldn't be longer than 2 * Router Lifetime. -- at the end of the day, with a default Router Lifetime of 1800 seconds, the Valid Lifetime would become 1 hour.
And if the router disappears for that long, e.g. TCP timers would ahve already gone off way before.
The reason I set the Valid Lifetime as 28 * Router Lifetime was because a colleague expressed concerns (on the ietf v6ops list) about a short Valid Lifetime, and I didn't want to have the draft shot down just because of that. At the end of the day, even reducing the Valid Lifetime from 1 month to 1 day was a big win ;-)
On the other hand I don't think we are in a big hurry to get this in and we can wait a bit on how this plays out. Administrators can already set these values by themselves in rad.conf I'm also not opposed to this and I do agree that the current defaults from RFC 4861 are way to high (7 days prefered lifetime and 30 days valid lifetime).
FWIW, Linux radvd() has long deviated from RFC4861. They use: Valid Lifetime: 86400 seconds (1 day) Preferred Lifetime: 14400 seconds (4 hours) (see https://github.com/reubenhwk/radvd/blob/master/radvd.conf.5.man)
Due to hardware issues I'm currently not using rad(8). My ISP provided cpe sends router advertisements with considerably lower lifetimes then the rfc defaults. The vltime is 14.5 hours and 12 seconds(?!) and the pltime 6.5 hours and 12 seconds.
The values in the RFC are insane. In fact, when talking to one of the original authors of Neighbor Discovery, he noted that it's probably an historical artifact how the timers got screwed up the way they are (in the spec).
Thoughts from people who are actually running this? Oh, and we need to update the manpage. p.s.: And I see that tab vs. space is still messed up in the defines even after I tried to fix it :/ Maybe I should just let that part go
I can try clean that up and update the manpage if that helps. Thanks! -- Fernando Gont e-mail: [email protected] || [email protected] PGP Fingerprint: 7809 84F5 322E 45C7 F1C9 3945 96EE A9EF D076 FFF1
