From: Amit Cohen <amco...@nvidia.com>

Currently mlxsw_sp_switchdev_vxlan_vlan_add() always calls
mlxsw_sp_bridge_8021q_vxlan_join() because VLANs were only ever added to
a VLAN-filtering bridge, which is only 802.1q bridge.

This set adds support for VxLAN with 802.1ad bridge, so VLAN-filtering
bridge is not only 802.1q.

Call ops->vxlan_join(), so mlxsw_sp_bridge_802{1q, 1ad}_vxlan_join()
will be called according to bridge type.

This is needed to ensure that VxLAN with 802.1ad bridge will be vetoed
in Spectrum-1 with the next patch.

Signed-off-by: Amit Cohen <amco...@nvidia.com>
Reviewed-by: Petr Machata <pe...@nvidia.com>
Signed-off-by: Ido Schimmel <ido...@nvidia.com>
---
 .../net/ethernet/mellanox/mlxsw/spectrum_switchdev.c   | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/drivers/net/ethernet/mellanox/mlxsw/spectrum_switchdev.c 
b/drivers/net/ethernet/mellanox/mlxsw/spectrum_switchdev.c
index 051a77440afe..73290f71eb9c 100644
--- a/drivers/net/ethernet/mellanox/mlxsw/spectrum_switchdev.c
+++ b/drivers/net/ethernet/mellanox/mlxsw/spectrum_switchdev.c
@@ -3320,8 +3320,8 @@ mlxsw_sp_switchdev_vxlan_vlan_add(struct mlxsw_sp 
*mlxsw_sp,
        if (!fid) {
                if (!flag_untagged || !flag_pvid)
                        return 0;
-               return mlxsw_sp_bridge_8021q_vxlan_join(bridge_device,
-                                                       vxlan_dev, vid, extack);
+               return bridge_device->ops->vxlan_join(bridge_device, vxlan_dev,
+                                                     vid, extack);
        }
 
        /* Second case: FID is associated with the VNI and the VLAN associated
@@ -3360,16 +3360,14 @@ mlxsw_sp_switchdev_vxlan_vlan_add(struct mlxsw_sp 
*mlxsw_sp,
        if (!flag_untagged)
                return 0;
 
-       err = mlxsw_sp_bridge_8021q_vxlan_join(bridge_device, vxlan_dev, vid,
-                                              extack);
+       err = bridge_device->ops->vxlan_join(bridge_device, vxlan_dev, vid, 
extack);
        if (err)
                goto err_vxlan_join;
 
        return 0;
 
 err_vxlan_join:
-       mlxsw_sp_bridge_8021q_vxlan_join(bridge_device, vxlan_dev, old_vid,
-                                        NULL);
+       bridge_device->ops->vxlan_join(bridge_device, vxlan_dev, old_vid, NULL);
        return err;
 }
 
-- 
2.28.0

Reply via email to