On Wed, Mar 16, 2011 at 1:21 AM, Johannes Berg
[off-list ref] wrote:

On Tue, 2011-03-15 at 16:49 -0700, Daniel Halperin wrote:

quoted

x-T-R1-y-R2,

quoted

                      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