mbox series

[0/2] wifi: ath11k: fix data out of sync for channel list for reg update

Message ID 20230329091235.19500-1-quic_wgong@quicinc.com (mailing list archive)
Headers show
Series wifi: ath11k: fix data out of sync for channel list for reg update | expand

Message

Wen Gong March 29, 2023, 9:12 a.m. UTC
Currently there are two threads to updating/fetch data of channel 
list, and there are no synchronization for the data, it leads data out 
of sync for channel list for reg update.

So change the call flow to make sure the fetch data running after the 
update data finished, then data of channel list become synchronization.

Fixes: f45cb6b29cd3 ("wifi: ath11k: avoid deadlock during regulatory update in ath11k_regd_update()")

Wen Gong (2):
  wifi: ath11k: move update channel list from update reg worker to reg
    notifier
  wifi: ath11k: move udpate channel list to worker for wait flag

 drivers/net/wireless/ath/ath11k/core.c |   1 +
 drivers/net/wireless/ath/ath11k/core.h |   4 +
 drivers/net/wireless/ath/ath11k/mac.c  |  13 ++++
 drivers/net/wireless/ath/ath11k/reg.c  | 100 ++++++++++++++++---------
 drivers/net/wireless/ath/ath11k/reg.h  |   1 +
 drivers/net/wireless/ath/ath11k/wmi.h  |   1 +
 6 files changed, 86 insertions(+), 34 deletions(-)


base-commit: 5aa9ff08c0b71be15a607341fa5344546e19ceb2

Comments

Wen Gong July 10, 2023, 3:03 a.m. UTC | #1
On 3/29/2023 5:12 PM, Wen Gong wrote:
> Currently there are two threads to updating/fetch data of channel
> list, and there are no synchronization for the data, it leads data out
> of sync for channel list for reg update.
>
> So change the call flow to make sure the fetch data running after the
> update data finished, then data of channel list become synchronization.
>
> Fixes: f45cb6b29cd3 ("wifi: ath11k: avoid deadlock during regulatory update in ath11k_regd_update()")
>
> Wen Gong (2):
>    wifi: ath11k: move update channel list from update reg worker to reg
>      notifier
>    wifi: ath11k: move udpate channel list to worker for wait flag
>
>   drivers/net/wireless/ath/ath11k/core.c |   1 +
>   drivers/net/wireless/ath/ath11k/core.h |   4 +
>   drivers/net/wireless/ath/ath11k/mac.c  |  13 ++++
>   drivers/net/wireless/ath/ath11k/reg.c  | 100 ++++++++++++++++---------
>   drivers/net/wireless/ath/ath11k/reg.h  |   1 +
>   drivers/net/wireless/ath/ath11k/wmi.h  |   1 +
>   6 files changed, 86 insertions(+), 34 deletions(-)
>
>
> base-commit: 5aa9ff08c0b71be15a607341fa5344546e19ceb2

Hi Kalle,

Will you plan to merge it or will you give comments?
Wen Gong Aug. 1, 2023, 5:32 a.m. UTC | #2
Hi Kalle,

Will you plan to merge it or will you give comments since it has 
deferred/pending here for 4 month ? 
Wen Gong Aug. 22, 2023, 6:51 a.m. UTC | #3
Hi Kalle,

May I change the subject to "wifi: ath11k: fix firmware crash and wrong 
reg domain used in firmware and hardware" and sent new version again?

Or will you merge this version?

On 8/1/2023 1:32 PM, Wen Gong wrote:
> Hi Kalle,
>
> Will you plan to merge it or will you give comments since it has 
> deferred/pending here for 4 month ?