2012-06-18 07:19:37

by Arik Nemtsov

[permalink] [raw]
Subject: [PATCH] mac80211: set only VO as a U-APSD enabled AC

Some APs experience problems when working with U-APSD. Decrease the
probability of that happening by using legacy mode for all ACs but VO.

The AP that caused us troubles was a Cisco 4410N. It ignores our
setting, and always treats non-VO ACs as legacy.

Signed-off-by: Arik Nemtsov <[email protected]>
---
net/mac80211/ieee80211_i.h | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/net/mac80211/ieee80211_i.h b/net/mac80211/ieee80211_i.h
index ddf768f..58436c3 100644
--- a/net/mac80211/ieee80211_i.h
+++ b/net/mac80211/ieee80211_i.h
@@ -56,10 +56,7 @@ struct ieee80211_local;
#define TU_TO_EXP_TIME(x) (jiffies + TU_TO_JIFFIES(x))

#define IEEE80211_DEFAULT_UAPSD_QUEUES \
- (IEEE80211_WMM_IE_STA_QOSINFO_AC_BK | \
- IEEE80211_WMM_IE_STA_QOSINFO_AC_BE | \
- IEEE80211_WMM_IE_STA_QOSINFO_AC_VI | \
- IEEE80211_WMM_IE_STA_QOSINFO_AC_VO)
+ IEEE80211_WMM_IE_STA_QOSINFO_AC_VO

#define IEEE80211_DEFAULT_MAX_SP_LEN \
IEEE80211_WMM_IE_STA_QOSINFO_SP_ALL
--
1.7.9.5



2012-06-18 07:23:10

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH] mac80211: set only VO as a U-APSD enabled AC

Arik Nemtsov <[email protected]> writes:

> Some APs experience problems when working with U-APSD. Decrease the
> probability of that happening by using legacy mode for all ACs but VO.
>
> The AP that caused us troubles was a Cisco 4410N. It ignores our
> setting, and always treats non-VO ACs as legacy.
>
> Signed-off-by: Arik Nemtsov <[email protected]>

[...]

> --- a/net/mac80211/ieee80211_i.h
> +++ b/net/mac80211/ieee80211_i.h
> @@ -56,10 +56,7 @@ struct ieee80211_local;
> #define TU_TO_EXP_TIME(x) (jiffies + TU_TO_JIFFIES(x))
>
> #define IEEE80211_DEFAULT_UAPSD_QUEUES \
> - (IEEE80211_WMM_IE_STA_QOSINFO_AC_BK | \
> - IEEE80211_WMM_IE_STA_QOSINFO_AC_BE | \
> - IEEE80211_WMM_IE_STA_QOSINFO_AC_VI | \
> - IEEE80211_WMM_IE_STA_QOSINFO_AC_VO)
> + IEEE80211_WMM_IE_STA_QOSINFO_AC_VO

Am I in your kill file now? :)

http://www.spinics.net/lists/linux-wireless/msg92159.html

--
Kalle Valo

2012-06-18 07:25:53

by Arik Nemtsov

[permalink] [raw]
Subject: Re: [PATCH] mac80211: set only VO as a U-APSD enabled AC

On Mon, Jun 18, 2012 at 10:23 AM, Kalle Valo <[email protected]> wrote:
> Arik Nemtsov <[email protected]> writes:
>
>> Some APs experience problems when working with U-APSD. Decrease the
>> probability of that happening by using legacy mode for all ACs but VO.
>>
>> The AP that caused us troubles was a Cisco 4410N. It ignores our
>> setting, and always treats non-VO ACs as legacy.
>>
>> Signed-off-by: Arik Nemtsov <[email protected]>
>
> [...]
>
>> --- a/net/mac80211/ieee80211_i.h
>> +++ b/net/mac80211/ieee80211_i.h
>> @@ -56,10 +56,7 @@ struct ieee80211_local;
>> ?#define TU_TO_EXP_TIME(x) ? ?(jiffies + TU_TO_JIFFIES(x))
>>
>> ?#define IEEE80211_DEFAULT_UAPSD_QUEUES \
>> - ? ? (IEEE80211_WMM_IE_STA_QOSINFO_AC_BK | ? \
>> - ? ? ?IEEE80211_WMM_IE_STA_QOSINFO_AC_BE | ? \
>> - ? ? ?IEEE80211_WMM_IE_STA_QOSINFO_AC_VI | ? \
>> - ? ? ?IEEE80211_WMM_IE_STA_QOSINFO_AC_VO)
>> + ? ? IEEE80211_WMM_IE_STA_QOSINFO_AC_VO
>
> Am I in your kill file now? :)
>
> http://www.spinics.net/lists/linux-wireless/msg92159.html

Oh missed your comment. I'll resend.