All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kalle Valo <kalle.valo@iki.fi>
To: "Luis R. Rodriguez" <mcgrof@gmail.com>
Cc: Johannes Berg <johannes@sipsolutions.net>,
	Dan Williams <dcbw@redhat.com>, "Guy\,
	Wey-Yi W" <wey-yi.w.guy@intel.com>,
	linux-wireless <linux-wireless@vger.kernel.org>,
	Matthew Garrett <mjg@redhat.com>,
	Marcel Holtmann <marcel@holtmann.org>, Jouni Malinen <j@w1.fi>
Subject: Re: wireless powersaving (in NM?)
Date: Thu, 26 Mar 2009 10:19:48 +0200	[thread overview]
Message-ID: <87iqlw4rx7.fsf@litku.valot.fi> (raw)
In-Reply-To: <43e72e890903260100w75896d0fi9121f227cc7ca8dc@mail.gmail.com> (Luis R. Rodriguez's message of "Thu\, 26 Mar 2009 01\:00\:31 -0700")

"Luis R. Rodriguez" <mcgrof@gmail.com> writes:

> On Wed, Mar 25, 2009 at 8:48 AM, Johannes Berg
> <johannes@sipsolutions.net> wrote:
>> On Wed, 2009-03-25 at 11:31 -0400, Dan Williams wrote:
>>
>>> Yeah, sounds OK. =A0Just to be clear, problems with this code will =
appear
>>> on a *per-AP* basis, not a per-card basis, right? =A0ie there are s=
ome
>>> stupid APs that won't work with it, but it won't be the case that a
>>> chipset will be broken for all APs, right?
>>
>> Right. If a particular chipset doesn't properly supported PS then we
>> shouldn't actually support it at all -- but we cannot know if the AP=
 is
>> broken.
>
> How about leaving it to the supplicant to guess if the AP is borked o=
r not.

Very difficult and not worth the trouble IMHO.

> Kalle, what would happen with these broken APs?

Usually there's packet loss and for the user it shows as data
connections not working reliably.

> If we can determine this through some heuristics then we can label
> such BSSes as borked in the supplicant and then the user won't have t=
o
> worry about this. This would only be dealt with each BSS once at most
> as we would have saved the AP's power save borkedness after our first
> determination of this.
>
> If one wants to do manual debugging you can simple add the flag
> through the supplicant for the bss. Just don't see the need to confus=
e
> a user with an option if we can get away with figuring it out if
> possible.

If the ui option to disable power is disliked, just hide it somewhere.
=46or example, a gconf flag which can be enabled with a command line
tool.

--=20
Kalle Valo
--
To unsubscribe from this list: send the line "unsubscribe linux-wireles=
s" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2009-03-26  8:19 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-24 10:39 wireless powersaving (in NM?) Johannes Berg
2009-03-24 10:32 ` Marcel Holtmann
2009-03-25 10:45   ` Johannes Berg
2009-03-24 13:47 ` John W. Linville
2009-03-24 13:53 ` Johannes Berg
2009-03-24 14:47   ` Guy, Wey-Yi W
2009-03-24 15:03     ` Kalle Valo
2009-03-24 14:17 ` Kalle Valo
2009-03-25 10:57   ` Johannes Berg
2009-03-25 19:53     ` Kalle Valo
2009-03-25 20:06       ` Johannes Berg
2009-03-25 20:09         ` Johannes Berg
2009-03-25 20:22         ` Kalle Valo
2009-03-24 15:59 ` Dan Williams
2009-03-25 11:04   ` Johannes Berg
2009-03-25 15:31     ` Dan Williams
2009-03-25 15:48       ` Johannes Berg
2009-03-26  8:00         ` Luis R. Rodriguez
2009-03-26  8:11           ` Johannes Berg
2009-03-26  8:39             ` Luis R. Rodriguez
2009-03-26  8:54               ` Johannes Berg
2009-03-26  9:01                 ` Luis R. Rodriguez
2009-03-26 14:29                   ` Georgy Berdyshev
2009-03-26  8:19           ` Kalle Valo [this message]
2009-03-26  8:40             ` Luis R. Rodriguez
2009-03-31 16:28 ` Johannes Berg
2009-04-01 11:04 ` more thoughts on power saving (was: wireless powersaving (in NM?)) Johannes Berg
2009-04-01 11:12   ` Kalle Valo
2009-04-01 14:29     ` Guy, Wey-Yi W

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=87iqlw4rx7.fsf@litku.valot.fi \
    --to=kalle.valo@iki.fi \
    --cc=dcbw@redhat.com \
    --cc=j@w1.fi \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    --cc=marcel@holtmann.org \
    --cc=mcgrof@gmail.com \
    --cc=mjg@redhat.com \
    --cc=wey-yi.w.guy@intel.com \
    /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.