2008-01-13 16:29:39

by Ron Rindjunsky

[permalink] [raw]
Subject: [PATCH 2/2] mac80211: A-MPDU Rx remove stop_rx_ba_session warning print

This patch removes a warning print from ieee80211_sta_stop_rx_ba_session
in case the tid is inactive when interface goes down.

Signed-off-by: Ron Rindjunsky <[email protected]>
---
net/mac80211/ieee80211_sta.c | 3 ---
1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/net/mac80211/ieee80211_sta.c b/net/mac80211/ieee80211_sta.c
index 866eb80..bb73aac 100644
--- a/net/mac80211/ieee80211_sta.c
+++ b/net/mac80211/ieee80211_sta.c
@@ -1215,9 +1215,6 @@ void ieee80211_sta_stop_rx_ba_session(struct net_device *dev, u8 *ra, u16 tid,
if (sta->ampdu_mlme.tid_rx[tid].state
!= HT_AGG_STATE_OPERATIONAL) {
spin_unlock_bh(&sta->ampdu_mlme.ampdu_rx);
- if (net_ratelimit())
- printk(KERN_DEBUG "rx BA session requested to stop on "
- "inactive tid %d\n", tid);
sta_info_put(sta);
return;
}
--
1.5.3.7



2008-01-13 21:44:08

by Johannes Berg

[permalink] [raw]
Subject: Re: [PATCH 2/2] mac80211: A-MPDU Rx remove stop_rx_ba_session warning print


On Sun, 2008-01-13 at 18:25 +0200, Ron Rindjunsky wrote:
> This patch removes a warning print from ieee80211_sta_stop_rx_ba_session
> in case the tid is inactive when interface goes down.

Thanks.

> Signed-off-by: Ron Rindjunsky <[email protected]>

Acked-by: Johannes Berg <[email protected]>

> ---
> net/mac80211/ieee80211_sta.c | 3 ---
> 1 files changed, 0 insertions(+), 3 deletions(-)
>
> diff --git a/net/mac80211/ieee80211_sta.c b/net/mac80211/ieee80211_sta.c
> index 866eb80..bb73aac 100644
> --- a/net/mac80211/ieee80211_sta.c
> +++ b/net/mac80211/ieee80211_sta.c
> @@ -1215,9 +1215,6 @@ void ieee80211_sta_stop_rx_ba_session(struct net_device *dev, u8 *ra, u16 tid,
> if (sta->ampdu_mlme.tid_rx[tid].state
> != HT_AGG_STATE_OPERATIONAL) {
> spin_unlock_bh(&sta->ampdu_mlme.ampdu_rx);
> - if (net_ratelimit())
> - printk(KERN_DEBUG "rx BA session requested to stop on "
> - "inactive tid %d\n", tid);
> sta_info_put(sta);
> return;
> }

Attachments:
signature.asc (828.00 B)
This is a digitally signed message part