All of lore.kernel.org
 help / color / mirror / Atom feed
From: Johannes Stezenbach <js@sig21.net>
To: Aleksandar Milivojevic <alex@milivojevic.org>
Cc: Kalle Valo <kvalo@adurom.com>,
	linux-wireless@vger.kernel.org, Ivo Van Doorn <ivdoorn@gmail.com>
Subject: Re: [rt2x00-users] [RFC] rt2x00: Add autowakeup timer for receiving beacons while in powersave mode
Date: Wed, 9 Feb 2011 15:31:15 +0100	[thread overview]
Message-ID: <20110209143115.GA6356@sig21.net> (raw)
In-Reply-To: <AANLkTikgASqJ_7omn-Z4JWgAJ1dMtOYoSO8oa=GDtB=a@mail.gmail.com>

On Tue, Feb 08, 2011 at 11:11:26AM -0800, Aleksandar Milivojevic wrote:
> On Tue, Feb 8, 2011 at 12:48 AM, Kalle Valo <kvalo@adurom.com> wrote:
> >
> > There is CONFIG_CFG80211_DEFAULT_PS kconfig option, but I'm not aware
> > of any other way to globally do this. I can only think of using iw or
> > iwconfig to disable power save per device.
> 
> I'm using iwconfig currently to disable power saving for wireless card
> on my desktop after it boots up.  However, from user perspective of
> things, it would be awesome if default was not hard coded in the
> kernel.

E.g. on Debian you can add iwconfig settings to /etc/network/interfaces,
see http://linux.die.net/man/7/wireless but I've no idea if that
works with NetworkManager or wicd etc.

I am not sure a global config setting for PS would not be an improvement
because you usually want to enable it for the hw which supports it
(unless you have specific low latency requirements).

But I think on some distributions acpid scripts run iwconfig to change
PS depending on if you're on AC or battery power.

FWIW, I had sent a patch to change the rt2800 USB driver
to default to PS off but Ivo seemed determined to
fix power saving instead of disabling it.
http://rt2x00.serialmonkey.com/pipermail/users_rt2x00.serialmonkey.com/2011-January/003017.html
(This RFC patch only changes the default, if it turns out PS
really cannot be fixed we should remove IEEE80211_HW_SUPPORTS_PS instead.)


Johannes

  parent reply	other threads:[~2011-02-09 14:31 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-31 15:00 [RFC] rt2x00: Add autowakeup timer for receiving beacons while in powersave mode Ivo van Doorn
2011-01-31 15:22 ` Johannes Berg
2011-01-31 15:38   ` Ivo Van Doorn
2011-01-31 15:54     ` [rt2x00-users] " Johannes Stezenbach
2011-01-31 18:17 ` Johannes Stezenbach
2011-01-31 19:00   ` Ivo Van Doorn
2011-01-31 21:05     ` Johannes Stezenbach
2011-02-02 17:42 ` Kalle Valo
2011-02-02 19:09   ` Ivo Van Doorn
2011-02-10  5:24     ` RA-Jay Hung
2011-02-14 10:14       ` Ivo Van Doorn
2011-02-14 10:25         ` Johannes Berg
2011-02-02 21:29   ` Johannes Stezenbach
2011-02-07 23:33     ` [rt2x00-users] " Aleksandar Milivojevic
2011-02-08  8:48       ` Kalle Valo
2011-02-08 19:11         ` Aleksandar Milivojevic
2011-02-09 12:17           ` Kalle Valo
2011-02-09 14:31           ` Johannes Stezenbach [this message]
2011-02-08  8:27     ` Kalle Valo
2011-02-08 20:09       ` Johannes Stezenbach

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=20110209143115.GA6356@sig21.net \
    --to=js@sig21.net \
    --cc=alex@milivojevic.org \
    --cc=ivdoorn@gmail.com \
    --cc=kvalo@adurom.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 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.