diff mbox series

[net-next,v2] ipv6: Document that max_size sysctl is depreciated

Message ID 20230117221858.734596-1-jmaxwell37@gmail.com (mailing list archive)
State Superseded
Delegated to: Netdev Maintainers
Headers show
Series [net-next,v2] ipv6: Document that max_size sysctl is depreciated | expand

Checks

Context Check Description
netdev/tree_selection success Clearly marked for net-next
netdev/fixes_present success Fixes tag not required for -next series
netdev/subject_prefix success Link
netdev/cover_letter success Single patches do not need cover letters
netdev/patch_count success Link
netdev/header_inline success No static functions without inline keyword in header files
netdev/build_32bit success Errors and warnings before: 0 this patch: 0
netdev/cc_maintainers warning 2 maintainers not CCed: linux-doc@vger.kernel.org corbet@lwn.net
netdev/build_clang success Errors and warnings before: 0 this patch: 0
netdev/module_param success Was 0 now: 0
netdev/verify_signedoff success Signed-off-by tag matches author and committer
netdev/check_selftest success No net selftest shell script
netdev/verify_fixes success No Fixes tag
netdev/build_allmodconfig_warn success Errors and warnings before: 0 this patch: 0
netdev/checkpatch success total: 0 errors, 0 warnings, 0 checks, 9 lines checked
netdev/kdoc success Errors and warnings before: 0 this patch: 0
netdev/source_inline success Was 0 now: 0

Commit Message

Jonathan Maxwell Jan. 17, 2023, 10:18 p.m. UTC
v2: use correct commit syntax.

Document that max_size is depreciated due to:

af6d10345ca7 ("ipv6: remove max_size check inline with ipv4")

Signed-off-by: Jon Maxwell <jmaxwell37@gmail.com>
---
 Documentation/networking/ip-sysctl.rst | 3 +++
 1 file changed, 3 insertions(+)

Comments

Jakub Kicinski Jan. 19, 2023, 4:38 a.m. UTC | #1
On Wed, 18 Jan 2023 09:18:58 +1100 Jon Maxwell wrote:
> Subject: [net-next v2] ipv6: Document that max_size sysctl is depreciated
>
> v2: use correct commit syntax.

change log under the --- lines

> Document that max_size is depreciated due to:
> 
> af6d10345ca7 ("ipv6: remove max_size check inline with ipv4")

 ^ commit

the word "commit" should be there before the hash

> Signed-off-by: Jon Maxwell <jmaxwell37@gmail.com>
> ---
>  Documentation/networking/ip-sysctl.rst | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/Documentation/networking/ip-sysctl.rst b/Documentation/networking/ip-sysctl.rst
> index 7fbd060d6047..edf1fcd10c5c 100644
> --- a/Documentation/networking/ip-sysctl.rst
> +++ b/Documentation/networking/ip-sysctl.rst
> @@ -156,6 +156,9 @@ route/max_size - INTEGER
>  	From linux kernel 3.6 onwards, this is deprecated for ipv4
>  	as route cache is no longer used.
>  
> +	From linux kernel 6.2 onwards, this is deprecated for ipv6

6.2 or 6.3? 6.2 is what's currently in Linus's tree, net-next 
is 6.3 and the commit in the commit msg is in net-next only.

> +	as garbage collection manages cached route entries.
Jonathan Maxwell Jan. 19, 2023, 4:42 a.m. UTC | #2
On Thu, Jan 19, 2023 at 3:38 PM Jakub Kicinski <kuba@kernel.org> wrote:
>
> On Wed, 18 Jan 2023 09:18:58 +1100 Jon Maxwell wrote:
> > Subject: [net-next v2] ipv6: Document that max_size sysctl is depreciated
> >
> > v2: use correct commit syntax.
>
> change log under the --- lines
>
> > Document that max_size is depreciated due to:
> >
> > af6d10345ca7 ("ipv6: remove max_size check inline with ipv4")
>
>  ^ commit

Okay I'll add that.

>
> the word "commit" should be there before the hash
>
> > Signed-off-by: Jon Maxwell <jmaxwell37@gmail.com>
> > ---
> >  Documentation/networking/ip-sysctl.rst | 3 +++
> >  1 file changed, 3 insertions(+)
> >
> > diff --git a/Documentation/networking/ip-sysctl.rst b/Documentation/networking/ip-sysctl.rst
> > index 7fbd060d6047..edf1fcd10c5c 100644
> > --- a/Documentation/networking/ip-sysctl.rst
> > +++ b/Documentation/networking/ip-sysctl.rst
> > @@ -156,6 +156,9 @@ route/max_size - INTEGER
> >       From linux kernel 3.6 onwards, this is deprecated for ipv4
> >       as route cache is no longer used.
> >
> > +     From linux kernel 6.2 onwards, this is deprecated for ipv6
>
> 6.2 or 6.3? 6.2 is what's currently in Linus's tree, net-next
> is 6.3 and the commit in the commit msg is in net-next only.
>

I'll change this to 6.3.

Regards

Jon

> > +     as garbage collection manages cached route entries.
diff mbox series

Patch

diff --git a/Documentation/networking/ip-sysctl.rst b/Documentation/networking/ip-sysctl.rst
index 7fbd060d6047..edf1fcd10c5c 100644
--- a/Documentation/networking/ip-sysctl.rst
+++ b/Documentation/networking/ip-sysctl.rst
@@ -156,6 +156,9 @@  route/max_size - INTEGER
 	From linux kernel 3.6 onwards, this is deprecated for ipv4
 	as route cache is no longer used.
 
+	From linux kernel 6.2 onwards, this is deprecated for ipv6
+	as garbage collection manages cached route entries.
+
 neigh/default/gc_thresh1 - INTEGER
 	Minimum number of entries to keep.  Garbage collector will not
 	purge entries if there are fewer than this number.