On Fri, Mar 27, 2020 at 11:10:25PM -0300, Fernando Gont wrote:
> Florian/folks,
>
> This is an improved version:
>
> Essentially, if the lifetime of a prefix is not specified (i.e., the admin
> relies on the default values), the Preferred Lifetime is set to the Router
> Lifetime, and the Valid Lifetime is set to Router Lifetime * 48 (one day)
>
> This improve the state of affairs for renumbering events on the router side.
>
After talking to Fernando I came up with this much simpler patch:
diff --git parse.y parse.y
index 8e0a899470f..88b31364816 100644
--- parse.y
+++ parse.y
@@ -964,8 +964,8 @@ conf_get_ra_prefix(struct in6_addr *addr, int prefixlen)
if (prefix == NULL)
errx(1, "%s: calloc", __func__);
prefix->prefixlen = prefixlen;
- prefix->vltime = ADV_VALID_LIFETIME;
- prefix->pltime = ADV_PREFERRED_LIFETIME;
+ prefix->pltime = ra_options->router_lifetime;
+ prefix->vltime = VLTIME_PLTIME_FACTOR * prefix->pltime;
prefix->lflag = 1;
prefix->aflag = 1;
diff --git rad.h rad.h
index 09cc9cf204e..3d58c3558fc 100644
--- rad.h
+++ rad.h
@@ -29,8 +29,7 @@
#define MAX_RTR_ADV_INTERVAL 600
#define MIN_RTR_ADV_INTERVAL 200
#define ADV_DEFAULT_LIFETIME 3 * MAX_RTR_ADV_INTERVAL
-#define ADV_PREFERRED_LIFETIME 604800 /* 7 days */
-#define ADV_VALID_LIFETIME 2592000 /* 30 days */
+#define VLTIME_PLTIME_FACTOR 48
#define MAX_SEARCH 1025 /* MAXDNAME in arpa/nameser.h */
#define DEFAULT_RDNS_LIFETIME 600 * 1.5
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.
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).
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.
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'm not entirely sure you are real.
