On Wed, 2012-02-01 at 14:06 +0200, Eliad Peller wrote: > The current code checks for stored_mpdu_num > 1, causing > the reorder_timer to be triggered indefinitely, but the > frame is never timed-out (until the next packet is received) Acked-by: Johannes Berg <johannes@xxxxxxxxxxxxxxxx> Looks kinda obvious & trivial, want to CC stable? > Signed-off-by: Eliad Peller <eliad@xxxxxxxxxx> > --- > net/mac80211/rx.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/net/mac80211/rx.c b/net/mac80211/rx.c > index f407427..7e3980f 100644 > --- a/net/mac80211/rx.c > +++ b/net/mac80211/rx.c > @@ -611,7 +611,7 @@ static void ieee80211_sta_reorder_release(struct ieee80211_hw *hw, > index = seq_sub(tid_agg_rx->head_seq_num, tid_agg_rx->ssn) % > tid_agg_rx->buf_size; > if (!tid_agg_rx->reorder_buf[index] && > - tid_agg_rx->stored_mpdu_num > 1) { > + tid_agg_rx->stored_mpdu_num) { > /* > * No buffers ready to be released, but check whether any > * frames in the reorder buffer have timed out. -- 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