On Wed, Mar 16, 2011 at 1:21 AM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Tue, 2011-03-15 at 16:49 -0700, Daniel Halperin wrote: > >> x-T-R1-y-R2, > >> if (!time_after(jiffies, tid_agg_rx->reorder_time[j] + >> - HT_RX_REORDER_BUF_TIMEOUT)) >> + HT_RX_REORDER_BUF_TIMEOUT) && skipped) > > Wait, your previous example of xT---R worked fine, but if you say x-T- > this patch won't work I think? Basically you're saying if we received > frames 2 and 4 after 3, and 3 times out, we can release 2 through 4. I > agree, but your code won't do that since skipped starts out at 1 due to > the first x. Or am I misreading this? > The syntax I used in our private mail was confusing and I tried to simplify. Here, hyphens are just separators between frames (since R1, R2 have two-letter identifiers). Does that clear it up? Dan -- 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
