<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN">

<html><body>

Hi Felix,<br /><br />Sorry for the late reply.<br /><br />I've seen the check is already implemented here in hostapd since 2011: <a href="http://w1.fi/cgit/hostap/commit/?id=70619a5d8a3d32faa43d66bcb1b670cacf0c243e">http://w1.fi/cgit/hostap/commit/?id=70619a5d8a3d32faa43d66bcb1b670cacf0c243e<br /><br /></a>So maybe we should just set uapsd as default enabled, and let hostapd check the actual support.<br /><br />Do you think this would be fine?<br /><br />Cheers,<br />Vittorio<br />

<div> </div>

<br />

<p>Il 18.10.2014 21:37 Felix Fietkau ha scritto:</p>

<blockquote type="cite" style="padding-left:5px; border-left:#1010ff 2px solid; margin-left:5px; width:100%"><!-- html ignored --><!-- head ignored --><!-- meta ignored -->

<pre>On 2014-10-14 20:05, Vittorio G (VittGam) wrote:</pre>

<blockquote type="cite" style="padding-left:5px; border-left:#1010ff 2px solid; margin-left:5px; width:100%">The uapsd option sets the uapsd_advertisement_enabled flag in hostapd, only if the phy actually supports the option. Signed-off-by: Vittorio Gambaletta <<a href="mailto:openwrt@vittgam.net">openwrt@vittgam.net</a>> --- package/network/services/hostapd/files/netifd.sh | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/package/network/services/hostapd/files/netifd.sh b/package/network/services/hostapd/files/netifd.sh index ccea5ff..48bcc9d 100644 --- a/package/network/services/hostapd/files/netifd.sh +++ b/package/network/services/hostapd/files/netifd.sh</blockquote>

<blockquote type="cite" style="padding-left:5px; border-left:#1010ff 2px solid; margin-left:5px; width:100%">@@ -189,6 +190,10 @@ hostapd_set_bss_options() { append bss_conf "wmm_enabled=$wmm" "$N" append bss_conf "ignore_broadcast_ssid=$hidden" "$N" + [ -n "$phy" ] && iw phy "$phy" info 2>/dev/null | grep -q 'Device supports AP-side u-APSD.' && { + append bss_conf "uapsd_advertisement_enabled=$uapsd" "$N" + }</blockquote>

<pre>hostapd's netifd.sh should not hardcode cfg80211/mac80211 specific

calls, it may be called from other drivers in the future.

- Felix

</pre>

</blockquote>

</body></html>