net/mlx5: Disable irq when locking lag_lock

author Vlad Buslov <vladbu@nvidia.com>

Thu, 11 Aug 2022 11:46:36 +0000 (13:46 +0200)

committer Saeed Mahameed <saeedm@nvidia.com>

Mon, 22 Aug 2022 19:57:07 +0000 (12:57 -0700)
author Vlad Buslov <vladbu@nvidia.com>
Thu, 11 Aug 2022 11:46:36 +0000 (13:46 +0200)
committer Saeed Mahameed <saeedm@nvidia.com>
Mon, 22 Aug 2022 19:57:07 +0000 (12:57 -0700)
diff --git a/drivers/net/ethernet/mellanox/mlx5/core/lag/lag.c b/drivers/net/ethernet/mellanox/mlx5/core/lag/lag.c

index f67d29164962c78859dd3df46172a9487ac7b660..065102278cb8080a8059a930d34dd77aa0582e4f 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlx5/core/lag/lag.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/lag/lag.c
@@ -1067,30 +1067,32 @@ static void mlx5_ldev_add_netdev(struct mlx5_lag *ldev,
                                  struct net_device *netdev)
  {
         unsigned int fn = mlx5_get_dev_index(dev);
+       unsigned long flags;
  
         if (fn >= ldev->ports)
                 return;
  
-       spin_lock(&lag_lock);
+       spin_lock_irqsave(&lag_lock, flags);
         ldev->pf[fn].netdev = netdev;
         ldev->tracker.netdev_state[fn].link_up = 0;
         ldev->tracker.netdev_state[fn].tx_enabled = 0;
-       spin_unlock(&lag_lock);
+       spin_unlock_irqrestore(&lag_lock, flags);
  }
  
  static void mlx5_ldev_remove_netdev(struct mlx5_lag *ldev,
                                     struct net_device *netdev)
  {
+       unsigned long flags;
         int i;
  
-       spin_lock(&lag_lock);
+       spin_lock_irqsave(&lag_lock, flags);
         for (i = 0; i < ldev->ports; i++) {
                 if (ldev->pf[i].netdev == netdev) {
                         ldev->pf[i].netdev = NULL;
                         break;
                 }
         }
-       spin_unlock(&lag_lock);
+       spin_unlock_irqrestore(&lag_lock, flags);
  }
  
  static void mlx5_ldev_add_mdev(struct mlx5_lag *ldev,
@@ -1246,12 +1248,13 @@ void mlx5_lag_add_netdev(struct mlx5_core_dev *dev,
  bool mlx5_lag_is_roce(struct mlx5_core_dev *dev)
  {
         struct mlx5_lag *ldev;
+       unsigned long flags;
         bool res;
  
-       spin_lock(&lag_lock);
+       spin_lock_irqsave(&lag_lock, flags);
         ldev = mlx5_lag_dev(dev);
         res  = ldev && __mlx5_lag_is_roce(ldev);
-       spin_unlock(&lag_lock);
+       spin_unlock_irqrestore(&lag_lock, flags);
  
         return res;
  }
@@ -1260,12 +1263,13 @@ EXPORT_SYMBOL(mlx5_lag_is_roce);
  bool mlx5_lag_is_active(struct mlx5_core_dev *dev)
  {
         struct mlx5_lag *ldev;
+       unsigned long flags;
         bool res;
  
-       spin_lock(&lag_lock);
+       spin_lock_irqsave(&lag_lock, flags);
         ldev = mlx5_lag_dev(dev);
         res  = ldev && __mlx5_lag_is_active(ldev);
-       spin_unlock(&lag_lock);
+       spin_unlock_irqrestore(&lag_lock, flags);
  
         return res;
  }
@@ -1274,13 +1278,14 @@ EXPORT_SYMBOL(mlx5_lag_is_active);
  bool mlx5_lag_is_master(struct mlx5_core_dev *dev)
  {
         struct mlx5_lag *ldev;
+       unsigned long flags;
         bool res;
  
-       spin_lock(&lag_lock);
+       spin_lock_irqsave(&lag_lock, flags);
         ldev = mlx5_lag_dev(dev);
         res = ldev && __mlx5_lag_is_active(ldev) &&
                 dev == ldev->pf[MLX5_LAG_P1].dev;
-       spin_unlock(&lag_lock);
+       spin_unlock_irqrestore(&lag_lock, flags);
  
         return res;
  }
@@ -1289,12 +1294,13 @@ EXPORT_SYMBOL(mlx5_lag_is_master);
  bool mlx5_lag_is_sriov(struct mlx5_core_dev *dev)
  {
         struct mlx5_lag *ldev;
+       unsigned long flags;
         bool res;
  
-       spin_lock(&lag_lock);
+       spin_lock_irqsave(&lag_lock, flags);
         ldev = mlx5_lag_dev(dev);
         res  = ldev && __mlx5_lag_is_sriov(ldev);
-       spin_unlock(&lag_lock);
+       spin_unlock_irqrestore(&lag_lock, flags);
  
         return res;
  }
@@ -1303,13 +1309,14 @@ EXPORT_SYMBOL(mlx5_lag_is_sriov);
  bool mlx5_lag_is_shared_fdb(struct mlx5_core_dev *dev)
  {
         struct mlx5_lag *ldev;
+       unsigned long flags;
         bool res;
  
-       spin_lock(&lag_lock);
+       spin_lock_irqsave(&lag_lock, flags);
         ldev = mlx5_lag_dev(dev);
         res = ldev && __mlx5_lag_is_sriov(ldev) &&
               test_bit(MLX5_LAG_MODE_FLAG_SHARED_FDB, &ldev->mode_flags);
-       spin_unlock(&lag_lock);
+       spin_unlock_irqrestore(&lag_lock, flags);
  
         return res;
  }
@@ -1352,9 +1359,10 @@ struct net_device *mlx5_lag_get_roce_netdev(struct mlx5_core_dev *dev)
  {
         struct net_device *ndev = NULL;
         struct mlx5_lag *ldev;
+       unsigned long flags;
         int i;
  
-       spin_lock(&lag_lock);
+       spin_lock_irqsave(&lag_lock, flags);
         ldev = mlx5_lag_dev(dev);
  
         if (!(ldev && __mlx5_lag_is_roce(ldev)))
@@ -1373,7 +1381,7 @@ struct net_device *mlx5_lag_get_roce_netdev(struct mlx5_core_dev *dev)
                 dev_hold(ndev);
  
  unlock:
-       spin_unlock(&lag_lock);
+       spin_unlock_irqrestore(&lag_lock, flags);
  
         return ndev;
  }
@@ -1383,10 +1391,11 @@ u8 mlx5_lag_get_slave_port(struct mlx5_core_dev *dev,
                            struct net_device *slave)
  {
         struct mlx5_lag *ldev;
+       unsigned long flags;
         u8 port = 0;
         int i;
  
-       spin_lock(&lag_lock);
+       spin_lock_irqsave(&lag_lock, flags);
         ldev = mlx5_lag_dev(dev);
         if (!(ldev && __mlx5_lag_is_roce(ldev)))
                 goto unlock;
@@ -1401,7 +1410,7 @@ u8 mlx5_lag_get_slave_port(struct mlx5_core_dev *dev,
         port = ldev->v2p_map[port * ldev->buckets];
  
  unlock:
-       spin_unlock(&lag_lock);
+       spin_unlock_irqrestore(&lag_lock, flags);
         return port;
  }
  EXPORT_SYMBOL(mlx5_lag_get_slave_port);
@@ -1422,8 +1431,9 @@ struct mlx5_core_dev *mlx5_lag_get_peer_mdev(struct mlx5_core_dev *dev)
  {
         struct mlx5_core_dev *peer_dev = NULL;
         struct mlx5_lag *ldev;
+       unsigned long flags;
  
-       spin_lock(&lag_lock);
+       spin_lock_irqsave(&lag_lock, flags);
         ldev = mlx5_lag_dev(dev);
         if (!ldev)
                 goto unlock;
@@ -1433,7 +1443,7 @@ struct mlx5_core_dev *mlx5_lag_get_peer_mdev(struct mlx5_core_dev *dev)
                            ldev->pf[MLX5_LAG_P1].dev;
  
  unlock:
-       spin_unlock(&lag_lock);
+       spin_unlock_irqrestore(&lag_lock, flags);
         return peer_dev;
  }
  EXPORT_SYMBOL(mlx5_lag_get_peer_mdev);
@@ -1446,6 +1456,7 @@ int mlx5_lag_query_cong_counters(struct mlx5_core_dev *dev,
         int outlen = MLX5_ST_SZ_BYTES(query_cong_statistics_out);
         struct mlx5_core_dev **mdev;
         struct mlx5_lag *ldev;
+       unsigned long flags;
         int num_ports;
         int ret, i, j;
         void *out;
@@ -1462,7 +1473,7 @@ int mlx5_lag_query_cong_counters(struct mlx5_core_dev *dev,
  
         memset(values, 0, sizeof(*values) * num_counters);
  
-       spin_lock(&lag_lock);
+       spin_lock_irqsave(&lag_lock, flags);
         ldev = mlx5_lag_dev(dev);
         if (ldev && __mlx5_lag_is_active(ldev)) {
                 num_ports = ldev->ports;
@@ -1472,7 +1483,7 @@ int mlx5_lag_query_cong_counters(struct mlx5_core_dev *dev,
                 num_ports = 1;
                 mdev[MLX5_LAG_P1] = dev;
         }
-       spin_unlock(&lag_lock);
+       spin_unlock_irqrestore(&lag_lock, flags);
  
         for (i = 0; i < num_ports; ++i) {
                 u32 in[MLX5_ST_SZ_DW(query_cong_statistics_in)] = {};
author	Vlad Buslov <vladbu@nvidia.com>
	Thu, 11 Aug 2022 11:46:36 +0000 (13:46 +0200)
committer	Saeed Mahameed <saeedm@nvidia.com>
	Mon, 22 Aug 2022 19:57:07 +0000 (12:57 -0700)