* [iproute PATCH 0/2] minor update to bridge.8 @ 2015-11-18 11:46 Phil Sutter 2015-11-18 11:46 ` [iproute PATCH 1/2] bridge.8: document fdb replace command Phil Sutter 2015-11-18 11:46 ` [iproute PATCH 2/2] bridge.8: minor formatting cleanup Phil Sutter 0 siblings, 2 replies; 6+ messages in thread From: Phil Sutter @ 2015-11-18 11:46 UTC (permalink / raw) To: Stephen Hemminger; +Cc: netdev This was originally some leftover in RHEL patches, as well. Though the question whether 'fdb replace' should be documented or not was not exactly trivial: - In August 2012, 'fdb change' and 'fdb replace' commands were removed (commit 57b9785). - In September 2012, bridge.8 was adjusted accordingly (commit d611682). - In July 2013 though, 'fdb replace' was added again (commit 45a82e5). The latter maybe should have been a revert, as it misses to add 'fdb change' as well. Anyway, this series fixes bridge.8 again and additionally reviews formatting. Phil Sutter (2): bridge.8: document fdb replace command bridge.8: minor formatting cleanup man/man8/bridge.8 | 57 +++++++++++++++++++++++++++++++------------------------ 1 file changed, 32 insertions(+), 25 deletions(-) -- 2.5.0 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [iproute PATCH 1/2] bridge.8: document fdb replace command 2015-11-18 11:46 [iproute PATCH 0/2] minor update to bridge.8 Phil Sutter @ 2015-11-18 11:46 ` Phil Sutter 2015-11-23 23:58 ` Stephen Hemminger 2015-11-18 11:46 ` [iproute PATCH 2/2] bridge.8: minor formatting cleanup Phil Sutter 1 sibling, 1 reply; 6+ messages in thread From: Phil Sutter @ 2015-11-18 11:46 UTC (permalink / raw) To: Stephen Hemminger; +Cc: netdev Despite commit 45a82e5 ("iproute vxlan add support for fdb replace command"), the 'fdb replace' command was not mentioned in bridge.8. Signed-off-by: Phil Sutter <phil@nwl.cc> --- man/man8/bridge.8 | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/man/man8/bridge.8 b/man/man8/bridge.8 index 222a438..3c5bc47 100644 --- a/man/man8/bridge.8 +++ b/man/man8/bridge.8 @@ -50,7 +50,7 @@ bridge \- show / manipulate bridge addresses and devices .IR DEV " ]" .ti -8 -.BR "bridge fdb" " { " add " | " append " | " del " } " +.BR "bridge fdb" " { " add " | " append " | " del " | " replace " } " .I LLADDR .B dev .IR DEV " { " @@ -407,6 +407,13 @@ This command removes an existing fdb entry. The arguments are the same as with .BR "bridge fdb add" , +.SS bridge fdb replace - replace a forwarding database entry +If no matching entry is found, a new one will be created instead. + +.PP +The arguments are the same as with +.BR "bridge fdb add" , + .SS bridge fdb show - list forwarding entries. This command displays the current forwarding table. -- 2.5.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [iproute PATCH 1/2] bridge.8: document fdb replace command 2015-11-18 11:46 ` [iproute PATCH 1/2] bridge.8: document fdb replace command Phil Sutter @ 2015-11-23 23:58 ` Stephen Hemminger 0 siblings, 0 replies; 6+ messages in thread From: Stephen Hemminger @ 2015-11-23 23:58 UTC (permalink / raw) To: Phil Sutter; +Cc: netdev On Wed, 18 Nov 2015 12:46:42 +0100 Phil Sutter <phil@nwl.cc> wrote: > Despite commit 45a82e5 ("iproute vxlan add support for fdb replace > command"), the 'fdb replace' command was not mentioned in bridge.8. > > Signed-off-by: Phil Sutter <phil@nwl.cc> Applied this one. ^ permalink raw reply [flat|nested] 6+ messages in thread
* [iproute PATCH 2/2] bridge.8: minor formatting cleanup 2015-11-18 11:46 [iproute PATCH 0/2] minor update to bridge.8 Phil Sutter 2015-11-18 11:46 ` [iproute PATCH 1/2] bridge.8: document fdb replace command Phil Sutter @ 2015-11-18 11:46 ` Phil Sutter 2015-11-23 23:57 ` Stephen Hemminger 1 sibling, 1 reply; 6+ messages in thread From: Phil Sutter @ 2015-11-18 11:46 UTC (permalink / raw) To: Stephen Hemminger; +Cc: netdev - Replace commas at end of subsection with dots. - Remove whitespace at EOL. - Replace double whitespace by single one. Signed-off-by: Phil Sutter <phil@nwl.cc> --- man/man8/bridge.8 | 50 +++++++++++++++++++++++++------------------------- 1 file changed, 25 insertions(+), 25 deletions(-) diff --git a/man/man8/bridge.8 b/man/man8/bridge.8 index 3c5bc47..1ae1e77 100644 --- a/man/man8/bridge.8 +++ b/man/man8/bridge.8 @@ -25,14 +25,14 @@ bridge \- show / manipulate bridge addresses and devices .ti -8 .BR "bridge link set" -.B dev +.B dev .IR DEV .IR " [ " -.B cost +.B cost .IR COST " ] [ " -.B priority -.IR PRIO " ] [ " -.B state +.B priority +.IR PRIO " ] [ " +.B state .IR STATE "] [" .BR guard " { " on " | " off " } ] [ " .BR hairpin " { " on " | " off " } ] [ " @@ -42,21 +42,21 @@ bridge \- show / manipulate bridge addresses and devices .BR learning_sync " { " on " | " off " } ] [ " .BR flood " { " on " | " off " } ] [ " .BR hwmode " { " vepa " | " veb " } ] [ " -.BR self " ] [ " master " ] " +.BR self " ] [ " master " ] " .ti -8 .BR "bridge link" " [ " show " ] [ " -.B dev +.B dev .IR DEV " ]" .ti -8 .BR "bridge fdb" " { " add " | " append " | " del " | " replace " } " .I LLADDR -.B dev +.B dev .IR DEV " { " .BR local " | " temp " } [ " .BR self " ] [ " master " ] [ " router " ] [ " use " ] [ " -.B dst +.B dst .IR IPADDR " ] [ " .B vni .IR VNI " ] [" @@ -67,12 +67,12 @@ bridge \- show / manipulate bridge addresses and devices .ti -8 .BR "bridge fdb" " [ " show " ] [ " -.B dev +.B dev .IR DEV " ]" .ti -8 .BR "bridge mdb" " { " add " | " del " } " -.B dev +.B dev .IR DEV .B port .IR PORT @@ -84,21 +84,21 @@ bridge \- show / manipulate bridge addresses and devices .ti -8 .BR "bridge mdb show " [ " -.B dev +.B dev .IR DEV " ]" .ti -8 .BR "bridge vlan" " { " add " | " del " } " -.B dev +.B dev .IR DEV -.B vid +.B vid .IR VID " [ " -.BR pvid " ] [ " untagged " ] [ " -.BR self " ] [ " master " ] " +.BR pvid " ] [ " untagged " ] [ " +.BR self " ] [ " master " ] " .ti -8 .BR "bridge vlan" " [ " show " ] [ " -.B dev +.B dev .IR DEV " ]" .ti -8 @@ -159,7 +159,7 @@ return code will be non zero. - Bridge port. .TP -.B fdb +.B fdb - Forwarding Database entry. .TP @@ -319,7 +319,7 @@ This command displays the current bridge port configuration and flags. .SH bridge fdb - forwarding database management .B fdb -objects contain known Ethernet addresses on a link. +objects contain known Ethernet addresses on a link. .P The corresponding commands display fdb entries, add new entries, @@ -384,7 +384,7 @@ If omitted the default value is used. .BI via " DEVICE" device name of the outgoing interface for the VXLAN device driver to reach the -remote VXLAN tunnel endpoint. +remote VXLAN tunnel endpoint. .SS bridge fdb append - append a forwarding database entry This command adds a new fdb entry with an already known @@ -398,21 +398,21 @@ sends a copy of the data packet to each entry found. .PP The arguments are the same as with -.BR "bridge fdb add" , +.BR "bridge fdb add" . .SS bridge fdb delete - delete a forwarding database entry This command removes an existing fdb entry. .PP The arguments are the same as with -.BR "bridge fdb add" , +.BR "bridge fdb add" . .SS bridge fdb replace - replace a forwarding database entry If no matching entry is found, a new one will be created instead. .PP The arguments are the same as with -.BR "bridge fdb add" , +.BR "bridge fdb add" . .SS bridge fdb show - list forwarding entries. @@ -548,7 +548,7 @@ This command displays the current VLAN filter table. The .B bridge -utility can monitor the state of devices and addresses +utility can monitor the state of devices and addresses continuously. This option has a slightly different format. Namely, the .B monitor @@ -560,7 +560,7 @@ command is the first in the command line and then the object list follows: .I OBJECT-LIST is the list of object types that we want to monitor. It may contain -.BR link ", " fdb ", and " mdb "." +.BR link ", " fdb ", and " mdb "." If no .B file argument is given, -- 2.5.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [iproute PATCH 2/2] bridge.8: minor formatting cleanup 2015-11-18 11:46 ` [iproute PATCH 2/2] bridge.8: minor formatting cleanup Phil Sutter @ 2015-11-23 23:57 ` Stephen Hemminger 2015-11-24 14:50 ` [iproute PATCH v2] " Phil Sutter 0 siblings, 1 reply; 6+ messages in thread From: Stephen Hemminger @ 2015-11-23 23:57 UTC (permalink / raw) To: Phil Sutter; +Cc: netdev On Wed, 18 Nov 2015 12:46:43 +0100 Phil Sutter <phil@nwl.cc> wrote: > - Replace commas at end of subsection with dots. > - Remove whitespace at EOL. > - Replace double whitespace by single one. > > Signed-off-by: Phil Sutter <phil@nwl.cc> Could you redo this patch. It crossed over with my changes to scrub all the trailing whitespace on man8 pages. ^ permalink raw reply [flat|nested] 6+ messages in thread
* [iproute PATCH v2] bridge.8: minor formatting cleanup 2015-11-23 23:57 ` Stephen Hemminger @ 2015-11-24 14:50 ` Phil Sutter 0 siblings, 0 replies; 6+ messages in thread From: Phil Sutter @ 2015-11-24 14:50 UTC (permalink / raw) To: Stephen Hemminger; +Cc: netdev - Replace commas at end of subsection with dots. - Replace double whitespace by single one. Signed-off-by: Phil Sutter <phil@nwl.cc> --- Changes since v1: - Resolved conflict with current master. - All whitespace at EOL already gone, adjusted commit message. --- man/man8/bridge.8 | 46 +++++++++++++++++++++++----------------------- 1 file changed, 23 insertions(+), 23 deletions(-) diff --git a/man/man8/bridge.8 b/man/man8/bridge.8 index 98a92eb..0ec6f17 100644 --- a/man/man8/bridge.8 +++ b/man/man8/bridge.8 @@ -25,14 +25,14 @@ bridge \- show / manipulate bridge addresses and devices .ti -8 .BR "bridge link set" -.B dev +.B dev .IR DEV .IR " [ " -.B cost +.B cost .IR COST " ] [ " -.B priority -.IR PRIO " ] [ " -.B state +.B priority +.IR PRIO " ] [ " +.B state .IR STATE "] [" .BR guard " { " on " | " off " } ] [ " .BR hairpin " { " on " | " off " } ] [ " @@ -42,21 +42,21 @@ bridge \- show / manipulate bridge addresses and devices .BR learning_sync " { " on " | " off " } ] [ " .BR flood " { " on " | " off " } ] [ " .BR hwmode " { " vepa " | " veb " } ] [ " -.BR self " ] [ " master " ] " +.BR self " ] [ " master " ] " .ti -8 .BR "bridge link" " [ " show " ] [ " -.B dev +.B dev .IR DEV " ]" .ti -8 .BR "bridge fdb" " { " add " | " append " | " del " | " replace " } " .I LLADDR -.B dev +.B dev .IR DEV " { " .BR local " | " temp " } [ " .BR self " ] [ " master " ] [ " router " ] [ " use " ] [ " -.B dst +.B dst .IR IPADDR " ] [ " .B vni .IR VNI " ] [" @@ -67,12 +67,12 @@ bridge \- show / manipulate bridge addresses and devices .ti -8 .BR "bridge fdb" " [ " show " ] [ " -.B dev +.B dev .IR DEV " ]" .ti -8 .BR "bridge mdb" " { " add " | " del " } " -.B dev +.B dev .IR DEV .B port .IR PORT @@ -84,21 +84,21 @@ bridge \- show / manipulate bridge addresses and devices .ti -8 .BR "bridge mdb show " [ " -.B dev +.B dev .IR DEV " ]" .ti -8 .BR "bridge vlan" " { " add " | " del " } " -.B dev +.B dev .IR DEV -.B vid +.B vid .IR VID " [ " -.BR pvid " ] [ " untagged " ] [ " -.BR self " ] [ " master " ] " +.BR pvid " ] [ " untagged " ] [ " +.BR self " ] [ " master " ] " .ti -8 .BR "bridge vlan" " [ " show " ] [ " -.B dev +.B dev .IR DEV " ]" .ti -8 @@ -319,7 +319,7 @@ This command displays the current bridge port configuration and flags. .SH bridge fdb - forwarding database management .B fdb -objects contain known Ethernet addresses on a link. +objects contain known Ethernet addresses on a link. .P The corresponding commands display fdb entries, add new entries, @@ -398,21 +398,21 @@ sends a copy of the data packet to each entry found. .PP The arguments are the same as with -.BR "bridge fdb add" , +.BR "bridge fdb add" . .SS bridge fdb delete - delete a forwarding database entry This command removes an existing fdb entry. .PP The arguments are the same as with -.BR "bridge fdb add" , +.BR "bridge fdb add" . .SS bridge fdb replace - replace a forwarding database entry If no matching entry is found, a new one will be created instead. .PP The arguments are the same as with -.BR "bridge fdb add" , +.BR "bridge fdb add" . .SS bridge fdb show - list forwarding entries. @@ -548,7 +548,7 @@ This command displays the current VLAN filter table. The .B bridge -utility can monitor the state of devices and addresses +utility can monitor the state of devices and addresses continuously. This option has a slightly different format. Namely, the .B monitor @@ -560,7 +560,7 @@ command is the first in the command line and then the object list follows: .I OBJECT-LIST is the list of object types that we want to monitor. It may contain -.BR link ", " fdb ", and " mdb "." +.BR link ", " fdb ", and " mdb "." If no .B file argument is given, -- 2.5.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-11-24 14:50 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-11-18 11:46 [iproute PATCH 0/2] minor update to bridge.8 Phil Sutter 2015-11-18 11:46 ` [iproute PATCH 1/2] bridge.8: document fdb replace command Phil Sutter 2015-11-23 23:58 ` Stephen Hemminger 2015-11-18 11:46 ` [iproute PATCH 2/2] bridge.8: minor formatting cleanup Phil Sutter 2015-11-23 23:57 ` Stephen Hemminger 2015-11-24 14:50 ` [iproute PATCH v2] " Phil Sutter
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).