From: Arik Nemtsov <arik@wizery.com> To: Marek Puzyniak <marek.puzyniak@tieto.com> Cc: Johannes Berg <johannes@sipsolutions.net>, Arik Nemtsov <arikx.nemtsov@intel.com>, "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>, "ath10k@lists.infradead.org" <ath10k@lists.infradead.org> Subject: Re: [PATCH v2 3/4] mac80211: initialize rate control earlier for tdls station Date: Wed, 4 Mar 2015 18:30:50 +0200 [thread overview] Message-ID: <CA+XVXfeG0TZ+jdRKhcZHzUjxffFbmzJBPtW2nrQ0+JjKC_+mbQ@mail.gmail.com> (raw) In-Reply-To: <CADnKcVjWiHi1RsYwiE6buMduq19i3A5qt8w93VLz6dRg-qLOLQ@mail.gmail.com> [-- Attachment #1: Type: text/plain, Size: 638 bytes --] On Wed, Mar 4, 2015 at 12:38 PM, Marek Puzyniak <marek.puzyniak@tieto.com> wrote: > On 4 March 2015 at 11:04, Arik Nemtsov <arik@wizery.com> wrote: >> I can do it, but I'll probably only get to it next week. > > I will try to prepare patch this week, if not I will wait for Arik's > proposition. > > For ath10k rate control need to be initialised before moving to > STA_AUTHORIZED, so initialising rate control before STA_ASSOC is > perfectly fine. Ok I did get to it today :) With the attached patch applied, and the patch "mac80211: update TDLS sta spatial streams before auth" reverted, I'm getting good performance on iwlwifi. Arik [-- Attachment #2: 0001-mac80211-initialize-rate-control-earlier-for-tdls-st.patch --] [-- Type: text/x-patch, Size: 1915 bytes --] From 133a9b280c23fa0d49e93dae8655b9514ac71fee Mon Sep 17 00:00:00 2001 From: Marek Puzyniak <marek.puzyniak@tieto.com> Date: Wed, 25 Feb 2015 08:55:10 +0100 Subject: [PATCH] mac80211: initialize rate control earlier for tdls station Currently when TDLS station in driver goes from assoc to authorized state it can not use rate control parameters because rate control is not initialized yet. Some drivers require parameters already initialized by rate control when entering authorized state. It can be done by initializing rate control after station transition to authorized state but before notifying driver about that. Signed-off-by: Marek Puzyniak <marek.puzyniak@tieto.com> Signed-off-by: Arik Nemtsov <arikx.nemtsov@intel.com> --- net/mac80211/cfg.c | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/net/mac80211/cfg.c b/net/mac80211/cfg.c index c13b1af..55b88c7 100644 --- a/net/mac80211/cfg.c +++ b/net/mac80211/cfg.c @@ -977,6 +977,14 @@ static int sta_apply_auth_flags(struct ieee80211_local *local, if (mask & BIT(NL80211_STA_FLAG_ASSOCIATED) && set & BIT(NL80211_STA_FLAG_ASSOCIATED) && !test_sta_flag(sta, WLAN_STA_ASSOC)) { + /* + * When peer becomes authorized, init rate control as + * well. Some drivers require rate control initialized + * before drv_sta_state() is called. + */ + if (test_sta_flag(sta, WLAN_STA_TDLS_PEER)) + rate_control_rate_init(sta); + ret = sta_info_move_state(sta, IEEE80211_STA_ASSOC); if (ret) return ret; @@ -1381,11 +1389,6 @@ static int ieee80211_change_station(struct wiphy *wiphy, if (err) goto out_err; - /* When peer becomes authorized, init rate control as well */ - if (test_sta_flag(sta, WLAN_STA_TDLS_PEER) && - test_sta_flag(sta, WLAN_STA_AUTHORIZED)) - rate_control_rate_init(sta); - mutex_unlock(&local->sta_mtx); if ((sdata->vif.type == NL80211_IFTYPE_AP || -- 2.1.0
WARNING: multiple messages have this Message-ID (diff)
From: Arik Nemtsov <arik@wizery.com> To: Marek Puzyniak <marek.puzyniak@tieto.com> Cc: Johannes Berg <johannes@sipsolutions.net>, "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>, "ath10k@lists.infradead.org" <ath10k@lists.infradead.org>, Arik Nemtsov <arikx.nemtsov@intel.com> Subject: Re: [PATCH v2 3/4] mac80211: initialize rate control earlier for tdls station Date: Wed, 4 Mar 2015 18:30:50 +0200 [thread overview] Message-ID: <CA+XVXfeG0TZ+jdRKhcZHzUjxffFbmzJBPtW2nrQ0+JjKC_+mbQ@mail.gmail.com> (raw) In-Reply-To: <CADnKcVjWiHi1RsYwiE6buMduq19i3A5qt8w93VLz6dRg-qLOLQ@mail.gmail.com> [-- Attachment #1: Type: text/plain, Size: 638 bytes --] On Wed, Mar 4, 2015 at 12:38 PM, Marek Puzyniak <marek.puzyniak@tieto.com> wrote: > On 4 March 2015 at 11:04, Arik Nemtsov <arik@wizery.com> wrote: >> I can do it, but I'll probably only get to it next week. > > I will try to prepare patch this week, if not I will wait for Arik's > proposition. > > For ath10k rate control need to be initialised before moving to > STA_AUTHORIZED, so initialising rate control before STA_ASSOC is > perfectly fine. Ok I did get to it today :) With the attached patch applied, and the patch "mac80211: update TDLS sta spatial streams before auth" reverted, I'm getting good performance on iwlwifi. Arik [-- Attachment #2: 0001-mac80211-initialize-rate-control-earlier-for-tdls-st.patch --] [-- Type: text/x-patch, Size: 1915 bytes --] From 133a9b280c23fa0d49e93dae8655b9514ac71fee Mon Sep 17 00:00:00 2001 From: Marek Puzyniak <marek.puzyniak@tieto.com> Date: Wed, 25 Feb 2015 08:55:10 +0100 Subject: [PATCH] mac80211: initialize rate control earlier for tdls station Currently when TDLS station in driver goes from assoc to authorized state it can not use rate control parameters because rate control is not initialized yet. Some drivers require parameters already initialized by rate control when entering authorized state. It can be done by initializing rate control after station transition to authorized state but before notifying driver about that. Signed-off-by: Marek Puzyniak <marek.puzyniak@tieto.com> Signed-off-by: Arik Nemtsov <arikx.nemtsov@intel.com> --- net/mac80211/cfg.c | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/net/mac80211/cfg.c b/net/mac80211/cfg.c index c13b1af..55b88c7 100644 --- a/net/mac80211/cfg.c +++ b/net/mac80211/cfg.c @@ -977,6 +977,14 @@ static int sta_apply_auth_flags(struct ieee80211_local *local, if (mask & BIT(NL80211_STA_FLAG_ASSOCIATED) && set & BIT(NL80211_STA_FLAG_ASSOCIATED) && !test_sta_flag(sta, WLAN_STA_ASSOC)) { + /* + * When peer becomes authorized, init rate control as + * well. Some drivers require rate control initialized + * before drv_sta_state() is called. + */ + if (test_sta_flag(sta, WLAN_STA_TDLS_PEER)) + rate_control_rate_init(sta); + ret = sta_info_move_state(sta, IEEE80211_STA_ASSOC); if (ret) return ret; @@ -1381,11 +1389,6 @@ static int ieee80211_change_station(struct wiphy *wiphy, if (err) goto out_err; - /* When peer becomes authorized, init rate control as well */ - if (test_sta_flag(sta, WLAN_STA_TDLS_PEER) && - test_sta_flag(sta, WLAN_STA_AUTHORIZED)) - rate_control_rate_init(sta); - mutex_unlock(&local->sta_mtx); if ((sdata->vif.type == NL80211_IFTYPE_AP || -- 2.1.0 [-- Attachment #3: Type: text/plain, Size: 146 bytes --] _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k
next prev parent reply other threads:[~2015-03-04 16:31 UTC|newest] Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-02-25 7:55 [PATCH v2 0/4] ath10k: add basic tdls support Marek Puzyniak 2015-02-25 7:55 ` Marek Puzyniak 2015-02-25 7:55 ` [PATCH v2 1/4] ath10k: unify tx mode and dispatch Marek Puzyniak 2015-02-25 7:55 ` Marek Puzyniak 2015-02-25 7:55 ` [PATCH v2 2/4] ath10k: make peer type configurable Marek Puzyniak 2015-02-25 7:55 ` Marek Puzyniak 2015-02-25 7:55 ` [PATCH v2 3/4] mac80211: initialize rate control earlier for tdls station Marek Puzyniak 2015-02-25 7:55 ` Marek Puzyniak 2015-02-27 12:50 ` Johannes Berg 2015-02-27 12:50 ` Johannes Berg 2015-03-01 8:21 ` Arik Nemtsov 2015-03-01 8:21 ` Arik Nemtsov 2015-03-03 9:18 ` Johannes Berg 2015-03-03 9:18 ` Johannes Berg 2015-03-03 10:02 ` Arik Nemtsov 2015-03-03 10:02 ` Arik Nemtsov 2015-03-03 10:06 ` Johannes Berg 2015-03-03 10:06 ` Johannes Berg 2015-03-03 10:07 ` Arik Nemtsov 2015-03-03 10:07 ` Arik Nemtsov 2015-03-04 8:16 ` Johannes Berg 2015-03-04 8:16 ` Johannes Berg 2015-03-04 10:04 ` Arik Nemtsov 2015-03-04 10:04 ` Arik Nemtsov 2015-03-04 10:38 ` Marek Puzyniak 2015-03-04 10:38 ` Marek Puzyniak 2015-03-04 16:30 ` Arik Nemtsov [this message] 2015-03-04 16:30 ` Arik Nemtsov 2015-03-04 18:30 ` Johannes Berg 2015-03-04 18:30 ` Johannes Berg 2015-03-04 15:01 ` Kalle Valo 2015-03-04 15:01 ` Kalle Valo 2015-02-25 7:55 ` [PATCH v2 4/4] ath10k: introduce basic tdls functionality Marek Puzyniak 2015-02-25 7:55 ` Marek Puzyniak 2015-02-25 10:34 ` Michal Kazior 2015-02-25 10:34 ` Michal Kazior
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=CA+XVXfeG0TZ+jdRKhcZHzUjxffFbmzJBPtW2nrQ0+JjKC_+mbQ@mail.gmail.com \ --to=arik@wizery.com \ --cc=arikx.nemtsov@intel.com \ --cc=ath10k@lists.infradead.org \ --cc=johannes@sipsolutions.net \ --cc=linux-wireless@vger.kernel.org \ --cc=marek.puzyniak@tieto.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.