ath10k: clean up num_peers locking

The var was supposed to be protected by data_lock
but it wasn't so in all instances. It's actually
not necessary to have a spinlock protected
num_peers so drop it.

All instances of num_peers are already within
conf_mutex sections so use that.

Signed-off-by: Michal Kazior <michal.kazior@tieto.com>
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>

diff --git a/drivers/net/wireless/ath/ath10k/core.h b/drivers/net/wireless/ath/ath10k/core.h
index cd8815a..0a01821 100644 (file)
--- a/drivers/net/wireless/ath/ath10k/core.h
+++ b/drivers/net/wireless/ath/ath10k/core.h
@@ -564,7 +564,7 @@ struct ath10k {
        struct list_head peers;
        wait_queue_head_t peer_mapping_wq;
 
        struct list_head peers;
        wait_queue_head_t peer_mapping_wq;
 

-       /* number of created peers; protected by data_lock */
+       /* protected by conf_mutex */

        int num_peers;
 
        struct work_struct offchan_tx_work;
        int num_peers;
 
        struct work_struct offchan_tx_work;

diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c
index f364b84..bb138ce 100644 (file)
--- a/drivers/net/wireless/ath/ath10k/mac.c
+++ b/drivers/net/wireless/ath/ath10k/mac.c
@@ -368,9 +368,8 @@ static int ath10k_peer_create(struct ath10k *ar, u32 vdev_id, const u8 *addr)
                            addr, vdev_id, ret);
                return ret;
        }
                            addr, vdev_id, ret);
                return ret;
        }

-       spin_lock_bh(&ar->data_lock);
+

        ar->num_peers++;
        ar->num_peers++;

-       spin_unlock_bh(&ar->data_lock);

 
        return 0;
 }
 
        return 0;
 }


@@ -461,9 +460,7 @@ static int ath10k_peer_delete(struct ath10k *ar, u32 vdev_id, const u8 *addr)
        if (ret)
                return ret;
 
        if (ret)
                return ret;
 

-       spin_lock_bh(&ar->data_lock);

        ar->num_peers--;
        ar->num_peers--;

-       spin_unlock_bh(&ar->data_lock);

 
        return 0;
 }
 
        return 0;
 }


@@ -500,8 +497,9 @@ static void ath10k_peer_cleanup_all(struct ath10k *ar)
                list_del(&peer->list);
                kfree(peer);
        }
                list_del(&peer->list);
                kfree(peer);
        }

-       ar->num_peers = 0;

        spin_unlock_bh(&ar->data_lock);
        spin_unlock_bh(&ar->data_lock);

+
+       ar->num_peers = 0;

 }
 
 /************************/
 }
 
 /************************/