linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Johannes Berg <johannes@sipsolutions.net>
To: Adrian Chadd <adrian@freebsd.org>
Cc: "Luis R. Rodriguez" <mcgrof@frijolero.org>,
	linux-wireless@vger.kernel.org
Subject: Re: [RFC 03/15] mac80211: also expire filtered frames
Date: Tue, 27 Sep 2011 09:50:07 +0200	[thread overview]
Message-ID: <1317109807.4082.4.camel@jlt3.sipsolutions.net> (raw)
In-Reply-To: <CAJ-Vmo=9MUaKph=EXwF2VnE+=nHg4s=j=qhR2jnwa-xsXMno7g@mail.gmail.com> (sfid-20110927_042854_028200_8F3D1D54)

On Tue, 2011-09-27 at 10:26 +0800, Adrian Chadd wrote:
> On 27 September 2011 06:30, Luis R. Rodriguez <mcgrof@frijolero.org> wrote:
> 
> > No particular comments on the code yet but latency issues has got me
> > thinking about the filtered frames stuff and if we really need it. How
> > much benefit does keeping these frames give us instead of just
> > dropping them?
> 
> Do you guys keep statistics about this sort of thing?

Not sure.

> My macbook pro ends up causing my FreeBSD AP to miss TX'ing a lot of
> frames, even if it's close by. This only occurs when it's on battery
> power.
> 
> I have a feeling it's going to be doing aggressiveish power saving
> stuff and this'll be fixed by me porting over and tidying up the
> filtered frames support.

I'm surprised you don't have filtered frames support! It's pretty
important unless you keep hardware queues almost empty since otherwise
exactly what you describe will happen. The hardware is supposed to keep
track of when a station goes to sleep (only WAKE->DOZE transition, must
rely on driver/stack for DOZE->WAKE transition to avoid packet
reordering) and reject ("filter") frames on the queue in that case.

johannes


  reply	other threads:[~2011-09-27  7:50 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-22 15:47 [RFC 00/15] mac80211 uAPSD support Johannes Berg
2011-09-22 15:47 ` [RFC 01/15] mac80211: let drivers inform it about per TID buffered frames Johannes Berg
2011-09-22 15:47 ` [RFC 02/15] mac80211: unify TIM bit handling Johannes Berg
2011-09-22 15:47 ` [RFC 03/15] mac80211: also expire filtered frames Johannes Berg
2011-09-26 22:30   ` Luis R. Rodriguez
2011-09-27  2:26     ` Adrian Chadd
2011-09-27  7:50       ` Johannes Berg [this message]
2011-09-27 12:24         ` Adrian Chadd
2011-09-27 12:25           ` Adrian Chadd
2011-09-27  7:47     ` Johannes Berg
2011-09-22 15:47 ` [RFC 04/15] mac80211: split PS buffers into ACs Johannes Berg
2011-09-27 20:51   ` Luis R. Rodriguez
2011-09-28  7:10     ` Johannes Berg
2011-09-22 15:47 ` [RFC 05/15] mac80211: remove return value from add_pending_skbs Johannes Berg
2011-09-22 15:47 ` [RFC 06/15] mac80211: clear more-data bit on filtered frames Johannes Berg
2011-09-22 15:47 ` [RFC 07/15] mac80211: allow releasing driver-buffered frames Johannes Berg
2011-09-22 15:47 ` [RFC 08/15] mac80211: implement uAPSD Johannes Berg
2011-09-22 15:47 ` [RFC 09/15] mac80211: send (QoS) Null if no buffered frames Johannes Berg
2011-09-22 15:47 ` [RFC 10/15] mac80211: reply only once to each PS-poll Johannes Berg
2011-09-22 15:47 ` [RFC 11/15] mac80211: optimise station flags Johannes Berg
2011-09-22 15:47 ` [RFC 12/15] mac80211: add missing station flags to debugfs Johannes Berg
2011-09-22 15:47 ` [RFC 13/15] mac80211: explicitly notify drivers of frame release Johannes Berg
2011-09-22 15:47 ` [RFC 14/15] mac80211: allow out-of-band EOSP notification Johannes Berg
2011-09-22 15:47 ` [RFC 15/15] mac80211: document client powersave Johannes Berg
2011-09-22 15:52 ` [RFC 00/15] mac80211 uAPSD support Johannes Berg
2011-09-22 23:25   ` Luis R. Rodriguez
2011-09-23  8:59     ` Johannes Berg
2011-09-23 18:06 ` Dmitry Tarnyagin
2011-09-23 18:14   ` Johannes Berg

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=1317109807.4082.4.camel@jlt3.sipsolutions.net \
    --to=johannes@sipsolutions.net \
    --cc=adrian@freebsd.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=mcgrof@frijolero.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;
as well as URLs for NNTP newsgroup(s).