On Tue, 2009-03-24 at 11:32 +0100, Marcel Holtmann wrote: > I prefer that we not export this over WEXT at all and only via nl80211 > so we get async notification if some one changes it. We should extend > wpa_supplicant with an option to set power saving on/off/default and > then make sure it gets exported via its CLI or D-Bus. So if Network > Manger or ConnMan cares to change these values they can. > > However personally I think the need for enabling or disabling power > savings is not really useful except for testing purposes. In a real life > deployed system it should be enabled all the time. Broken drivers or > hardware that is limited can disable it on kernel level, but for > userspace we only need this for debugging. Kalle makes a point about broken APs, which I personally haven't run into but I'm sure exist -- and I see no way to automatically detect that. Therefore, it seems necessary to allow overriding this even for non-debugging purposes. I have no problem with just changing the default to be enabled though, but that requires some work to turn off automatically for most virtual interface combinations. As I said before -- this needs to be fairly conservative and only when applications register their requirements can we go to lower power states with higher latency. johannes
Attachment:
signature.asc
Description: This is a digitally signed message part
