Search Linux Wireless

Re: [RFC] mac80211: Add a timeout for frames in the RX reorder buffer

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Apr 30, 2009 at 2:41 AM, Jouni Malinen <j@xxxxx> wrote:
> This patch allows skbs to be released from the RX reorder buffer in
> case they have been there for an unexpectedly long time without us
> having received the missing frames before them. Previously, these
> frames were only released when the reorder window moved and that could
> take very long time unless new frames were received constantly (e.g.,
> TCP connections could be killed more or less indefinitely).
>
> This situation should not happen very frequently, but it looks like
> there are some scenarious that trigger for some reason. As such, this
> should be considered mostly a workaround to speed up recovery from
> unexpected siutation that could result in connections hanging for long
> periods of time.
>
> The changes here will only check for timeout situation when adding new
> RX frames to the reorder buffer. It does not handle all possible
> cases, but seems to help for most cases that could result from common
> network usage (e.g., TCP retrying at least couple of times). For more
> completely coverage, a timer could be used to periodically check
> whether there are any frames remaining in the reorder buffer if no new
> frames are received.
>
> Signed-off-by: Jouni Malinen <jouni.malinen@xxxxxxxxxxx>

Been testing this for a while and it helps. How about splitting the
add of ieee80211_release_reorder_frame() as a helper into a separate
patch to make the work around clearer smaller and clearer?

  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

[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux