diff mbox series

[iproute2-next,v2,4/6] man: ip-link: document new bcast_flood flag on bridge ports

Message ID 20220309071716.2678952-5-troglobit@gmail.com (mailing list archive)
State Superseded
Delegated to: David Ahern
Headers show
Series bridge: support for controlling broadcast flooding per port | expand

Checks

Context Check Description
netdev/tree_selection success Not a local patch

Commit Message

Joachim Wiberg March 9, 2022, 7:17 a.m. UTC
Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
---
 man/man8/ip-link.8.in | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Stephen Hemminger March 9, 2022, 7:28 a.m. UTC | #1
On Wed,  9 Mar 2022 08:17:14 +0100
Joachim Wiberg <troglobit@gmail.com> wrote:

> Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
> ---
>  man/man8/ip-link.8.in | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/man/man8/ip-link.8.in b/man/man8/ip-link.8.in
> index 19a0c9ca..6dd18f7b 100644
> --- a/man/man8/ip-link.8.in
> +++ b/man/man8/ip-link.8.in
> @@ -2366,6 +2366,8 @@ the following additional arguments are supported:
>  ] [
>  .BR mcast_flood " { " on " | " off " }"
>  ] [
> +.BR bcast_flood " { " on " | " off " }"
> +] [
>  .BR mcast_to_unicast " { " on " | " off " }"
>  ] [
>  .BR group_fwd_mask " MASK"
> @@ -2454,6 +2456,10 @@ option above.
>  - controls whether a given port will flood multicast traffic for which
>    there is no MDB entry.
>  
> +.BR bcast_flood " { " on " | " off " }"
> +- controls flooding of broadcast traffic on the given port. By default
> +this flag is on.
> +
>  .BR mcast_to_unicast " { " on " | " off " }"
>  - controls whether a given port will replicate packets using unicast
>    instead of multicast. By default this flag is off.

Minor nit, would be better to put options in alphabetical order in document.
Certainly not splitting the two mcast options.
Joachim Wiberg March 9, 2022, 12:34 p.m. UTC | #2
On Tue, Mar 08, 2022 at 23:28, Stephen Hemminger <stephen@networkplumber.org> wrote:
> Minor nit, would be better to put options in alphabetical order in document.
> Certainly not splitting the two mcast options.

Ah, of course!  I'll fix it up in a v3 later today, thanks!

 /J
diff mbox series

Patch

diff --git a/man/man8/ip-link.8.in b/man/man8/ip-link.8.in
index 19a0c9ca..6dd18f7b 100644
--- a/man/man8/ip-link.8.in
+++ b/man/man8/ip-link.8.in
@@ -2366,6 +2366,8 @@  the following additional arguments are supported:
 ] [
 .BR mcast_flood " { " on " | " off " }"
 ] [
+.BR bcast_flood " { " on " | " off " }"
+] [
 .BR mcast_to_unicast " { " on " | " off " }"
 ] [
 .BR group_fwd_mask " MASK"
@@ -2454,6 +2456,10 @@  option above.
 - controls whether a given port will flood multicast traffic for which
   there is no MDB entry.
 
+.BR bcast_flood " { " on " | " off " }"
+- controls flooding of broadcast traffic on the given port. By default
+this flag is on.
+
 .BR mcast_to_unicast " { " on " | " off " }"
 - controls whether a given port will replicate packets using unicast
   instead of multicast. By default this flag is off.