From: Johannes Berg <johannes@sipsolutions.net>
To: Jouni Malinen <j@w1.fi>
Cc: linux-wireless@vger.kernel.org
Subject: Re: [RFC] per-sta reassembly buffers
Date: Thu, 06 Sep 2007 14:03:58 +0200 [thread overview]
Message-ID: <1189080238.28781.63.camel@johannes.berg> (raw)
In-Reply-To: <20070906020611.GJ5273@jm.kir.nu>
[-- Attachment #1: Type: text/plain, Size: 1953 bytes --]
On Wed, 2007-09-05 at 19:06 -0700, Jouni Malinen wrote:
> I would have expected STAs to transmit only one MSDU at a time to any
> given destination STA.
Yeah, me too, but I came to challenge that expectation recently :)
> This was the way it was done before QoS and I did
> not find any clear point in QoS changes that would have changed the
> behavior. The changes in the duplicate detection are, though, implicitly
> pointing out that there may actually be separate fragmentation for each
> TID.
Right. Not very specific. Maybe something for 802.11ma, if it hadn't
been dissolved ;)
> The main reason for the original design was in being conservative with
> memory use. After all, this was used for embedded devices with very
> limited memory capabilities and the AP was actually allowing up to 2007
> STAs to be associated. If each associated STA were to use fragmentation
> and if there were large number of lost (un-ACK'ed) frames, the total
> memory buffers needed for the frames could go quite high.
Yes, I understand, hence my question.
> Since fragments are commonly sent out in a burst that includes the full
> MSDU, the number of pending fragments from unfinished MSDUs is often
> quite a bit smaller than the worst case.
Obviously. Since we're talking about a single station having a
fragmented frame outstanding in each access category, this would mean
that others stations would have to contend very much in the voice
category to even allow this situation to happen. Not very likely.
> Anyway, I would be fine with changing this as long as there is a way of
> enforcing the total number of buffered frames to be below a reasonable
> (and preferably configurable) limit.
Good plan actually, implementing it would probably require having the
skbs on two lists: one per station to lookup and one global list to age
them. Shouldn't be hard to implement at all, I'll look into it.
johannes
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 190 bytes --]
prev parent reply other threads:[~2007-09-06 12:02 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-09-05 10:28 [RFC] per-sta reassembly buffers Johannes Berg
2007-09-06 2:06 ` Jouni Malinen
2007-09-06 12:03 ` Johannes Berg [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1189080238.28781.63.camel@johannes.berg \
--to=johannes@sipsolutions.net \
--cc=j@w1.fi \
--cc=linux-wireless@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox