[PATCH] cfg80211: lower dynamic PS timeout to 100ms

[PATCH] cfg80211: lower dynamic PS timeout to 100ms

[Johannes Berg]

The default of 500ms is pretty high, and leads
to the device being awake at least 50% of the
time under such light traffic conditions as a
simple 1 second interval ping. Reduce to just
100ms -- it should have a similar effect while
providing a better sleep time.

Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
---
 net/wireless/core.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- wireless-testing.orig/net/wireless/core.c	2009-07-31 11:15:32.000000000 +0200
+++ wireless-testing/net/wireless/core.c	2009-07-31 11:15:40.000000000 +0200
@@ -673,7 +673,7 @@ static int cfg80211_netdev_notifier_call
 		wdev->wext.default_mgmt_key = -1;
 		wdev->wext.connect.auth_type = NL80211_AUTHTYPE_AUTOMATIC;
 		wdev->wext.ps = CONFIG_CFG80211_DEFAULT_PS_VALUE;
-		wdev->wext.ps_timeout = 500;
+		wdev->wext.ps_timeout = 100;
 		if (rdev->ops->set_power_mgmt)
 			if (rdev->ops->set_power_mgmt(wdev->wiphy, dev,
 						      wdev->wext.ps,

Re: [PATCH] cfg80211: lower dynamic PS timeout to 100ms

[Kalle Valo]

Johannes Berg <johannes@sipsolutions.net> writes:

> The default of 500ms is pretty high, and leads
> to the device being awake at least 50% of the
> time under such light traffic conditions as a
> simple 1 second interval ping. Reduce to just
> 100ms -- it should have a similar effect while
> providing a better sleep time.
>
> Signed-off-by: Johannes Berg <johannes@sipsolutions.net>

I agree, 500 ms is too long.

Reviewed-by: Kalle Valo <kalle.valo@iki.fi>

-- 
Kalle Valo