* [PATCH v2] net: bridge: allow IPv6 when multicast flood is disabled
@ 2017-03-01 9:55 Mike Manning
2017-03-01 11:06 ` [Bridge] " Nikolay Aleksandrov
2017-03-02 4:57 ` David Miller
0 siblings, 2 replies; 4+ messages in thread
From: Mike Manning @ 2017-03-01 9:55 UTC (permalink / raw)
To: netdev; +Cc: Nikolay Aleksandrov
Even with multicast flooding turned off, IPv6 ND should still work so
that IPv6 connectivity is provided. Allow this by continuing to flood
multicast traffic originated by us.
Fixes: b6cb5ac8331b ("net: bridge: add per-port multicast flood flag")
Cc: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
Signed-off-by: Mike Manning <mmanning@brocade.com>
---
net/bridge/br_forward.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/net/bridge/br_forward.c b/net/bridge/br_forward.c
index 6bfac29..902af6b 100644
--- a/net/bridge/br_forward.c
+++ b/net/bridge/br_forward.c
@@ -186,8 +186,9 @@ void br_flood(struct net_bridge *br, struct sk_buff *skb,
/* Do not flood unicast traffic to ports that turn it off */
if (pkt_type == BR_PKT_UNICAST && !(p->flags & BR_FLOOD))
continue;
+ /* Do not flood if mc off, except for traffic we originate */
if (pkt_type == BR_PKT_MULTICAST &&
- !(p->flags & BR_MCAST_FLOOD))
+ !(p->flags & BR_MCAST_FLOOD) && skb->dev != br->dev)
continue;
/* Do not flood to ports that enable proxy ARP */
--
2.1.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2] net: bridge: allow IPv6 when multicast flood is disabled
2017-03-01 9:55 [PATCH v2] net: bridge: allow IPv6 when multicast flood is disabled Mike Manning
@ 2017-03-01 11:06 ` Nikolay Aleksandrov
2017-03-02 4:57 ` David Miller
1 sibling, 0 replies; 4+ messages in thread
From: Nikolay Aleksandrov via Bridge @ 2017-03-01 11:06 UTC (permalink / raw)
To: Mike Manning, netdev, Stephen Hemminger, bridge
On 01/03/17 11:55, Mike Manning wrote:
> Even with multicast flooding turned off, IPv6 ND should still work so
> that IPv6 connectivity is provided. Allow this by continuing to flood
> multicast traffic originated by us.
>
> Fixes: b6cb5ac8331b ("net: bridge: add per-port multicast flood flag")
> Cc: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
> Signed-off-by: Mike Manning <mmanning@brocade.com>
> ---
> net/bridge/br_forward.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/net/bridge/br_forward.c b/net/bridge/br_forward.c
> index 6bfac29..902af6b 100644
> --- a/net/bridge/br_forward.c
> +++ b/net/bridge/br_forward.c
> @@ -186,8 +186,9 @@ void br_flood(struct net_bridge *br, struct sk_buff *skb,
> /* Do not flood unicast traffic to ports that turn it off */
> if (pkt_type == BR_PKT_UNICAST && !(p->flags & BR_FLOOD))
> continue;
> + /* Do not flood if mc off, except for traffic we originate */
> if (pkt_type == BR_PKT_MULTICAST &&
> - !(p->flags & BR_MCAST_FLOOD))
> + !(p->flags & BR_MCAST_FLOOD) && skb->dev != br->dev)
> continue;
>
> /* Do not flood to ports that enable proxy ARP */
>
+CC bridge maintainers
Acked-by: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Bridge] [PATCH v2] net: bridge: allow IPv6 when multicast flood is disabled
@ 2017-03-01 11:06 ` Nikolay Aleksandrov
0 siblings, 0 replies; 4+ messages in thread
From: Nikolay Aleksandrov @ 2017-03-01 11:06 UTC (permalink / raw)
To: Mike Manning, netdev, Stephen Hemminger, bridge
On 01/03/17 11:55, Mike Manning wrote:
> Even with multicast flooding turned off, IPv6 ND should still work so
> that IPv6 connectivity is provided. Allow this by continuing to flood
> multicast traffic originated by us.
>
> Fixes: b6cb5ac8331b ("net: bridge: add per-port multicast flood flag")
> Cc: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
> Signed-off-by: Mike Manning <mmanning@brocade.com>
> ---
> net/bridge/br_forward.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/net/bridge/br_forward.c b/net/bridge/br_forward.c
> index 6bfac29..902af6b 100644
> --- a/net/bridge/br_forward.c
> +++ b/net/bridge/br_forward.c
> @@ -186,8 +186,9 @@ void br_flood(struct net_bridge *br, struct sk_buff *skb,
> /* Do not flood unicast traffic to ports that turn it off */
> if (pkt_type == BR_PKT_UNICAST && !(p->flags & BR_FLOOD))
> continue;
> + /* Do not flood if mc off, except for traffic we originate */
> if (pkt_type == BR_PKT_MULTICAST &&
> - !(p->flags & BR_MCAST_FLOOD))
> + !(p->flags & BR_MCAST_FLOOD) && skb->dev != br->dev)
> continue;
>
> /* Do not flood to ports that enable proxy ARP */
>
+CC bridge maintainers
Acked-by: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] net: bridge: allow IPv6 when multicast flood is disabled
2017-03-01 9:55 [PATCH v2] net: bridge: allow IPv6 when multicast flood is disabled Mike Manning
2017-03-01 11:06 ` [Bridge] " Nikolay Aleksandrov
@ 2017-03-02 4:57 ` David Miller
1 sibling, 0 replies; 4+ messages in thread
From: David Miller @ 2017-03-02 4:57 UTC (permalink / raw)
To: mmanning; +Cc: netdev, nikolay
From: Mike Manning <mmanning@brocade.com>
Date: Wed, 1 Mar 2017 09:55:28 +0000
> Even with multicast flooding turned off, IPv6 ND should still work so
> that IPv6 connectivity is provided. Allow this by continuing to flood
> multicast traffic originated by us.
>
> Fixes: b6cb5ac8331b ("net: bridge: add per-port multicast flood flag")
> Cc: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
> Signed-off-by: Mike Manning <mmanning@brocade.com>
Applied and queued up for -stable.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-03-02 6:14 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-01 9:55 [PATCH v2] net: bridge: allow IPv6 when multicast flood is disabled Mike Manning
2017-03-01 11:06 ` Nikolay Aleksandrov via Bridge
2017-03-01 11:06 ` [Bridge] " Nikolay Aleksandrov
2017-03-02 4:57 ` David Miller
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.