All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
* [iproute PATCH] man: ip-route.8: Document nexthop limit
@ 2018-11-12 22:21 Phil Sutter
  2018-11-13  0:37 ` David Ahern
  2018-11-14 19:20 ` Stephen Hemminger
  0 siblings, 2 replies; 4+ messages in thread
From: Phil Sutter @ 2018-11-12 22:21 UTC (permalink / raw
  To: Stephen Hemminger; +Cc: netdev

Add a note to 'nexthop' description stating the maximum number of
nexthops per command and pointing at 'append' command as a workaround.

Signed-off-by: Phil Sutter <phil@nwl.cc>
---
 man/man8/ip-route.8.in | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/man/man8/ip-route.8.in b/man/man8/ip-route.8.in
index a33ce1f0f4006..383178c11331e 100644
--- a/man/man8/ip-route.8.in
+++ b/man/man8/ip-route.8.in
@@ -589,6 +589,13 @@ argument lists:
 route reflecting its relative bandwidth or quality.
 .in -8
 
+The internal buffer used in iproute2 limits the maximum number of nexthops to
+be specified in one go. If only a gateway address is given, the current buffer
+size allows for 144 IPv6 nexthops and 253 IPv4 ones. If more are required, they
+may be added to the existing route using
+.B "ip route append"
+command.
+
 .TP
 .BI scope " SCOPE_VAL"
 the scope of the destinations covered by the route prefix.
-- 
2.19.0

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

* Re: [iproute PATCH] man: ip-route.8: Document nexthop limit
  2018-11-12 22:21 [iproute PATCH] man: ip-route.8: Document nexthop limit Phil Sutter
@ 2018-11-13  0:37 ` David Ahern
  2018-11-13 12:37   ` Phil Sutter
  2018-11-14 19:20 ` Stephen Hemminger
  1 sibling, 1 reply; 4+ messages in thread
From: David Ahern @ 2018-11-13  0:37 UTC (permalink / raw
  To: Phil Sutter, Stephen Hemminger; +Cc: netdev

On 11/12/18 2:21 PM, Phil Sutter wrote:
> diff --git a/man/man8/ip-route.8.in b/man/man8/ip-route.8.in
> index a33ce1f0f4006..383178c11331e 100644
> --- a/man/man8/ip-route.8.in
> +++ b/man/man8/ip-route.8.in
> @@ -589,6 +589,13 @@ argument lists:
>  route reflecting its relative bandwidth or quality.
>  .in -8
>  
> +The internal buffer used in iproute2 limits the maximum number of nexthops to
> +be specified in one go. If only a gateway address is given, the current buffer
> +size allows for 144 IPv6 nexthops and 253 IPv4 ones. If more are required, they
> +may be added to the existing route using
> +.B "ip route append"
> +command.
> +

That is not true for IPv4. 'ip ro append' adds a new route after the
existing route - an entry that can not be hit unless all of the nexthops
in the first route are down. 'ip ro prepend' adds a new entry before the
existing one meaning it takes precedence over the existing entries.

For IPv6, 'append' and 'prepend' both add new nexthops to the existing
entry.

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

* Re: [iproute PATCH] man: ip-route.8: Document nexthop limit
  2018-11-13  0:37 ` David Ahern
@ 2018-11-13 12:37   ` Phil Sutter
  0 siblings, 0 replies; 4+ messages in thread
From: Phil Sutter @ 2018-11-13 12:37 UTC (permalink / raw
  To: David Ahern; +Cc: Stephen Hemminger, netdev

Hi David,

On Mon, Nov 12, 2018 at 04:37:48PM -0800, David Ahern wrote:
> On 11/12/18 2:21 PM, Phil Sutter wrote:
> > diff --git a/man/man8/ip-route.8.in b/man/man8/ip-route.8.in
> > index a33ce1f0f4006..383178c11331e 100644
> > --- a/man/man8/ip-route.8.in
> > +++ b/man/man8/ip-route.8.in
> > @@ -589,6 +589,13 @@ argument lists:
> >  route reflecting its relative bandwidth or quality.
> >  .in -8
> >  
> > +The internal buffer used in iproute2 limits the maximum number of nexthops to
> > +be specified in one go. If only a gateway address is given, the current buffer
> > +size allows for 144 IPv6 nexthops and 253 IPv4 ones. If more are required, they
> > +may be added to the existing route using
> > +.B "ip route append"
> > +command.
> > +
> 
> That is not true for IPv4. 'ip ro append' adds a new route after the
> existing route - an entry that can not be hit unless all of the nexthops
> in the first route are down. 'ip ro prepend' adds a new entry before the
> existing one meaning it takes precedence over the existing entries.

Oh, thanks for clarifying. I'll follow-up with a fixed version.

> For IPv6, 'append' and 'prepend' both add new nexthops to the existing
> entry.

'ip route prepend' is not even documented. :(

Thanks, Phil

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

* Re: [iproute PATCH] man: ip-route.8: Document nexthop limit
  2018-11-12 22:21 [iproute PATCH] man: ip-route.8: Document nexthop limit Phil Sutter
  2018-11-13  0:37 ` David Ahern
@ 2018-11-14 19:20 ` Stephen Hemminger
  1 sibling, 0 replies; 4+ messages in thread
From: Stephen Hemminger @ 2018-11-14 19:20 UTC (permalink / raw
  To: Phil Sutter; +Cc: netdev

On Mon, 12 Nov 2018 23:21:01 +0100
Phil Sutter <phil@nwl.cc> wrote:

> Add a note to 'nexthop' description stating the maximum number of
> nexthops per command and pointing at 'append' command as a workaround.
> 
> Signed-off-by: Phil Sutter <phil@nwl.cc>

Applied

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

end of thread, other threads:[~2018-11-15  5:25 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-11-12 22:21 [iproute PATCH] man: ip-route.8: Document nexthop limit Phil Sutter
2018-11-13  0:37 ` David Ahern
2018-11-13 12:37   ` Phil Sutter
2018-11-14 19:20 ` Stephen Hemminger

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.