On Tue, 2010-10-05 at 20:52 +0200, Johannes Berg wrote: > --- wireless-testing.orig/net/mac80211/agg-tx.c 2010-10-05 20:40:02.000000000 +0200 > +++ wireless-testing/net/mac80211/agg-tx.c 2010-10-05 20:51:59.000000000 +0200 > @@ -163,6 +163,7 @@ int ___ieee80211_stop_tx_ba_session(stru > /* not even started yet! */ > rcu_assign_pointer(sta->ampdu_mlme.tid_tx[tid], NULL); > spin_unlock_bh(&sta->lock); > + del_timer_sync(&tid_tx->addba_resp_timer); > call_rcu(&tid_tx->rcu_head, kfree_tid_tx); > return 0; > } That one's unnecessary, if we get into this branch the timer can't have been started yet. johannes -- 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