mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH 1/1] update ip-sysctl.txt documentation
@ 2015-01-06 23:36 Ani Sinha
  2015-01-07 10:11 ` David Laight
  0 siblings, 1 reply; 7+ messages in thread
From: Ani Sinha @ 2015-01-06 23:36 UTC (permalink / raw)
  To: corbet, davem, edumazet, linux-doc, linux-kernel, ani, P, netdev,
	fruggeri

Update documentation to reflect the fact that
/proc/sys/net/ipv4/route/max_size is no longer used for ipv4.

Signed-off-by: Ani Sinha <ani@arista.com>
---
 Documentation/networking/ip-sysctl.txt |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/Documentation/networking/ip-sysctl.txt b/Documentation/networking/ip-sysctl.txt
index 9bffdfc..2a28261 100644
--- a/Documentation/networking/ip-sysctl.txt
+++ b/Documentation/networking/ip-sysctl.txt
@@ -64,8 +64,10 @@ fwmark_reflect - BOOLEAN
 	Default: 0
 
 route/max_size - INTEGER
-	Maximum number of routes allowed in the kernel.  Increase
-	this when using large numbers of interfaces and/or routes.
+	Post linux kernel 3.6, this is deprecated for ipv4 as route cache is no
+	longer used. For ipv6, this is used to limit the maximum number of ipv6
+	routes allowed in the kernel.  Increase this when using large numbers of
+	interfaces and/or routes.
 
 neigh/default/gc_thresh1 - INTEGER
 	Minimum number of entries to keep.  Garbage collector will not
-- 
1.7.4.4


^ permalink raw reply	[flat|nested] 7+ messages in thread

* RE: [PATCH 1/1] update ip-sysctl.txt documentation
  2015-01-06 23:36 [PATCH 1/1] update ip-sysctl.txt documentation Ani Sinha
@ 2015-01-07 10:11 ` David Laight
  2015-01-07 23:46   ` Ani Sinha
  0 siblings, 1 reply; 7+ messages in thread
From: David Laight @ 2015-01-07 10:11 UTC (permalink / raw)
  To: 'Ani Sinha',
	corbet, davem, edumazet, linux-doc, linux-kernel, P, netdev,
	fruggeri

From: Ani Sinha
> Update documentation to reflect the fact that
> /proc/sys/net/ipv4/route/max_size is no longer used for ipv4.
> 
> Signed-off-by: Ani Sinha <ani@arista.com>
> ---
>  Documentation/networking/ip-sysctl.txt |    6 ++++--
>  1 files changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/networking/ip-sysctl.txt b/Documentation/networking/ip-sysctl.txt
> index 9bffdfc..2a28261 100644
> --- a/Documentation/networking/ip-sysctl.txt
> +++ b/Documentation/networking/ip-sysctl.txt
> @@ -64,8 +64,10 @@ fwmark_reflect - BOOLEAN
>  	Default: 0
> 
>  route/max_size - INTEGER
> -	Maximum number of routes allowed in the kernel.  Increase
> -	this when using large numbers of interfaces and/or routes.
> +	Post linux kernel 3.6, this is deprecated for ipv4 as route cache is no
> +	longer used. For ipv6, this is used to limit the maximum number of ipv6
> +	routes allowed in the kernel.  Increase this when using large numbers of
> +	interfaces and/or routes.

Now imagine you are reading ip-sysctl.txt, the new text is all wrong.
Something like:
	Limit on the size of the ip route caches.
	Ignored for ipv4 after kernel 3.6 because the ipv4 route cache was removed.
	Increase this when using large numbers of interfaces and/or routes.
would read better.

	David


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/1] update ip-sysctl.txt documentation
  2015-01-07 10:11 ` David Laight
@ 2015-01-07 23:46   ` Ani Sinha
  0 siblings, 0 replies; 7+ messages in thread
From: Ani Sinha @ 2015-01-07 23:46 UTC (permalink / raw)
  To: David Laight
  Cc: corbet, davem, edumazet, linux-doc, linux-kernel, P, netdev, fruggeri

On Wed, Jan 7, 2015 at 2:11 AM, David Laight <David.Laight@aculab.com> wrote:
> From: Ani Sinha
>> Update documentation to reflect the fact that
>> /proc/sys/net/ipv4/route/max_size is no longer used for ipv4.
>>
>> Signed-off-by: Ani Sinha <ani@arista.com>
>> ---
>>  Documentation/networking/ip-sysctl.txt |    6 ++++--
>>  1 files changed, 4 insertions(+), 2 deletions(-)
>>
>> diff --git a/Documentation/networking/ip-sysctl.txt b/Documentation/networking/ip-sysctl.txt
>> index 9bffdfc..2a28261 100644
>> --- a/Documentation/networking/ip-sysctl.txt
>> +++ b/Documentation/networking/ip-sysctl.txt
>> @@ -64,8 +64,10 @@ fwmark_reflect - BOOLEAN
>>       Default: 0
>>
>>  route/max_size - INTEGER
>> -     Maximum number of routes allowed in the kernel.  Increase
>> -     this when using large numbers of interfaces and/or routes.
>> +     Post linux kernel 3.6, this is deprecated for ipv4 as route cache is no
>> +     longer used. For ipv6, this is used to limit the maximum number of ipv6
>> +     routes allowed in the kernel.  Increase this when using large numbers of
>> +     interfaces and/or routes.
>
> Now imagine you are reading ip-sysctl.txt, the new text is all wrong.
> Something like:
>         Limit on the size of the ip route caches.
>         Ignored for ipv4 after kernel 3.6 because the ipv4 route cache was removed.
>         Increase this when using large numbers of interfaces and/or routes.
> would read better.

Ok please take a look at my patch v2.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/1] update ip-sysctl.txt documentation
  2015-01-06 22:40 ` David Miller
@ 2015-01-07  2:08   ` Ani Sinha
  0 siblings, 0 replies; 7+ messages in thread
From: Ani Sinha @ 2015-01-07  2:08 UTC (permalink / raw)
  To: David Miller
  Cc: corbet, Eric Dumazet, linux-doc, linux-kernel,
	Pádraig Brady, netdev, fruggeri

On Tue, Jan 6, 2015 at 2:40 PM, David Miller <davem@davemloft.net> wrote:
> From: Ani Sinha <ani@arista.com>
> Date: Tue,  6 Jan 2015 14:33:45 -0800
>
>> @@ -64,8 +64,10 @@ fwmark_reflect - BOOLEAN
>>       Default: 0
>>
>>  route/max_size - INTEGER
>> -     Maximum number of routes allowed in the kernel.  Increase
>> -     this when using large numbers of interfaces and/or routes.
>> +        Post linux kernel 3.6, this is depricated for ipv4 as route cache is no
>> +        longer used. For ipv6, this is used to limit the maximum number of ipv6
>> +        routes allowed in the kernel.  Increase this when using large numbers of
>> +        interfaces and/or routes.
>
> Please do not change the TABs into sequenes of space characters.

Fixed in the latest patch.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/1] update ip-sysctl.txt documentation
  2015-01-06 22:33 Ani Sinha
@ 2015-01-06 22:40 ` David Miller
  2015-01-07  2:08   ` Ani Sinha
  0 siblings, 1 reply; 7+ messages in thread
From: David Miller @ 2015-01-06 22:40 UTC (permalink / raw)
  To: ani; +Cc: corbet, edumazet, linux-doc, linux-kernel, P, netdev, fruggeri

From: Ani Sinha <ani@arista.com>
Date: Tue,  6 Jan 2015 14:33:45 -0800

> @@ -64,8 +64,10 @@ fwmark_reflect - BOOLEAN
>  	Default: 0
>  
>  route/max_size - INTEGER
> -	Maximum number of routes allowed in the kernel.  Increase
> -	this when using large numbers of interfaces and/or routes.
> +        Post linux kernel 3.6, this is depricated for ipv4 as route cache is no
> +        longer used. For ipv6, this is used to limit the maximum number of ipv6
> +        routes allowed in the kernel.  Increase this when using large numbers of
> +        interfaces and/or routes.

Please do not change the TABs into sequenes of space characters.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH 1/1] update ip-sysctl.txt documentation
@ 2015-01-06 22:33 Ani Sinha
  2015-01-06 22:40 ` David Miller
  0 siblings, 1 reply; 7+ messages in thread
From: Ani Sinha @ 2015-01-06 22:33 UTC (permalink / raw)
  To: corbet, davem, edumazet, linux-doc, linux-kernel, ani, P, netdev,
	fruggeri

Update documentation to reflect the fact that
/proc/sys/net/ipv4/route/max_size is no longer used for
ipv4.

Signed-off-by: Ani Sinha <ani@arista.com>
---
 Documentation/networking/ip-sysctl.txt |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/Documentation/networking/ip-sysctl.txt b/Documentation/networking/ip-sysctl.txt
index 9bffdfc..c8a7e37 100644
--- a/Documentation/networking/ip-sysctl.txt
+++ b/Documentation/networking/ip-sysctl.txt
@@ -64,8 +64,10 @@ fwmark_reflect - BOOLEAN
 	Default: 0
 
 route/max_size - INTEGER
-	Maximum number of routes allowed in the kernel.  Increase
-	this when using large numbers of interfaces and/or routes.
+        Post linux kernel 3.6, this is depricated for ipv4 as route cache is no
+        longer used. For ipv6, this is used to limit the maximum number of ipv6
+        routes allowed in the kernel.  Increase this when using large numbers of
+        interfaces and/or routes.
 
 neigh/default/gc_thresh1 - INTEGER
 	Minimum number of entries to keep.  Garbage collector will not
-- 
1.7.4.4


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH 1/1] update ip-sysctl.txt documentation
@ 2015-01-06 20:04 Ani Sinha
  0 siblings, 0 replies; 7+ messages in thread
From: Ani Sinha @ 2015-01-06 20:04 UTC (permalink / raw)
  To: corbet, davem, edumazet, linux-doc, linux-kernel, ani, P

Update documentation to reflect the fact that
/proc/sys/net/ipv4/route/max_size is no longer used for
ipv4.

Signed-off-by: Ani Sinha <ani@arista.com>
---
 Documentation/networking/ip-sysctl.txt |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/Documentation/networking/ip-sysctl.txt b/Documentation/networking/ip-sysctl.txt
index 9bffdfc..c8a7e37 100644
--- a/Documentation/networking/ip-sysctl.txt
+++ b/Documentation/networking/ip-sysctl.txt
@@ -64,8 +64,10 @@ fwmark_reflect - BOOLEAN
 	Default: 0
 
 route/max_size - INTEGER
-	Maximum number of routes allowed in the kernel.  Increase
-	this when using large numbers of interfaces and/or routes.
+        Post linux kernel 3.6, this is depricated for ipv4 as route cache is no
+        longer used. For ipv6, this is used to limit the maximum number of ipv6
+        routes allowed in the kernel.  Increase this when using large numbers of
+        interfaces and/or routes.
 
 neigh/default/gc_thresh1 - INTEGER
 	Minimum number of entries to keep.  Garbage collector will not
-- 
1.7.4.4


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2015-01-07 23:47 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-06 23:36 [PATCH 1/1] update ip-sysctl.txt documentation Ani Sinha
2015-01-07 10:11 ` David Laight
2015-01-07 23:46   ` Ani Sinha
  -- strict thread matches above, loose matches on Subject: below --
2015-01-06 22:33 Ani Sinha
2015-01-06 22:40 ` David Miller
2015-01-07  2:08   ` Ani Sinha
2015-01-06 20:04 Ani Sinha

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®