diff options
| author | Vladimir Oltean <vladimir.oltean@nxp.com> | 2022-02-25 10:22:23 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2022-02-27 12:06:14 +0100 |
| commit | 06b9cce42634a50f2840777a66553b02320db5ef (patch) | |
| tree | 4ed5ab6bc3f7efe290468c12189575b85f3c2b0a /net/dsa/switch.c | |
| parent | net: dsa: request drivers to perform FDB isolation (diff) | |
| download | linux-06b9cce42634a50f2840777a66553b02320db5ef.tar.xz linux-06b9cce42634a50f2840777a66553b02320db5ef.zip | |
net: dsa: pass extack to .port_bridge_join driver methods
As FDB isolation cannot be enforced between VLAN-aware bridges in lack
of hardware assistance like extra FID bits, it seems plausible that many
DSA switches cannot do it. Therefore, they need to reject configurations
with multiple VLAN-aware bridges from the two code paths that can
transition towards that state:
- joining a VLAN-aware bridge
- toggling VLAN awareness on an existing bridge
The .port_vlan_filtering method already propagates the netlink extack to
the driver, let's propagate it from .port_bridge_join too, to make sure
that the driver can use the same function for both.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/dsa/switch.c')
| -rw-r--r-- | net/dsa/switch.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/net/dsa/switch.c b/net/dsa/switch.c index 1d3c161e3131..327d66bf7b47 100644 --- a/net/dsa/switch.c +++ b/net/dsa/switch.c @@ -96,7 +96,8 @@ static int dsa_switch_bridge_join(struct dsa_switch *ds, return -EOPNOTSUPP; err = ds->ops->port_bridge_join(ds, info->port, info->bridge, - &info->tx_fwd_offload); + &info->tx_fwd_offload, + info->extack); if (err) return err; } @@ -105,7 +106,8 @@ static int dsa_switch_bridge_join(struct dsa_switch *ds, ds->ops->crosschip_bridge_join) { err = ds->ops->crosschip_bridge_join(ds, info->tree_index, info->sw_index, - info->port, info->bridge); + info->port, info->bridge, + info->extack); if (err) return err; } |