Search Linux Wireless

Re: [RFC 1/2] mac80211: reset beacon monitor at tx ack notify

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

 



On Sat, 2011-12-17 at 20:35 +0530, Rajkumar Manoharan wrote:
> This patch moves the beacon loss timer to future on receiving tx
> ack status from the associated AP. So that we can avoid unnecessary
> AP probing.
> 
> Signed-off-by: Rajkumar Manoharan <rmanohar@xxxxxxxxxxxxxxxx>
> ---
>  net/mac80211/mlme.c |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
> 
> diff --git a/net/mac80211/mlme.c b/net/mac80211/mlme.c
> index a984f1f..65f1262 100644
> --- a/net/mac80211/mlme.c
> +++ b/net/mac80211/mlme.c
> @@ -1209,8 +1209,10 @@ void ieee80211_sta_tx_notify(struct ieee80211_sub_if_data *sdata,
>  	if (!ieee80211_is_data(hdr->frame_control))
>  	    return;
>  
> -	if (ack)
> +	if (ack) {
>  		ieee80211_sta_reset_conn_monitor(sdata);
> +		ieee80211_sta_reset_beacon_monitor(sdata);
> +	}

This doesn't make a lot of sense. You can argue that beacon monitoring
isn't worthwhile, but side-stepping it completely when there's traffic
seems not what's intended as it is?

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