Re: [PATCH] mac80211: ath9k: Use RCU protection calling ieee80211_get_tx_rates

2013-06-12 Thread Calvin Owens
On Tuesday 06/11 at 21:55 +0200, Johannes Berg wrote: > Now the subject should no longer say "mac80211:" :) > > However... given that ieee80211_get_tx_rates() actually *copies* the > rates into the parameter, I guess it should do the rcu_read_lock() > internally. I guess I wasn't paying attention

Re: [PATCH] mac80211: ath9k: Use RCU protection calling ieee80211_get_tx_rates

2013-06-12 Thread Kalle Valo
Johannes Berg writes: > Now the subject should no longer say "mac80211:" :) It did have a change to mac80211.h as well, but IMHO that should be in a separate patch. -- Kalle Valo -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

Re: [PATCH] mac80211: ath9k: Use RCU protection calling ieee80211_get_tx_rates

2013-06-12 Thread Kalle Valo
Johannes Berg johan...@sipsolutions.net writes: Now the subject should no longer say mac80211: :) It did have a change to mac80211.h as well, but IMHO that should be in a separate patch. -- Kalle Valo -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a

Re: [PATCH] mac80211: ath9k: Use RCU protection calling ieee80211_get_tx_rates

2013-06-12 Thread Calvin Owens
On Tuesday 06/11 at 21:55 +0200, Johannes Berg wrote: Now the subject should no longer say mac80211: :) However... given that ieee80211_get_tx_rates() actually *copies* the rates into the parameter, I guess it should do the rcu_read_lock() internally. I guess I wasn't paying attention

Re: [PATCH] mac80211: ath9k: Use RCU protection calling ieee80211_get_tx_rates

2013-06-11 Thread Johannes Berg
Now the subject should no longer say "mac80211:" :) However... given that ieee80211_get_tx_rates() actually *copies* the rates into the parameter, I guess it should do the rcu_read_lock() internally. I guess I wasn't paying attention previously. Felix? johannes -- To unsubscribe from this

[PATCH] mac80211: ath9k: Use RCU protection calling ieee80211_get_tx_rates

2013-06-11 Thread Calvin Owens
Signed-off-by: Calvin Owens --- drivers/net/wireless/ath/ath9k/xmit.c | 2 ++ drivers/net/wireless/mac80211_hwsim.c | 6 ++ include/net/mac80211.h| 3 ++- 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/drivers/net/wireless/ath/ath9k/xmit.c

[PATCH] mac80211: ath9k: Use RCU protection calling ieee80211_get_tx_rates

2013-06-11 Thread Calvin Owens
Signed-off-by: Calvin Owens jcalvinow...@gmail.com --- drivers/net/wireless/ath/ath9k/xmit.c | 2 ++ drivers/net/wireless/mac80211_hwsim.c | 6 ++ include/net/mac80211.h| 3 ++- 3 files changed, 10 insertions(+), 1 deletion(-) diff --git

Re: [PATCH] mac80211: ath9k: Use RCU protection calling ieee80211_get_tx_rates

2013-06-11 Thread Johannes Berg
Now the subject should no longer say mac80211: :) However... given that ieee80211_get_tx_rates() actually *copies* the rates into the parameter, I guess it should do the rcu_read_lock() internally. I guess I wasn't paying attention previously. Felix? johannes -- To unsubscribe from this list: