Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <2579553c-2e7f-787c-ad6b-9d21f8a08b39@quicinc.com>
Date:   Mon, 10 Jul 2023 11:03:39 +0800
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.12.0
Subject: Re: [PATCH 0/2] wifi: ath11k: fix data out of sync for channel list
 for reg update
Content-Language: en-US
To:     <ath11k@lists.infradead.org>
CC:     <linux-wireless@vger.kernel.org>
References: <20230329091235.19500-1-quic_wgong@quicinc.com>
From:   Wen Gong <quic_wgong@quicinc.com>
In-Reply-To: <20230329091235.19500-1-quic_wgong@quicinc.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

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?