On Tue, Mar 19, 2013 at 1:44 PM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Tue, 2013-03-19 at 04:08 -0700, Luis R. Rodriguez wrote: >> From: "Luis R. Rodriguez" <mcgrof@xxxxxxxxxxxxxxxx> >> >> If mac80211 tears down the BA sessions with del_timer_sync() >> technically mod_timer() can still rearm them. If mac80211 >> tears the BA sessions down we don't want to rearm the timers >> at a later time so avoid this possibility. > > Hmm, you may be onto something, but I'm not sure I want to apply this. > You don't seem to actually understand what you're doing -- two of the > three changes you're making don't really seem to be needed and the third > one should probably be fixed differently? If I get to review it further to the point I don't disappoint, sure, otherwise at least you now know of my suspicions. Luis -- 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