]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commitdiff
net/mlx5: EQ, Remove redundant completion EQ list lock
authorSaeed Mahameed <saeedm@mellanox.com>
Mon, 19 Nov 2018 18:52:34 +0000 (10:52 -0800)
committerLeon Romanovsky <leonro@mellanox.com>
Tue, 20 Nov 2018 18:06:23 +0000 (20:06 +0200)
Completion EQs list is only modified on driver load/unload, locking is
not required, remove it.

Signed-off-by: Saeed Mahameed <saeedm@mellanox.com>
Reviewed-by: Leon Romanovsky <leonro@mellanox.com>
Reviewed-by: Tariq Toukan <tariqt@mellanox.com>
Signed-off-by: Leon Romanovsky <leonro@mellanox.com>
drivers/net/ethernet/mellanox/mlx5/core/eq.c
drivers/net/ethernet/mellanox/mlx5/core/main.c
include/linux/mlx5/driver.h

index fd5926daa0a6f0fbaa6fec6f331c95243b6bae36..e75272503027c148c5cae066bdce30d9757ab221 100644 (file)
@@ -810,8 +810,6 @@ int mlx5_eq_init(struct mlx5_core_dev *dev)
 {
        int err;
 
-       spin_lock_init(&dev->priv.eq_table.lock);
-
        err = mlx5_eq_debugfs_init(dev);
 
        return err;
index f5e6d375a8cc57376a5f3525d3bf8df31d3ab8e2..f692c2a421303de39754ba65f3c70597650110a4 100644 (file)
@@ -704,7 +704,6 @@ int mlx5_vector2eqn(struct mlx5_core_dev *dev, int vector, int *eqn,
        int err = -ENOENT;
        int i = 0;
 
-       spin_lock(&table->lock);
        list_for_each_entry_safe(eq, n, &table->comp_eqs_list, list) {
                if (i++ == vector) {
                        *eqn = eq->eqn;
@@ -713,7 +712,6 @@ int mlx5_vector2eqn(struct mlx5_core_dev *dev, int vector, int *eqn,
                        break;
                }
        }
-       spin_unlock(&table->lock);
 
        return err;
 }
@@ -724,14 +722,11 @@ struct mlx5_eq *mlx5_eqn2eq(struct mlx5_core_dev *dev, int eqn)
        struct mlx5_eq_table *table = &dev->priv.eq_table;
        struct mlx5_eq *eq;
 
-       spin_lock(&table->lock);
-       list_for_each_entry(eq, &table->comp_eqs_list, list)
-               if (eq->eqn == eqn) {
-                       spin_unlock(&table->lock);
+       list_for_each_entry(eq, &table->comp_eqs_list, list) {
+               if (eq->eqn == eqn)
                        return eq;
-               }
+       }
 
-       spin_unlock(&table->lock);
 
        return ERR_PTR(-ENOENT);
 }
@@ -747,17 +742,13 @@ static void free_comp_eqs(struct mlx5_core_dev *dev)
                dev->rmap = NULL;
        }
 #endif
-       spin_lock(&table->lock);
        list_for_each_entry_safe(eq, n, &table->comp_eqs_list, list) {
                list_del(&eq->list);
-               spin_unlock(&table->lock);
                if (mlx5_destroy_unmap_eq(dev, eq))
                        mlx5_core_warn(dev, "failed to destroy EQ 0x%x\n",
                                       eq->eqn);
                kfree(eq);
-               spin_lock(&table->lock);
        }
-       spin_unlock(&table->lock);
 }
 
 static int alloc_comp_eqs(struct mlx5_core_dev *dev)
@@ -798,9 +789,7 @@ static int alloc_comp_eqs(struct mlx5_core_dev *dev)
                        goto clean;
                }
                mlx5_core_dbg(dev, "allocated completion EQN %d\n", eq->eqn);
-               spin_lock(&table->lock);
                list_add_tail(&eq->list, &table->comp_eqs_list);
-               spin_unlock(&table->lock);
        }
 
        return 0;
index 4b62d71825c1dbbabf56e7786f40d4f1a44a798a..852e397c762457786006a6d0070c256d138ff387 100644 (file)
@@ -484,9 +484,6 @@ struct mlx5_eq_table {
        struct mlx5_eq          pfault_eq;
 #endif
        int                     num_comp_vectors;
-       /* protect EQs list
-        */
-       spinlock_t              lock;
 };
 
 struct mlx5_uars_page {