On Tue, 2009-05-12 at 01:09 +0200, Christian Lamparter wrote:
> + if ((ar->tx_stats[i].len < ar->tx_stats[i].limit) &&
> + (ieee80211_queue_stopped(ar->hw, i)))
> + ieee80211_wake_queue(ar->hw, i);
There's no reason to test _stopped first, and doing so is racy.
> if (changed & IEEE80211_CONF_CHANGE_CHANNEL) {
> + err = ar9170_stop_and_flush_queues(ar);
> + if (err)
> + goto out;
Why don't you do that in the pre-scan method?
johannes
Attachment:
signature.asc
Description: This is a digitally signed message part
