Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932845Ab1CXKK4 (ORCPT ); Thu, 24 Mar 2011 06:10:56 -0400 Received: from purkki.adurom.net ([80.68.90.206]:36215 "EHLO purkki.adurom.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752680Ab1CXKKq convert rfc822-to-8bit (ORCPT ); Thu, 24 Mar 2011 06:10:46 -0400 X-Greylist: delayed 2270 seconds by postgrey-1.27 at vger.kernel.org; Thu, 24 Mar 2011 06:10:46 EDT To: Johannes Berg Cc: Arve =?utf-8?B?SGrDuG5uZXbDpWc=?= , "Luis R. Rodriguez" , linux-kernel@vger.kernel.org, "Rafael J. Wysocki" , devel@linuxdriverproject.org, Naveen Singh , Benjamin Herrenschmidt , Jouni Malinen , Vipin Mehta , linux-wireless Subject: Re: Update on Android PM enhancements - CONFIG_HAS_EARLYSUSPEND and friends References: <1300957495.3807.7.camel@jlt3.sipsolutions.net> <1300957590.3807.8.camel@jlt3.sipsolutions.net> From: Kalle Valo Date: Thu, 24 Mar 2011 11:32:40 +0200 In-Reply-To: <1300957590.3807.8.camel@jlt3.sipsolutions.net> (Johannes Berg's message of "Thu\, 24 Mar 2011 10\:06\:30 +0100") Message-ID: <87k4fouamf.fsf@purkki.adurom.net> User-Agent: Gnus/5.11 (Gnus v5.11) Emacs/22.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1971 Lines: 49 Johannes Berg writes: > On Thu, 2011-03-24 at 10:04 +0100, Johannes Berg wrote: >> On Wed, 2011-03-23 at 17:47 -0700, Arve Hjønnevåg wrote: >> >> > There is no planned replacement for early suspend, and it will >> > probably be dropped form the android kernel at some point. The wifi >> > driver use it to enter a lower operating state when the screen is off. >> > If you driver does have any other way to put the driver in this state >> > you need to add an interface that lets user-space select that state >> > directly. >> >> The interface is called "ip link set wlan0 down". > > Or, in fact, just powersaving mode. Exactly. > But I don't think for a wifi driver there's any need for special > driver specific states. In maemo, to save power, we decreased the dynamic power save timeout whenever display was turned off. This was all done from user space and wifi driver in the kernel was completely unaware of the display state. But I think this is still a hack and we should save as much as possible power in all cases, despite the display state. Instead power save parameters should automatically change based on the transfer characteristics. Few examples what I mean: o if idle or very little traffic -> more aggressive power save o if latency critical packets (dns requests, higher qos classes etc) -> decrease latency o if active data transfer -> increase throughput And before someone suggests adding dynamic timeout variable to nl80211: I think it's very much implementation specific parameter. Not all drivers support it and it might be that even in future mac80211 uses something else. So having the timeout variable in nl80211 is not a long term solution. -- Kalle Valo -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/