Linux wireless drivers development
 help / color / mirror / Atom feed
From: Simon Wunderlich <simon.wunderlich@s2003.tu-chemnitz.de>
To: Helmut Schaa <helmut.schaa@googlemail.com>
Cc: Simon Wunderlich <simon.wunderlich@s2003.tu-chemnitz.de>,
	linux-wireless@vger.kernel.org, linville@tuxdriver.com,
	Johannes Berg <johannes@sipsolutions.net>,
	Simon Wunderlich <siwu@hrz.tu-chemnitz.de>,
	Mathias Kretschmer <mathias.kretschmer@fokus.fraunhofer.de>
Subject: Re: [PATCH] mac80211: Add NoAck per WMM Queue Support
Date: Wed, 16 Nov 2011 16:21:17 +0100	[thread overview]
Message-ID: <20111116152117.GB10956@pandem0nium> (raw)
In-Reply-To: <CAGXE3d-Q_8zXX7dHp=eK=wnK7b_AZ3M+fZNekzu50ZOpJoQ9Rw@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1331 bytes --]

Hello Helmut,

thanks for pointing that out!

As far as I understand minstrel, it is using sent packets and ack statistics
to find out the percentage of successfulness. Therefore, minstrel should
probably not consider NOACK frames for its statistics.

If there are only AC_VO frames, there would not really be any data for
minstrel to build its statistics, so one could either rely on the
control traffic which may go over another queue or use fixed rates.
For example, SIP uses RTP for the voice data and SIP packets for
session control ...

best regards,
        Simon

[Helmut, sorry for the resend, I accidently replied personally to only you first]

On Wed, Nov 16, 2011 at 02:58:18PM +0100, Helmut Schaa wrote:
> On Wed, Nov 16, 2011 at 12:21 AM, Simon Wunderlich
> <simon.wunderlich@s2003.tu-chemnitz.de> wrote:
> > This patch adds support for NoAck per WMM Queue. The Unicast QoS
> > Header is adapted accordingly for each outgoing frame.
> > The support is turned on and off through nl80211 by extending
> > the WMM TX Queue Parameters, but can be triggered separately.
> 
> Just a thought: Assuming you've configured AC_VO to use NoAck and
> you've only got VO traffic going on, is minstrel still able to collect enough
> statistics to do proper rate selection?
> 
> Thanks,
> Helmut
> 

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

      reply	other threads:[~2011-11-16 15:21 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-15 23:21 [PATCH] mac80211: Add NoAck per WMM Queue Support Simon Wunderlich
2011-11-15 23:21 ` [PATCH] iw: add support for wmm NoAck per WMM Queue Simon Wunderlich
2011-11-16  7:46 ` [PATCH] mac80211: Add NoAck per WMM Queue Support Felix Fietkau
2011-11-16  7:50   ` Johannes Berg
2011-11-16  7:52 ` Johannes Berg
2011-11-16 10:51   ` Simon Wunderlich
2011-11-16 10:57     ` Johannes Berg
2011-11-16 12:29       ` Simon Wunderlich
2011-11-16 13:58 ` Helmut Schaa
2011-11-16 15:21   ` Simon Wunderlich [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=20111116152117.GB10956@pandem0nium \
    --to=simon.wunderlich@s2003.tu-chemnitz.de \
    --cc=helmut.schaa@googlemail.com \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=mathias.kretschmer@fokus.fraunhofer.de \
    --cc=siwu@hrz.tu-chemnitz.de \
    /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