All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michal Kazior <michal.kazior@tieto.com>
To: Krishna Chaitanya <chaitanya.mgit@gmail.com>
Cc: Johannes Berg <johannes@sipsolutions.net>,
	linux-wireless <linux-wireless@vger.kernel.org>
Subject: Re: [PATCH] mac80211: disable u-APSD queues by default
Date: Tue, 3 Mar 2015 11:28:50 +0100	[thread overview]
Message-ID: <CA+BoTQnTvqYB6VS=bzk8bcV1xHPAZtyq0oJa0C2oe0Gd0wj+NQ@mail.gmail.com> (raw)
In-Reply-To: <CABPxzYLbF-b5DKBBNo2PK5iDn64HuSo9N63mfT2fNfvgLhEFrQ@mail.gmail.com>

On 3 March 2015 at 11:15, Krishna Chaitanya <chaitanya.mgit@gmail.com> wrote:
> On Tue, Mar 3, 2015 at 3:06 PM, Johannes Berg <johannes@sipsolutions.net> wrote:
>>
>>> >> Cisco 4410N originally forced us to enable VO by
>>> >> default only because it treated non-VO ACs as
>>> >> legacy.
>>
>>> Mixed mode (uapsd + legacy) is not really the optimal
>>> configuration, where we expect STA to send trigger frames periodically
>>> to retrieve the DL packets (especially very first packet).
>>>
>>> But still if we want power save, i suggest going for
>>> All u-apsd (0xF), so that AP will inform us by setting the TIM bit
>>> and we can retrieve the first frame.
>>> (We have this patch internally and it works well).
>>
>> Except where it doesn't work, like on the Cisco 4410N - see above.
> But for that single AP, we are forced to lose efficient power savings
> unless user configured explicitly.

Correct me if I'm wrong, but u-APSD makes sense only if you have
userspace apps aware of it. I don't know any. VoIP traffic uses and it
works just because it generate bi-directional traffic in most cases so
you have implicit trigger frames generated when you Tx, but that's
probably as far as the practical use-case goes, no? You'd still need
netfilter rules to reclassify your traffic anyway, wouldn't you? If
you mainly wait for data you need to send trigger frames periodically
and only userspace app knows what Rx latency is tolerated (that's why
relying on driver/firmware auto-trigger generation doesn't cut it).

Anyway I don't think you can ask users to do the opposite (i.e.
disable u-APSD if they experience problems with APs). Wi-Fi should
Just Work. Joys of IOT..


Michał

  reply	other threads:[~2015-03-03 10:28 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-10 11:48 [PATCH] mac80211: disable u-APSD queues by default Michal Kazior
2015-03-03  9:16 ` Johannes Berg
2015-03-03  9:33   ` Krishna Chaitanya
2015-03-03  9:36     ` Johannes Berg
2015-03-03 10:15       ` Krishna Chaitanya
2015-03-03 10:28         ` Michal Kazior [this message]
2015-03-03 11:20           ` Krishna Chaitanya

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='CA+BoTQnTvqYB6VS=bzk8bcV1xHPAZtyq0oJa0C2oe0Gd0wj+NQ@mail.gmail.com' \
    --to=michal.kazior@tieto.com \
    --cc=chaitanya.mgit@gmail.com \
    --cc=johannes@sipsolutions.net \
    --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 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.