diff mbox series

[v3,net-next,7/7] net: dsa: ocelot: request DSA to fix up lack of address learning on CPU port

Message ID 20201213140710.1198050-8-vladimir.oltean@nxp.com
State New
Headers show
Series Offload software learnt bridge addresses to DSA | expand

Commit Message

Vladimir Oltean Dec. 13, 2020, 2:07 p.m. UTC
Given the following setup:

ip link add br0 type bridge
ip link set eno0 master br0
ip link set swp0 master br0
ip link set swp1 master br0
ip link set swp2 master br0
ip link set swp3 master br0

Currently, packets received on a DSA slave interface (such as swp0)
which should be routed by the software bridge towards a non-switch port
(such as eno0) are also flooded towards the other switch ports (swp1,
swp2, swp3) because the destination is unknown to the hardware switch.

This patch addresses the issue by monitoring the addresses learnt by the
software bridge on eno0, and adding/deleting them as static FDB entries
on the CPU port accordingly.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
---
Changes in v2:
s/learning_broken/assisted_learning/

Changes in v2:
Patch is new.

 drivers/net/dsa/ocelot/felix.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Andrew Lunn Dec. 16, 2020, 3:43 p.m. UTC | #1
On Sun, Dec 13, 2020 at 04:07:10PM +0200, Vladimir Oltean wrote:
> Given the following setup:

> 

> ip link add br0 type bridge

> ip link set eno0 master br0

> ip link set swp0 master br0

> ip link set swp1 master br0

> ip link set swp2 master br0

> ip link set swp3 master br0

> 

> Currently, packets received on a DSA slave interface (such as swp0)

> which should be routed by the software bridge towards a non-switch port

> (such as eno0) are also flooded towards the other switch ports (swp1,

> swp2, swp3) because the destination is unknown to the hardware switch.

> 

> This patch addresses the issue by monitoring the addresses learnt by the

> software bridge on eno0, and adding/deleting them as static FDB entries

> on the CPU port accordingly.

> 

> Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>

> Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>


Reviewed-by: Andrew Lunn <andrew@lunn.ch>


    Andrew
diff mbox series

Patch

diff --git a/drivers/net/dsa/ocelot/felix.c b/drivers/net/dsa/ocelot/felix.c
index 7dc230677b78..90c3c76f21b2 100644
--- a/drivers/net/dsa/ocelot/felix.c
+++ b/drivers/net/dsa/ocelot/felix.c
@@ -629,6 +629,7 @@  static int felix_setup(struct dsa_switch *ds)
 
 	ds->mtu_enforcement_ingress = true;
 	ds->configure_vlan_while_not_filtering = true;
+	ds->assisted_learning_on_cpu_port = true;
 
 	return 0;
 }