2008-01-13 16:28:19

by Ron Rindjunsky

[permalink] [raw]

Subject: [PATCH 1/2] mac80211: A-MPDU Rx stop aggregation on proper dev

---

This patch adds a check to insure that Rx A-MPDU will be stopped only
for the proper device.

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

diff --git a/net/mac80211/ieee80211.c b/net/mac80211/ieee80211.c
index 1770402..5dcc2d6 100644
--- a/net/mac80211/ieee80211.c
+++ b/net/mac80211/ieee80211.c
@@ -299,9 +299,11 @@ static int ieee80211_stop(struct net_device *dev)
sdata = IEEE80211_DEV_TO_SUB_IF(dev);

list_for_each_entry(sta, &local->sta_list, list) {
- for (i = 0; i < STA_TID_NUM; i++)
- ieee80211_sta_stop_rx_ba_session(sta->dev, sta->addr,
- i, WLAN_BACK_RECIPIENT,
+ if (sta->dev == dev)
+ for (i = 0; i < STA_TID_NUM; i++)
+ ieee80211_sta_stop_rx_ba_session(sta->dev,
+ sta->addr, i,
+ WLAN_BACK_RECIPIENT,
WLAN_REASON_QSTA_LEAVE_QBSS);
}

--
1.5.3.7

2008-01-13 21:43:23

by Johannes Berg

[permalink] [raw]

Subject: Re: [PATCH 1/2] mac80211: A-MPDU Rx stop aggregation on proper dev

---

On Sun, 2008-01-13 at 18:21 +0200, Ron Rindjunsky wrote:
> This patch adds a check to insure that Rx A-MPDU will be stopped only
> for the proper device.

Thanks.

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

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

> ---
> net/mac80211/ieee80211.c | 8 +++++---
> 1 files changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/net/mac80211/ieee80211.c b/net/mac80211/ieee80211.c
> index 1770402..5dcc2d6 100644
> --- a/net/mac80211/ieee80211.c
> +++ b/net/mac80211/ieee80211.c
> @@ -299,9 +299,11 @@ static int ieee80211_stop(struct net_device *dev)
> sdata = IEEE80211_DEV_TO_SUB_IF(dev);
>
> list_for_each_entry(sta, &local->sta_list, list) {
> - for (i = 0; i < STA_TID_NUM; i++)
> - ieee80211_sta_stop_rx_ba_session(sta->dev, sta->addr,
> - i, WLAN_BACK_RECIPIENT,
> + if (sta->dev == dev)
> + for (i = 0; i < STA_TID_NUM; i++)
> + ieee80211_sta_stop_rx_ba_session(sta->dev,
> + sta->addr, i,
> + WLAN_BACK_RECIPIENT,
> WLAN_REASON_QSTA_LEAVE_QBSS);
> }
>

---

Attachments:

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