Re: git: 9e469437993b - main - inet.4: drop "experimetal use" from net0 and net240 descriptions

> On Nov 12, 2024, at 3:59 AM, Bjoern A. Zeeb <bz@FreeBSD.org> wrote:
> 
> On Mon, 11 Nov 2024, Ed Maste wrote:
> 
>> The branch main has been updated by emaste:
>> 
>> URL: https://cgit.FreeBSD.org/src/commit/?id=9e469437993bd17290d145fa7a615a68b2257875
>> 
>> commit 9e469437993bd17290d145fa7a615a68b2257875
>> Author:     Ed Maste <emaste@FreeBSD.org>
>> AuthorDate: 2024-11-10 15:41:02 +0000
>> Commit:     Ed Maste <emaste@FreeBSD.org>
>> CommitDate: 2024-11-11 12:46:58 +0000
>> 
>>   inet.4: drop "experimetal use" from net0 and net240 descriptions
>> 
>>   The sysctls enable any use of 0/8 and 240/4, not only experimental use.
>> 
>>   Reviewed by:    zlei
>>   Sponsored by:   The FreeBSD Foundation
>>   Differential Revision: https://reviews.freebsd.org/D47506
> 
> 
> They are labeled "experimental" because they are in no way official standard
> or their use being endorsed by any accepted Internet Standards Body,
> 
> Officially vendors at this point are not supposed to support it.
> The fact that most of them have a "simple switch" to turn it on but off
> by default as otherwise they would violate an RFC.

Emm, while I was reviewing this change, I also wondered if it is too early to remove the word *experimental* as the RFC is still in draft status. Well later I think ( maybe I'm wrong ) RFC actually lag behind the reality ( I'm saying FreeBSD as a vendor and move fast action ) so I think it is OK to remove now.

> 
> I would really prefer this to be backed out until an RFC is published or
> IANA decided to re-assign the (use of the) netblock for any
> official/private use.

No objection.

> 
> 
> I was fine with the original implementation if the default didn't change
> the status quo given it's experimental nature.
> 
> I'll follow-up on that in the other review.
> 
>> ---
>> share/man/man4/inet.4 | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>> 
>> diff --git a/share/man/man4/inet.4 b/share/man/man4/inet.4
>> index a12dec49caea..c9267558d6f3 100644
>> --- a/share/man/man4/inet.4
>> +++ b/share/man/man4/inet.4
>> @@ -169,10 +169,10 @@ The following general variables are defined:
>> .It Va accept_sourceroute
>> Boolean: enable/disable accepting of source-routed IP packets (default false).
>> .It Va allow_net0
>> -Boolean: allow experimental use of addresses in 0.0.0.0/8 as endpoints,
>> +Boolean: allow use of addresses in 0.0.0.0/8 as endpoints,
>> and allow forwarding of packets with these addresses.
>> .It Va allow_net240
>> -Boolean: allow experimental use of addresses in 240.0.0.0/4 as endpoints,
>> +Boolean: allow use of addresses in 240.0.0.0/4 as endpoints,
>> and allow forwarding of packets with these addresses.
>> .It Va curfrags
>> Integer: Current number of IPv4 fragments across all reassembly queues
>> 
> 
> -- 
> Bjoern A. Zeeb                                                     r15:7