Search Linux Wireless

Re: [PATCH] mac80211: uAPSD - fix IEEE80211_FCTL_MOREDATA bit setting

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

 



On Mon, 2011-11-07 at 09:47 +0200, Janusz.Dziedzic@xxxxxxxxx wrote:
> Set IEEE80211_FCTL_MOREDATA bit also in case we have buffered
> frames (more than one) only for one AC.
> 
> Signed-off-by: Janusz Dziedzic <janusz.dziedzic@xxxxxxxxx>

Reviewed-by: Johannes Berg <johannes@xxxxxxxxxxxxxxxx>

Thanks!

> ---
>  net/mac80211/sta_info.c |    8 ++++----
>  1 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/net/mac80211/sta_info.c b/net/mac80211/sta_info.c
> index ce962d2..8eaa746 100644
> --- a/net/mac80211/sta_info.c
> +++ b/net/mac80211/sta_info.c
> @@ -1354,12 +1354,12 @@ ieee80211_sta_ps_deliver_response(struct sta_info *sta,
>  			 * Use MoreData flag to indicate whether there are
>  			 * more buffered frames for this STA
>  			 */
> -			if (!more_data)
> -				hdr->frame_control &=
> -					cpu_to_le16(~IEEE80211_FCTL_MOREDATA);
> -			else
> +			if (more_data || !skb_queue_empty(&frames))
>  				hdr->frame_control |=
>  					cpu_to_le16(IEEE80211_FCTL_MOREDATA);
> +			else
> +				hdr->frame_control &=
> +					cpu_to_le16(~IEEE80211_FCTL_MOREDATA);
>  
>  			if (ieee80211_is_data_qos(hdr->frame_control) ||
>  			    ieee80211_is_qos_nullfunc(hdr->frame_control))


--
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