* [PATCH iproute2-next] ip: Clarify MACVLAN private mode
@ 2021-04-24 21:28 Jethro Beekman
2021-04-27 4:02 ` Stephen Hemminger
0 siblings, 1 reply; 2+ messages in thread
From: Jethro Beekman @ 2021-04-24 21:28 UTC (permalink / raw)
To: netdev
Traffic isn't really "disallowed" but rather some broadcast traffic is filtered.
Signed-off-by: Jethro Beekman <kernel@jbeekman.nl>
---
man/man8/ip-link.8.in | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/man/man8/ip-link.8.in b/man/man8/ip-link.8.in
index fd67e611..a4abae5f 100644
--- a/man/man8/ip-link.8.in
+++ b/man/man8/ip-link.8.in
@@ -1366,10 +1366,12 @@ the following additional arguments are supported:
.BR /dev/tapX " to be used just like a " tuntap " device."
.B mode private
-- Do not allow communication between
+- Do not allow broadcast communication between
.B macvlan
instances on the same physical interface, even if the external switch supports
-hairpin mode.
+hairpin mode. Unicast traffic is transmitted over the physical interface as in
+.B vepa
+mode, but the lack of ARP responses may hamper communication.
.B mode vepa
- Virtual Ethernet Port Aggregator mode. Data from one
@@ -1394,7 +1396,7 @@ forces the underlying interface into promiscuous mode. Passing the
using standard tools.
.B mode source
-- allows one to set a list of allowed mac address, which is used to match
+- Allows one to set a list of allowed mac address, which is used to match
against source mac address from received frames on underlying interface. This
allows creating mac based VLAN associations, instead of standard port or tag
based. The feature is useful to deploy 802.1x mac based behavior,
--
2.31.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH iproute2-next] ip: Clarify MACVLAN private mode
2021-04-24 21:28 [PATCH iproute2-next] ip: Clarify MACVLAN private mode Jethro Beekman
@ 2021-04-27 4:02 ` Stephen Hemminger
0 siblings, 0 replies; 2+ messages in thread
From: Stephen Hemminger @ 2021-04-27 4:02 UTC (permalink / raw)
To: Jethro Beekman; +Cc: netdev
On Sat, 24 Apr 2021 23:28:52 +0200
Jethro Beekman <kernel@jbeekman.nl> wrote:
> Traffic isn't really "disallowed" but rather some broadcast traffic is filtered.
>
> Signed-off-by: Jethro Beekman <kernel@jbeekman.nl>
> ---
> man/man8/ip-link.8.in | 8 +++++---
> 1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/man/man8/ip-link.8.in b/man/man8/ip-link.8.in
> index fd67e611..a4abae5f 100644
> --- a/man/man8/ip-link.8.in
> +++ b/man/man8/ip-link.8.in
> @@ -1366,10 +1366,12 @@ the following additional arguments are supported:
> .BR /dev/tapX " to be used just like a " tuntap " device."
>
> .B mode private
> -- Do not allow communication between
> +- Do not allow broadcast communication between
> .B macvlan
> instances on the same physical interface, even if the external switch supports
> -hairpin mode.
> +hairpin mode. Unicast traffic is transmitted over the physical interface as in
> +.B vepa
> +mode, but the lack of ARP responses may hamper communication.
The grammar here is a little awkward. It is using passive voice and the two clauses
in compound sentence don't match. Let me consult the grammar expert (my spouse is
a writer) and reword this in next release.
> .B mode vepa
> - Virtual Ethernet Port Aggregator mode. Data from one
> @@ -1394,7 +1396,7 @@ forces the underlying interface into promiscuous mode. Passing the
> using standard tools.
>
> .B mode source
> -- allows one to set a list of allowed mac address, which is used to match
> +- Allows one to set a list of allowed mac address, which is used to match
> against source mac address from received frames on underlying interface. This
> allows creating mac based VLAN associations, instead of standard port or tag
> based. The feature is useful to deploy 802.1x mac based behavior,
The original is also awkward wording here. Lets get rid of passive voice as well.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-04-27 4:02 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-24 21:28 [PATCH iproute2-next] ip: Clarify MACVLAN private mode Jethro Beekman
2021-04-27 4:02 ` 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.