Re: [ath9k-devel] [PATCH] mac80211: Use RCU protection in ieee80211_get_tx_rates()

2013-06-13 Thread Calvin Owens
On Wednesday 06/12 at 11:47 +0200, Felix Fietkau wrote: On 2013-06-12 10:00 AM, Calvin Owens wrote: Copying the rate table should be done in an RCU read-side critical section. I think this approach is wrong. The sta entry is also under RCU protection (no locking for read access in that part

Re: [ath9k-devel] [PATCH 5/6] ath10k: wait for CE to drain when shutting down

2013-06-13 Thread Kalle Valo
Michal Kazior michal.kaz...@tieto.com writes: ath10k_pci_process_ce() is used to process completions. Only one thread can do that though. If one thread starts handling completions then the other one (i.e. possibly PCI shutdown) would exit immediatelely and free up memory while completions

Re: [ath9k-devel] [PATCH 0/6] ath10k: fixes

2013-06-13 Thread Kalle Valo
Kalle Valo kv...@qca.qualcomm.com writes: Michal Kazior michal.kaz...@tieto.com writes: This includes 2 patches from my previous patchset that had conflicts. It is now rebased and should apply cleanly on the github master branch. Michal Kazior (6): ath10k: embed HTC struct inside ath10k

Re: [ath9k-devel] [PATCH v2 01/10] ath10k: decouple pci init/deinit logic

2013-06-13 Thread Kalle Valo
Michal Kazior michal.kaz...@tieto.com writes: Split logic that prepares the device for BMI phase/cleans up related resources. This is necessary for ath10k to be able to restart hw on the fly without reloading the module. Signed-off-by: Michal Kazior michal.kaz...@tieto.com Few comments: