All of lore.kernel.org
 help / color / mirror / Atom feed
From: Felix Fietkau <nbd@openwrt.org>
To: ath9k-devel@lists.ath9k.org
Subject: [ath9k-devel] Questions in 802.11 n timestamps traces for aggregated frames ?
Date: Sat, 16 Mar 2013 21:25:12 +0100	[thread overview]
Message-ID: <5144D528.9000403@openwrt.org> (raw)
In-Reply-To: <CAE+zGOh_K=w5YiyWhbitcxSkaj9RoQe6b81ZonLA1Po6tNFozQ@mail.gmail.com>

On 2013-03-16 9:19 PM, abhinav narain wrote:
> 
>     > guess aggr of 1462 bytes ?),
>     > but why are those seq. numbers never re-used ?
>     Your traces don't give you the full picture, because looking at this as
>     a per-frame thing is wrong.
>     If both IEEE80211_TX_CTL_AMPDU and IEEE80211_TX_STAT_AMPDU are set, it
>     means that the contained rate retry information and time stamp refer to
>     the whole A-MPDU, not just that one frame. Information about software
>     retry of frames is not passed on to mac80211 at the moment.
>     If you want accurate timestamp, rate and retry information, do it per
>     A-MPDU, not per MPDU.
> 
> Hi Felix, 
> I think I have understood you correctly, but let me know if I have got
> you wrong.
> 
> 
> 
> *594 *is the*first* frame of the aggregate and the *rate* information
> from its descriptor has to be used *for the whole aggregate frame* 594-603.
> Since the frames *595-603* seq. no. have the flag IEEE80211_TX_CTL_AMPDU
> set and flag  IEEE80211_TX_STAT_AMPDU not set, they are a part of the
> AMPDU starting at 594 and all the rate/retx information is ignored.
That's correct.

- Felix

  reply	other threads:[~2013-03-16 20:25 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-10 21:01 [ath9k-devel] Questions in 802.11 n timestamps traces for aggregated frames ? abhinav narain
2013-03-10 22:09 ` Adrian Chadd
2013-03-10 22:48   ` abhinav narain
2013-03-11  3:11     ` Adrian Chadd
2013-03-11 18:20       ` abhinav narain
2013-03-11 18:37         ` Felix Fietkau
2013-03-12 23:48           ` abhinav narain
2013-03-12 23:59             ` Felix Fietkau
2013-03-13 18:44               ` abhinav narain
2013-03-16 20:19               ` abhinav narain
2013-03-16 20:25                 ` Felix Fietkau [this message]
2013-03-12 23:59             ` Adrian Chadd
2013-03-13  0:13               ` abhinav narain

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=5144D528.9000403@openwrt.org \
    --to=nbd@openwrt.org \
    --cc=ath9k-devel@lists.ath9k.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.