mac80211: use listen interval 5 as default
authorHelmut Schaa <helmut.schaa@googlemail.com>
Wed, 24 Feb 2010 13:19:37 +0000 (14:19 +0100)
committerJohn W. Linville <linville@tuxdriver.com>
Fri, 26 Feb 2010 21:59:11 +0000 (16:59 -0500)
Currently if a driver does not set hw.max_listen_interval a listen
interval of 1 is negotiated with the AP. Thus, the AP could drop
buffered frames for us after just one beacon interval which can
easily happen with the current powersave and scan implementation.
To avoid this issue increase the default interval to 5 which should
be a reasonable safe default.

Signed-off-by: Helmut Schaa <helmut.schaa@googlemail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
net/mac80211/main.c

index ec8f767ba95b4979c334754866318f7767c15b08..06c33b68d8e545fdc78625b121594dbcfaf10a03 100644 (file)
@@ -558,8 +558,12 @@ int ieee80211_register_hw(struct ieee80211_hw *hw)
 
        debugfs_hw_add(local);
 
+       /*
+        * if the driver doesn't specify a max listen interval we
+        * use 5 which should be a safe default
+        */
        if (local->hw.max_listen_interval == 0)
-               local->hw.max_listen_interval = 1;
+               local->hw.max_listen_interval = 5;
 
        local->hw.conf.listen_interval = local->hw.max_listen_interval;