cfg80211: use lockdep to assert lock is held
Instead of annotating with a comment, add a lockdep annotation which also serves as documentation. Signed-off-by: Johannes Berg <johannes.berg@intel.com>
This commit is contained in:
parent
6b684db1db
commit
4b1af4792a
|
@ -41,22 +41,24 @@ static void bss_release(struct kref *ref)
|
||||||
kfree(bss);
|
kfree(bss);
|
||||||
}
|
}
|
||||||
|
|
||||||
/* must hold dev->bss_lock! */
|
|
||||||
static void __cfg80211_unlink_bss(struct cfg80211_registered_device *dev,
|
static void __cfg80211_unlink_bss(struct cfg80211_registered_device *dev,
|
||||||
struct cfg80211_internal_bss *bss)
|
struct cfg80211_internal_bss *bss)
|
||||||
{
|
{
|
||||||
|
lockdep_assert_held(&dev->bss_lock);
|
||||||
|
|
||||||
list_del_init(&bss->list);
|
list_del_init(&bss->list);
|
||||||
rb_erase(&bss->rbn, &dev->bss_tree);
|
rb_erase(&bss->rbn, &dev->bss_tree);
|
||||||
kref_put(&bss->ref, bss_release);
|
kref_put(&bss->ref, bss_release);
|
||||||
}
|
}
|
||||||
|
|
||||||
/* must hold dev->bss_lock! */
|
|
||||||
static void __cfg80211_bss_expire(struct cfg80211_registered_device *dev,
|
static void __cfg80211_bss_expire(struct cfg80211_registered_device *dev,
|
||||||
unsigned long expire_time)
|
unsigned long expire_time)
|
||||||
{
|
{
|
||||||
struct cfg80211_internal_bss *bss, *tmp;
|
struct cfg80211_internal_bss *bss, *tmp;
|
||||||
bool expired = false;
|
bool expired = false;
|
||||||
|
|
||||||
|
lockdep_assert_held(&dev->bss_lock);
|
||||||
|
|
||||||
list_for_each_entry_safe(bss, tmp, &dev->bss_list, list) {
|
list_for_each_entry_safe(bss, tmp, &dev->bss_list, list) {
|
||||||
if (atomic_read(&bss->hold))
|
if (atomic_read(&bss->hold))
|
||||||
continue;
|
continue;
|
||||||
|
|
Loading…
Reference in New Issue