diff mbox

[OpenWrt-Devel] Hostapd: wait longer for inactive client probe (empty data frame)

Message ID 20151006151350.f13a3e8597cfce6994de52a1@ubnt.com
State Accepted
Headers show

Commit Message

Dmitry Ivanov Oct. 6, 2015, 12:13 p.m. UTC
Hostapd: wait longer for inactive client probe (empty data frame).

One second is not enough for some devices to ackowledge null data frame
which is sent at the end of ap_max_inactivity interval. In particular,
this causes severe Wi-Fi instability with Apple iPhone which may take
up to 3 seconds to respond.

Signed-off-by: Dmitry Ivanov <dima@ubnt.com>
---
 .../hostapd/patches/470-wait-for-nullfunc-longer.patch        | 11 +++++++++++
 1 file changed, 11 insertions(+)
 create mode 100644 package/network/services/hostapd/patches/470-wait-for-nullfunc-longer.patch

Comments

Felix Fietkau Oct. 6, 2015, 12:35 p.m. UTC | #1
On 2015-10-06 14:13, Dmitry Ivanov wrote:
> Hostapd: wait longer for inactive client probe (empty data frame).
> 
> One second is not enough for some devices to ackowledge null data frame
> which is sent at the end of ap_max_inactivity interval. In particular,
> this causes severe Wi-Fi instability with Apple iPhone which may take
> up to 3 seconds to respond.
> 
> Signed-off-by: Dmitry Ivanov <dima@ubnt.com>
Applied, thanks. Please also submit this upstream!

Here's the same reminder as the last few times, I made a bit more flashy
this time, hoping that you'll read it ;)

*************** FRIENDLY REMINDER, PLEASE READ! ***************
* Please do NOT repeat the patch subject in the message body! *
***************************************************************

- Felix
diff mbox

Patch

diff --git a/package/network/services/hostapd/patches/470-wait-for-nullfunc-longer.patch b/package/network/services/hostapd/patches/470-wait-for-nullfunc-longer.patch
new file mode 100644
index 0000000..e6bbddd
--- /dev/null
+++ b/package/network/services/hostapd/patches/470-wait-for-nullfunc-longer.patch
@@ -0,0 +1,11 @@ 
+--- a/src/ap/sta_info.h
++++ b/src/ap/sta_info.h
+@@ -179,7 +179,7 @@ struct sta_info {
+  * AP_DISASSOC_DELAY seconds. Similarly, the station will be deauthenticated
+  * after AP_DEAUTH_DELAY seconds has passed after disassociation. */
+ #define AP_MAX_INACTIVITY (5 * 60)
+-#define AP_DISASSOC_DELAY (1)
++#define AP_DISASSOC_DELAY (3)
+ #define AP_DEAUTH_DELAY (1)
+ /* Number of seconds to keep STA entry with Authenticated flag after it has
+  * been disassociated. */