net/mlx5e: Vxlan, add sync lock for add/del vxlan port

Vxlan API can and will be called from different mlx5 modules, we should
not count on mlx5e private state lock only, hence we introduce a vxlan
private mutex to sync between add/del vxlan port operations.

Signed-off-by: Saeed Mahameed <saeedm@mellanox.com>
Reviewed-by: Or Gerlitz <ogerlitz@mellanox.com>

diff --git a/drivers/net/ethernet/mellanox/mlx5/core/vxlan.c b/drivers/net/ethernet/mellanox/mlx5/core/vxlan.c
index c9a50753ab234d98dd717a5eec261f461ef61002..9a8fd762167b3d5cdc66202395819e16b9bb1a99 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlx5/core/vxlan.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/vxlan.c
@@ -39,9 +39,10 @@
 struct mlx5_vxlan {
        struct mlx5_core_dev            *mdev;
        spinlock_t                      lock; /* protect vxlan table */
-       int                             num_ports;
        /* max_num_ports is usuallly 4, 16 buckets is more than enough */
        DECLARE_HASHTABLE(htable, 4);
+       int                             num_ports;
+       struct mutex                    sync_lock; /* sync add/del port HW operations */
 };
 
 struct mlx5_vxlan_port {
@@ -115,17 +116,18 @@ int mlx5_vxlan_add_port(struct mlx5_vxlan *vxlan, u16 port)
                return 0;
        }
 
+       mutex_lock(&vxlan->sync_lock);
        if (vxlan->num_ports >= mlx5_vxlan_max_udp_ports(vxlan->mdev)) {
                mlx5_core_info(vxlan->mdev,
                               "UDP port (%d) not offloaded, max number of UDP ports (%d) are already offloaded\n",
                               port, mlx5_vxlan_max_udp_ports(vxlan->mdev));
                ret = -ENOSPC;
-               return ret;
+               goto unlock;
        }
 
        ret = mlx5_vxlan_core_add_port_cmd(vxlan->mdev, port);
        if (ret)
-               return ret;
+               goto unlock;
 
        vxlanp = kzalloc(sizeof(*vxlanp), GFP_KERNEL);
        if (!vxlanp) {
@@ -141,10 +143,14 @@ int mlx5_vxlan_add_port(struct mlx5_vxlan *vxlan, u16 port)
        spin_unlock_bh(&vxlan->lock);
 
        vxlan->num_ports++;
+       mutex_unlock(&vxlan->sync_lock);
        return 0;
 
 err_delete_port:
        mlx5_vxlan_core_del_port_cmd(vxlan->mdev, port);
+
+unlock:
+       mutex_unlock(&vxlan->sync_lock);
        return ret;
 }
 
@@ -154,6 +160,8 @@ int mlx5_vxlan_del_port(struct mlx5_vxlan *vxlan, u16 port)
        bool remove = false;
        int ret = 0;
 
+       mutex_lock(&vxlan->sync_lock);
+
        spin_lock_bh(&vxlan->lock);
        vxlanp = mlx5_vxlan_lookup_port_locked(vxlan, port);
        if (!vxlanp) {
@@ -174,6 +182,9 @@ out_unlock:
                kfree(vxlanp);
                vxlan->num_ports--;
        }
+
+       mutex_unlock(&vxlan->sync_lock);
+
        return ret;
 }
 
@@ -189,6 +200,7 @@ struct mlx5_vxlan *mlx5_vxlan_create(struct mlx5_core_dev *mdev)
                return ERR_PTR(-ENOMEM);
 
        vxlan->mdev = mdev;
+       mutex_init(&vxlan->sync_lock);
        spin_lock_init(&vxlan->lock);
        hash_init(vxlan->htable);