On Wed, Jun 27, 2012 at 12:23 PM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Mon, 2012-06-25 at 13:58 +0300, Eliad Peller wrote: >> On deauth/disassoc we tear down all BA sessions. These >> DELBA packets are sent on the appropriate TID, while >> deauth/disassoc is always sent on VO. This sometimes >> ends with the DELBA being sent after the deauth was >> already sent. >> >> Fix it by flushing all the pending frames before >> sending deauth/disassoc (in ieee80211_send_deauth_disassoc). >> >> Signed-off-by: Eliad Peller <eliad@xxxxxxxxxx> >> --- >> i think flushing in ieee80211_send_deauth_disassoc is the >> easiest thing to do here, as it handles all the (future) cases. >> however, i don't care moving the flush to a higher function >> if someone prefers it. > > I thought about this a bit ... Currently we have two calls to this > function, one from the actual set_disassoc() and one from > ieee80211_mgd_deauth(). In the latter case, we don't actually need the > flush I think, and in set_disassoc() we already have a flush call, so I > think it makes more sense there? > sure. i'll change it. Eliad. -- 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