On 06/18/2013 08:52 AM, Johannes Berg wrote:
You mean ->current_bss? That should be handled in all the callbacks in
sme.c or so
Looks like much of the action happens on work-queues. I'm wondering if
we managed to delete wdev objects before we have completely cleaned up
in some cases...
Don't we flush work structs appropriately?
I'm still seeing leaks after all those patches I posted...so I'm still
looking. The leaked bss objects have a quite large refcount, in the
hundreds after an hour or so of running, so this is probably more than
a strange race somewhere.
This code looks questionable in wireless/mlme.c
int __cfg80211_mlme_assoc()
.....
It grabs a reference using cfg80211_get_bss, but it only
does a put on the reference if there was an error code.
The __cfg80211_mlme_auth a bit above always does a put on
the reference.
I'm thinking mlme_assoc should also always do put. Any
reason you can think of otherwise?
Thanks,
Ben
johannes
--
Ben Greear <greearb@xxxxxxxxxxxxxxx>
Candela Technologies Inc http://www.candelatech.com
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
