From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=95.215.58.39 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1794185821; b=dMPmatAFwsn1cHaoIujWheDW2+l1Ons95aKU/C6l1IYEwcVCxRWhKemQDIuW+Os/q11M aebXuM1b4tmgxZFTcp7kE4Z/zifi7xlaVCrJs4a13hTQvMRYZ7/itxvSZBZO0jpaRYnjL wbEAu9UGG1Cx2HgMiy1sHvD3YoQud1pyRGztKyy/CZ74wXmfduC0W4qnEAAo9mC9NBm4k zOe20YEe3fC+31nXknYe9YxdV3ufBOchULomnyzG6MDFa3w7HE0dXOMXLLInv1dFlk/eY y/GkLTV7RjVBomTW9I6ZFFizOADAuYpP0ONQhLxmlazGnmkN2DtUM33zQouEgotqsqQ== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1794185821; h=DMARC-Filter:X-Envelope-To:DKIM-Signature:X-Envelope-To:Received: X-Mizu-Trace-ID:X-Migadu-Flow:Date:From:To:Cc:Subject:Message-ID: References:MIME-Version:Content-Type:Content-Disposition:In-Reply-To; bh=vTOI75BFrAph/w738L52QukFiaVP1gQS2I9yBx6tGqE=; b=Ixw8d6zhyk3immaiNGtLB/1Z8uLMHcpFq18n5oxQXKKpPklNDZnudaOZ/QWmnWXQheBB NKpFCKoCsb7Zdjt84vMUrZiTthlkYEZih9FJAFgUjqZDLJCuEpXUdexgLKqTYt/1u3bMd sVPuD4IbnuR5WSJmlPQt7/NSaXlhaRzZKgGv2LCkeLh6w/S/t10gbNkT4/JbahGD/WYWN tyCJ9/+JEAR3Fli/gbimUR2xNnELzxkbPdUE8KZ7veaXEr9ctpYGQv4gbkWwhoSMJn7uS MbfpXtvWyvOaWf7V3QWYCiWRr8WEb60gJyE5u2MkBQNslxLXG6Fn9QLjhcPxmH7TAow== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass header.from=linux.dev; spf=pass smtp.mailfrom=linux.dev; dkim=pass header.d=linux.dev header.i=@linux.dev header.a=rsa-sha256 header.s=key1 header.b=rObXoNHP; arc=none smtp.remote-ip=95.215.58.39 Received: from mta1.migadu.com (out-39.mta1.migadu.com [95.215.58.39]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E01E3606A320 for ; Mon, 9 Nov 2026 00:56:59 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.2 smtp.subspace.kernel.org E01E3606A320 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key, unprotected) header.d=linux.dev header.i=@linux.dev header.a=rsa-sha256 header.s=key1 header.b=rObXoNHP X-Envelope-To: netdev@vger.kernel.org DKIM-Signature: a=rsa-sha256; bh=WLnjSDdFf2j7CWL26ddLuuNb98YQV5N7sHCyH+3jHrY=; c=simple/simple; d=linux.dev; h=from:to:subject:date:message-id:mime-version:content-type; s=key1; t=1794185817; v=1; x=1794790617; b=rObXoNHPifnj7ZeOHu29Bx2LwStgzqkw3l7WAJEet2lTCR+RRvtz5EyG8cimFB5lZp4SyArX FGSew/rQKSrVoCUrB5bit7F+Oy1LI6iRl0i3Y/vrYo0d4tbj8xytrJ2QkBV5ibB+/DWImFQQZoR ZvamjEm3LwrwQ4btT0sx3x3Q= X-Envelope-To: netdev@vger.kernel.org Received: by mta12.migadu.com with ESMTPS id 7fed51d2fabc6117; Mon, 09 Nov 2026 00:56:57 +0000 X-Mizu-Trace-ID: 7fed51d2fabc6117 X-Migadu-Flow: FLOW_OUT Date: Mon, 9 Nov 2026 08:56:48 +0800 From: Hangbin Liu To: Gopal Malaviya Cc: netdev@vger.kernel.org, "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Simon Horman , Jonathan Corbet , Shuah Khan , Randy Dunlap , linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH net-next] docs: net: fix outdated default values of four sysctls Message-ID: References: <20261108061746.32090-1-gopalmalaviya53@gmail.com> X-Mailing-List: netdev@vger.kernel.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20261108061746.32090-1-gopalmalaviya53@gmail.com> On Sun, Nov 08, 2026 at 06:17:46AM +0000, Gopal Malaviya wrote: > The defaults documented in ip-sysctl.rst for four sysctls differ from > the values the kernel actually uses: > > sysctl documented actual > cipso_rbm_strictvalid 0 1 > flowlabel_state_ranges 1 0 > router_solicitations 3 -1 > regen_max_retry 5 3 > > Each one went stale when the code changed and the documentation was not > updated, or was wrong from the start: > > - cipso_v4_rbm_strictvalid has been initialised to 1 since CIPSO was > merged, while the documentation added with it said 0. > - Commit be26849bfb6f ("ipv6: Disable flowlabel state ranges by > default") changed the default of flowlabel_state_ranges to 0. > - Commit bd11f0741fa5 ("ipv6 addrconf: implement RFC7559 router > solicitation backoff") changed MAX_RTR_SOLICITATIONS from 3 to -1, > meaning that router solicitations are sent without a limit. The > documentation was not updated and never mentioned -1. > - Commit 76f793e3a471 ("ipv6: updates to privacy addresses per RFC > 4941.") changed REGEN_MAX_RETRY from 5 to 3. > > Update the four defaults and note that -1 means no limit for > router_solicitations. > > The mismatches were found with the help of an AI assistant (Claude), > which compared the defaults documented in ip-sysctl.rst with the values > under /proc/sys/net on a running kernel and with the initial values in > the source. I confirmed each value with sysctl on a running kernel and > against the net-next source and the commits below. The commit message > was drafted with the same assistance. > > Tested by building the networking documentation with > "make SPHINXDIRS=networking htmldocs": the change adds no warnings and > the four defaults render as updated. > > Fixes: 8802f616f6de ("[NetLabel]: documentation") > Fixes: be26849bfb6f ("ipv6: Disable flowlabel state ranges by default") > Fixes: bd11f0741fa5 ("ipv6 addrconf: implement RFC7559 router solicitation backoff") > Fixes: 76f793e3a471 ("ipv6: updates to privacy addresses per RFC 4941.") > Signed-off-by: Gopal Malaviya > --- > Documentation/networking/ip-sysctl.rst | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/Documentation/networking/ip-sysctl.rst b/Documentation/networking/ip-sysctl.rst > index bc22400b53..0b7bfbd98a 100644 > --- a/Documentation/networking/ip-sysctl.rst > +++ b/Documentation/networking/ip-sysctl.rst > @@ -1587,7 +1587,7 @@ cipso_rbm_strictvalid - BOOLEAN > - 0 (disabled) > - 1 (enabled) > > - Default: 0 (disabled) > + Default: 1 (enabled) > > IP Variables > ============ > @@ -2418,7 +2418,7 @@ flowlabel_state_ranges - BOOLEAN > - 0 (disabled) > - 1 (enabled) > > - Default: 1 (enabled) > + Default: 0 (disabled) > > > flowlabel_reflect - INTEGER > @@ -2956,9 +2956,9 @@ router_solicitation_interval - INTEGER > > router_solicitations - INTEGER > Number of Router Solicitations to send until assuming no > - routers are present. > + routers are present. A value of -1 means no limit. > > - Default: 3 > + Default: -1 > > use_oif_addrs_only - BOOLEAN > When enabled, the candidate source addresses for destinations > @@ -3034,7 +3034,7 @@ regen_max_retry - INTEGER > Number of attempts before give up attempting to generate > valid temporary addresses. > > - Default: 5 > + Default: 3 > > max_addresses - INTEGER > Maximum number of autoconfigured addresses per interface. Setting > > base-commit: fa5196392691051963726787944db872420dfa75 > -- > 2.43.0 > Checked with the source and sysctl output, all works good to me. Reviewed-by: Hangbin Liu