Search Linux Wireless

Re: [PATCH 1/2] cfg80211: add SM PS hooks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2009-12-01 at 09:01 +0200, Kalle Valo wrote:
> Johannes Berg <johannes@xxxxxxxxxxxxxxxx> writes:
> 
> > Add spatial multiplexing power save configuration
> > hooks to cfg80211/nl80211.
> 
> Is it really necessary to export this to user space, at least via
> nl80211? If we add this to nl80211, we have to support this almost
> forever.
> 
> Also how are planning to use this? What components will use the
> interface and how?

Good questions.

> My concern here is that we will end up having, yet again, complicated
> user space interface for power save. The ideal situation would be that
> kernel would configure all this automatically and we would have a
> simple interface just to disable power save in cases where it doesn't
> work (broken APs etc).

I'm not actually sure that there are APs broken wrt. SM PS, it seems
highly unlikely -- even if there are (like mac80211!!) the rate control
algorithm would recover the situation.

As such, I suppose it's somewhat a debugging API. On the other hand,
there are actual traffic throughput consequences of enabling SM PS, even
when in powersave. Powersave will be turned off automatically if there's
enough traffic, but SM PS won't. The default, automatic, means it just
follows powersave (dynamic SM PS if on, no SM PS if off), but I'm not
sure that is typically desirable.

Of course, on the other hand, I don't see many users actually setting
it. An alternative could be to try to estimate traffic, but that gets
messy fairly quickly because the thresholds depend on the environment.

If you prefer to leave it out for now, I can probably move it to debugfs
until we have a more clearly defined use case, although if we decide
that we never want it then obviously the action frame handling can go
away again.

johannes

Attachment: signature.asc
Description: This is a digitally signed message part


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux