linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Johannes Berg <johannes@sipsolutions.net>
To: Krishna Chaitanya <chaitanya.mgit@gmail.com>,
	linux-wireless <linux-wireless@vger.kernel.org>
Subject: Re: RFC: Handling DL only traffic when already in Powersave
Date: Thu, 07 Apr 2016 22:07:46 +0200	[thread overview]
Message-ID: <1460059666.27154.16.camel@sipsolutions.net> (raw)
In-Reply-To: <CABPxzYLaWKgYrwjyjDpiZzWCbCpCzr7mKLorm9mzxspE5n0rHQ@mail.gmail.com> (sfid-20160407_150155_571489_8906D5AA)

On Thu, 2016-04-07 at 18:31 +0530, Krishna Chaitanya wrote:
> Hi,
> 
> When using HW_PS and running Downlink only traffic like UDP-RX,
> mac80211 has the mechanism to not to enter power-save (postpone the
> dynamic ps timer).
> 
> But if we are already in power-save it continues to stay in PS
> and retrieves the frames, for large traffic this is not acceptable.
> 
> The standard doesn't say anything, its left to vendor to implement
> their own mechanism.
> 
> I have the below questions
> 
> 1) Are any other vendors seeing this issue?

No. Nobody else really relies on the mac80211 powersave any more.

> 2) Are they using proprietary algorithms to solve this,
> implemented in their HW/FW?

Yes. "Proprietary" is such a big word.

> 3) If yes, how are they keeping the mac80211 and FW FSM in sync,
> as the driver cannot request/indicate the power save state to
> mac80211.
> (no feedback path).

No need, don't use the mac80211 implementation! Tell mac80211 you
implement PS and DYNAMIC_PS, and it will not do anything whatsoever.

> IMHO,
> 
> a) we should implemented a feedback path, where the vendor
> can implement proprietary mechanism to handle this and inform/request
> mac80211 for a specific power save state.

No.

> b) mac80211 just conveys the user/system preferences to the driver
> and driver handles the FSM on when to enter/exit PS? (true HW_PS).
> 
Done when you have PS and DYNAMIC_PS advertised to mac80211.

The mac80211 powersave code is pretty much beyond salvaging anyway.

What really should happen is that there should be some kind of library
functionality that drivers that need host powersave management can hook
into, *per interface*, and then do the right thing.

Then eventually we can remove all the powersave handling code from
mac80211, it's seriously broken anyway (doesn't support more than one
virtual interface, has problems like you describe above, etc.)

johannes

  reply	other threads:[~2016-04-07 20:07 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-07 13:01 RFC: Handling DL only traffic when already in Powersave Krishna Chaitanya
2016-04-07 20:07 ` Johannes Berg [this message]
2016-04-08 10:05   ` 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=1460059666.27154.16.camel@sipsolutions.net \
    --to=johannes@sipsolutions.net \
    --cc=chaitanya.mgit@gmail.com \
    --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;
as well as URLs for NNTP newsgroup(s).