linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Question about ieee802_1d_to_ac
@ 2012-10-16  8:04 David Shwatrz
  2012-10-16 13:00 ` Arend van Spriel
  0 siblings, 1 reply; 2+ messages in thread
From: David Shwatrz @ 2012-10-16  8:04 UTC (permalink / raw)
  To: linux-wireless

Hello,

I see in net/mac80211/wme.c:
...
const int ieee802_1d_to_ac[8] = {
	IEEE80211_AC_BE,
	IEEE80211_AC_BK,
	IEEE80211_AC_BK,
	IEEE80211_AC_BE,
	IEEE80211_AC_VI,
	IEEE80211_AC_VI,
	IEEE80211_AC_VO,
	IEEE80211_AC_VO
};
...

As I understand, this values stand for queues for different types of traffic:
IEEE80211_AC_BE is  best-effort traffic ,
IEEE80211_AC_BK is background traffic ,
IEEE80211_AC_VI is video traffic,
IEEE80211_AC_VO is voice traffic.

We see that entries 6,7 are  voice and entries 4,5 are video.
Is it not an error that entries 2,3 differ ? and also 0,1 ?

shouldn't it be:
const int ieee802_1d_to_ac[8] = {
	IEEE80211_AC_BE,
	IEEE80211_AC_BE,
	IEEE80211_AC_BK,
	IEEE80211_AC_BK,


Regards,
D Shwartz

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Question about ieee802_1d_to_ac
  2012-10-16  8:04 Question about ieee802_1d_to_ac David Shwatrz
@ 2012-10-16 13:00 ` Arend van Spriel
  0 siblings, 0 replies; 2+ messages in thread
From: Arend van Spriel @ 2012-10-16 13:00 UTC (permalink / raw)
  To: David Shwatrz; +Cc: linux-wireless

On 10/16/2012 10:04 AM, David Shwatrz wrote:
> Hello,
>
> I see in net/mac80211/wme.c:
> ...
> const int ieee802_1d_to_ac[8] = {
> 	IEEE80211_AC_BE,
> 	IEEE80211_AC_BK,
> 	IEEE80211_AC_BK,
> 	IEEE80211_AC_BE,
> 	IEEE80211_AC_VI,
> 	IEEE80211_AC_VI,
> 	IEEE80211_AC_VO,
> 	IEEE80211_AC_VO
> };
> ...

This is the correct mapping if you want to trust the IEEE802.11e 
standard (Table 20i).

Gr. AvS

> As I understand, this values stand for queues for different types of traffic:
> IEEE80211_AC_BE is  best-effort traffic ,
> IEEE80211_AC_BK is background traffic ,
> IEEE80211_AC_VI is video traffic,
> IEEE80211_AC_VO is voice traffic.
>
> We see that entries 6,7 are  voice and entries 4,5 are video.
> Is it not an error that entries 2,3 differ ? and also 0,1 ?
>
> shouldn't it be:
> const int ieee802_1d_to_ac[8] = {
> 	IEEE80211_AC_BE,
> 	IEEE80211_AC_BE,
> 	IEEE80211_AC_BK,
> 	IEEE80211_AC_BK,
>
>
> Regards,
> D Shwartz
> --
> To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-10-16 13:00 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-10-16  8:04 Question about ieee802_1d_to_ac David Shwatrz
2012-10-16 13:00 ` Arend van Spriel

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).