net/mlx4_en: Protect access to the statistics bitmap
authorEran Ben Elisha <eranbe@mellanox.com>
Mon, 30 Mar 2015 14:45:24 +0000 (17:45 +0300)
committerDavid S. Miller <davem@davemloft.net>
Tue, 31 Mar 2015 20:36:50 +0000 (16:36 -0400)
This will allow parallel access to the statistics bitmap.
A pre-step for adding PFC counters, where the statistics bitmap
can be dynamically changed when modifying the PFC setting.

Signed-off-by: Eran Ben Elisha <eranbe@mellanox.com>
Signed-off-by: Hadar Hen Zion <hadarh@mellanox.com>
Signed-off-by: Or Gerlitz <ogerlitz@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/mellanox/mlx4/en_ethtool.c
drivers/net/ethernet/mellanox/mlx4/en_netdev.c
drivers/net/ethernet/mellanox/mlx4/mlx4_en.h

index 1a29329d52c629f043ddee2229857cfed943760a..3e7ed39e8e76b34f0d6f307cf4cb57d00aa428fc 100644 (file)
@@ -263,7 +263,7 @@ static int mlx4_en_get_sset_count(struct net_device *dev, int sset)
        struct mlx4_en_priv *priv = netdev_priv(dev);
        struct bitmap_iterator it;
 
-       bitmap_iterator_init(&it, priv->stats_bitmap, NUM_ALL_STATS);
+       bitmap_iterator_init(&it, priv->stats_bitmap.bitmap, NUM_ALL_STATS);
 
        switch (sset) {
        case ETH_SS_STATS:
@@ -292,7 +292,7 @@ static void mlx4_en_get_ethtool_stats(struct net_device *dev,
        int i;
        struct bitmap_iterator it;
 
-       bitmap_iterator_init(&it, priv->stats_bitmap, NUM_ALL_STATS);
+       bitmap_iterator_init(&it, priv->stats_bitmap.bitmap, NUM_ALL_STATS);
 
        spin_lock_bh(&priv->stats_lock);
 
@@ -339,7 +339,7 @@ static void mlx4_en_get_strings(struct net_device *dev,
        int i, strings = 0;
        struct bitmap_iterator it;
 
-       bitmap_iterator_init(&it, priv->stats_bitmap, NUM_ALL_STATS);
+       bitmap_iterator_init(&it, priv->stats_bitmap.bitmap, NUM_ALL_STATS);
 
        switch (stringset) {
        case ETH_SS_TEST:
index b7710321a2ca722f0d95e29ae38cb0e4141b9740..4542bab9494b56068f03e9da93d68b4facc36af8 100644 (file)
@@ -2649,35 +2649,36 @@ int mlx4_en_netdev_event(struct notifier_block *this,
 }
 
 void mlx4_en_set_stats_bitmap(struct mlx4_dev *dev,
-                             unsigned long *stats_bitmap)
+                             struct mlx4_en_stats_bitmap *stats_bitmap)
 {
        int last_i = 0;
 
-       bitmap_zero(stats_bitmap, NUM_ALL_STATS);
+       mutex_init(&stats_bitmap->mutex);
+       bitmap_zero(stats_bitmap->bitmap, NUM_ALL_STATS);
 
        if (mlx4_is_slave(dev)) {
-               bitmap_set(stats_bitmap, last_i +
+               bitmap_set(stats_bitmap->bitmap, last_i +
                                         MLX4_FIND_NETDEV_STAT(rx_packets), 1);
-               bitmap_set(stats_bitmap, last_i +
+               bitmap_set(stats_bitmap->bitmap, last_i +
                                         MLX4_FIND_NETDEV_STAT(tx_packets), 1);
-               bitmap_set(stats_bitmap, last_i +
+               bitmap_set(stats_bitmap->bitmap, last_i +
                                         MLX4_FIND_NETDEV_STAT(rx_bytes), 1);
-               bitmap_set(stats_bitmap, last_i +
+               bitmap_set(stats_bitmap->bitmap, last_i +
                                         MLX4_FIND_NETDEV_STAT(tx_bytes), 1);
-               bitmap_set(stats_bitmap, last_i +
+               bitmap_set(stats_bitmap->bitmap, last_i +
                                         MLX4_FIND_NETDEV_STAT(rx_dropped), 1);
-               bitmap_set(stats_bitmap, last_i +
+               bitmap_set(stats_bitmap->bitmap, last_i +
                                         MLX4_FIND_NETDEV_STAT(tx_dropped), 1);
        } else {
-               bitmap_set(stats_bitmap, last_i, NUM_MAIN_STATS);
+               bitmap_set(stats_bitmap->bitmap, last_i, NUM_MAIN_STATS);
        }
        last_i += NUM_MAIN_STATS;
 
-       bitmap_set(stats_bitmap, last_i, NUM_PORT_STATS);
+       bitmap_set(stats_bitmap->bitmap, last_i, NUM_PORT_STATS);
        last_i += NUM_PORT_STATS;
 
        if (!mlx4_is_slave(dev))
-               bitmap_set(stats_bitmap, last_i, NUM_PKT_STATS);
+               bitmap_set(stats_bitmap->bitmap, last_i, NUM_PKT_STATS);
 }
 
 int mlx4_en_init_netdev(struct mlx4_en_dev *mdev, int port,
@@ -2913,7 +2914,7 @@ int mlx4_en_init_netdev(struct mlx4_en_dev *mdev, int port,
                queue_delayed_work(mdev->workqueue, &priv->service_task,
                                   SERVICE_TASK_DELAY);
 
-       mlx4_en_set_stats_bitmap(mdev->dev, priv->stats_bitmap);
+       mlx4_en_set_stats_bitmap(mdev->dev, &priv->stats_bitmap);
 
        return 0;
 
index a612bfd921f7b6c2a8d74f1238456bf64d3a66b7..d5d971a408f217f91d14c2a89441dd9048cd21cc 100644 (file)
@@ -485,6 +485,11 @@ enum {
 #define MLX4_EN_MAC_HASH_SIZE (1 << BITS_PER_BYTE)
 #define MLX4_EN_MAC_HASH_IDX 5
 
+struct mlx4_en_stats_bitmap {
+       DECLARE_BITMAP(bitmap, NUM_ALL_STATS);
+       struct mutex mutex; /* for mutual access to stats bitmap */
+};
+
 struct mlx4_en_priv {
        struct mlx4_en_dev *mdev;
        struct mlx4_en_port_profile *prof;
@@ -561,7 +566,7 @@ struct mlx4_en_priv {
        struct mlx4_en_perf_stats pstats;
        struct mlx4_en_pkt_stats pkstats;
        struct mlx4_en_port_stats port_stats;
-       DECLARE_BITMAP(stats_bitmap, NUM_ALL_STATS);
+       struct mlx4_en_stats_bitmap stats_bitmap;
        struct list_head mc_list;
        struct list_head curr_list;
        u64 broadcast_id;
@@ -731,7 +736,7 @@ int mlx4_en_start_port(struct net_device *dev);
 void mlx4_en_stop_port(struct net_device *dev, int detach);
 
 void mlx4_en_set_stats_bitmap(struct mlx4_dev *dev,
-                             unsigned long *stats_bitmap);
+                             struct mlx4_en_stats_bitmap *stats_bitmap);
 
 void mlx4_en_free_resources(struct mlx4_en_priv *priv);
 int mlx4_en_alloc_resources(struct mlx4_en_priv *priv);