net: dsa: {e}dsa: set offload_fwd_mark on received packets

lunn · davem330 · commit 13edbdb6ed8b · 2017-11-11T19:33:11.000+09:00
The software bridge needs to know if a packet has already been bridged
by hardware offload to ports in the same hardware offload, in order
that it does not re-flood them, causing duplicates. This is
particularly true for broadcast and multicast traffic which the host
has requested.

By setting offload_fwd_mark in the skb the bridge will only flood to
ports in other offloads and other netifs. Set this flag in the DSA and
EDSA tag driver.

Signed-off-by: Andrew Lunn &lt;andrew@lunn.ch&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
diff --git a/net/dsa/tag_dsa.c b/net/dsa/tag_dsa.c
@@ -141,6 +141,8 @@ static struct sk_buff *dsa_rcv(struct sk_buff *skb, struct net_device *dev,
 			2 * ETH_ALEN);
 	}
 
+	skb->offload_fwd_mark = 1;
+
 	return skb;
 }
 
diff --git a/net/dsa/tag_edsa.c b/net/dsa/tag_edsa.c
@@ -160,6 +160,8 @@ static struct sk_buff *edsa_rcv(struct sk_buff *skb, struct net_device *dev,
 			2 * ETH_ALEN);
 	}
 
+	skb->offload_fwd_mark = 1;
+
 	return skb;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -141,6 +141,8 @@ static struct sk_buff dsa_rcv(struct sk_buff skb, struct net_device *dev,`
`141`	`141`	`2 * ETH_ALEN);`
`142`	`142`	`}`
`143`	`143`
	`144`	`+ skb->offload_fwd_mark = 1;`
	`145`	`+`
`144`	`146`	`return skb;`
`145`	`147`	`}`
`146`	`148`
Original file line number	Diff line number	Diff line change
`@@ -160,6 +160,8 @@ static struct sk_buff edsa_rcv(struct sk_buff skb, struct net_device *dev,`
`160`	`160`	`2 * ETH_ALEN);`
`161`	`161`	`}`
`162`	`162`
	`163`	`+ skb->offload_fwd_mark = 1;`
	`164`	`+`
`163`	`165`	`return skb;`
`164`	`166`	`}`
`165`	`167`