Search Linux Wireless

Re: [RFC] mac80211: disable aggregatin for voice traffic

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 18 Nov 2010 17:18:54 -0800, "Luis R. Rodriguez"
<lrodriguez@xxxxxxxxxxx> wrote:
> This should help with latency issues which can happen when
> using aggregation.
> 
> Cc: Matt Smith <matt.smith@xxxxxxxxxxx>
> Cc: Senthil Balasubramanian <senthilkumar@xxxxxxxxxxx>
> Signed-off-by: Luis R. Rodriguez <lrodriguez@xxxxxxxxxxx>
> ---
> 
> We discussed this a while ago but never to around to it.
> 
>  net/mac80211/tx.c |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/net/mac80211/tx.c b/net/mac80211/tx.c
> index e694836..405ddc9 100644
> --- a/net/mac80211/tx.c
> +++ b/net/mac80211/tx.c
> @@ -1225,7 +1225,8 @@ ieee80211_tx_prepare(struct ieee80211_sub_if_data
> *sdata,
>  		tx->sta = sta_info_get(sdata, hdr->addr1);
>  
>  	if (tx->sta && ieee80211_is_data_qos(hdr->frame_control) &&
> -	    (local->hw.flags & IEEE80211_HW_AMPDU_AGGREGATION)) {
> +	    (local->hw.flags & IEEE80211_HW_AMPDU_AGGREGATION) &&
> +	    (ieee802_1d_to_ac[skb->priority] != IEEE80211_AC_VO)) {

wtf? total NACK. You're setting up an aggregation session and then not
using it? That's totally deluded ... just don't set up the session!!

johannes
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux