[ath9k-devel] [PATCH] ath10k: remove stale peers upon vif removal

2013-05-08 Thread Michal Kazior
Fixes leak of ath10k_peer structures if FW stops
responding suddenly and we unload the driver (e.g.
eject the device while we're associated and unload
the driver).

Signed-off-by: Michal Kazior 
---
 drivers/net/wireless/ath/ath10k/mac.c |   22 ++
 1 file changed, 22 insertions(+)

diff --git a/drivers/net/wireless/ath/ath10k/mac.c 
b/drivers/net/wireless/ath/ath10k/mac.c
index 930a092..818b8e6 100644
--- a/drivers/net/wireless/ath/ath10k/mac.c
+++ b/drivers/net/wireless/ath/ath10k/mac.c
@@ -350,6 +350,26 @@ static int ath10k_peer_delete(struct ath10k *ar, u32 
vdev_id, const u8 *addr)
return 0;
 }
 
+static void ath10k_peer_cleanup(struct ath10k *ar, u32 vdev_id)
+{
+   struct ath10k_peer *peer, *tmp;
+
+   lockdep_assert_held(&ar->conf_mutex);
+
+   spin_lock_bh(&ar->data_lock);
+   list_for_each_entry_safe(peer, tmp, &ar->peers, list) {
+   if (peer->vdev_id != vdev_id)
+   continue;
+
+   ath10k_warn("removing stale peer %pM from vdev_id %d\n",
+   peer->addr, vdev_id);
+
+   list_del(&peer->list);
+   kfree(peer);
+   }
+   spin_unlock_bh(&ar->data_lock);
+}
+
 //
 /* Interface management */
 //
@@ -1773,6 +1793,8 @@ static void ath10k_remove_interface(struct ieee80211_hw 
*hw,
if (arvif->vdev_type == WMI_VDEV_TYPE_MONITOR)
ar->monitor_present = false;
 
+   ath10k_peer_cleanup(ar, arvif->vdev_id);
+
mutex_unlock(&ar->conf_mutex);
 }
 
-- 
1.7.9.5

___
ath9k-devel mailing list
ath9k-devel@lists.ath9k.org
https://lists.ath9k.org/mailman/listinfo/ath9k-devel


Re: [ath9k-devel] [PATCH] ath10k: remove stale peers upon vif removal

2013-05-09 Thread Kalle Valo
Michal Kazior  writes:

> Fixes leak of ath10k_peer structures if FW stops
> responding suddenly and we unload the driver (e.g.
> eject the device while we're associated and unload
> the driver).
>
> Signed-off-by: Michal Kazior 

Thanks, applied.

-- 
Kalle Valo
___
ath9k-devel mailing list
ath9k-devel@lists.ath9k.org
https://lists.ath9k.org/mailman/listinfo/ath9k-devel