Instead of the atomic restart count, can the state be updated to ATH10K_STATE_RESTARTING while holding mutex_unlock(&ar->conf_mutex); I don't understand the bundles, but I wonder about the case when there are multiple packets (n_rx_pkts) and if pkt_bundle_len might be the one to check. Also if there needs to be a check that the len > sizeof HTC HDR. On Tue, Dec 31, 2019 at 1:37 AM <wgong@xxxxxxxxxxxxxx> wrote: > > On 2019-12-25 23:14, Justin Capella wrote: > > This does not only effect SDIO. > > > > Why a semaphore / count? Could the conf_mutex be held earlier, or > > perhaps change the state to ATH10K_STATE_RESTARTING first? > > ath10k_reconfig_complete is also called in mac.c when channel is > > changed so > patch v2: > https://patchwork.kernel.org/patch/11313853/ > https://patchwork.kernel.org/patch/11313859/