MIME-Version: 1.0
In-Reply-To: <c36f442934605c492525af72fa04ada9@codeaurora.org>
References: <8760trzoiw.fsf@kamboji.qca.qualcomm.com> <871t4fzn1x.fsf@kamboji.qca.qualcomm.com>
 <57504F05.3040200@candelatech.com> <1464887026467.72937@qti.qualcomm.com>
 <57506BA1.2090303@candelatech.com> <CAHbf0-GT0y1pEs-ToxbPAf+aRo7TNAyV_Emies_rjL27R1fk2A@mail.gmail.com>
 <c36f442934605c492525af72fa04ada9@codeaurora.org>
From: Mike Lothian <mike@fireburn.co.uk>
Date: Wed, 8 Jun 2016 18:41:30 +0100
Message-ID: <CAHbf0-EYKTo+ZyzVM5v8Ae1werwh6+enj_SEdZG6KbXB1TygXw@mail.gmail.com> (sfid-20160608_194155_435798_A90C27E4)
Subject: Re: Bug 119151 - [regression] ath10k no longer authenitcates and
 freezes system
To: Rajkumar Manoharan <rmanohar@codeaurora.org>
Cc: Ben Greear <greearb@candelatech.com>,
	"Manoharan, Rajkumar" <rmanohar@qti.qualcomm.com>,
	"Valo, Kalle" <kvalo@qca.qualcomm.com>, ath10k@lists.infradead.org,
	linux-wireless@vger.kernel.org
Content-Type: text/plain; charset=UTF-8
Sender: linux-wireless-owner@vger.kernel.org

Hi

Yes that fixes things locally

Thanks

Mike

On 8 June 2016 at 16:52, Rajkumar Manoharan <rmanohar@codeaurora.org> wrote:
> On 2016-06-02 23:03, Mike Lothian wrote:
>>
>> I've just tried those two changes, the machine now locks up before X
>> has even started
>>
> Mike,
>
> Sorry for the delay. Found root cause for dead lock. Can you please give a
> try with below change?
>
> diff --git a/drivers/net/wireless/ath/ath10k/htt_rx.c
> b/drivers/net/wireless/ath/ath10k/htt_rx.c
> index 3b35c7ab5680..80e645302b54 100644
> --- a/drivers/net/wireless/ath/ath10k/htt_rx.c
> +++ b/drivers/net/wireless/ath/ath10k/htt_rx.c
> @@ -1905,7 +1905,6 @@ static void ath10k_htt_rx_in_ord_ind(struct ath10k
> *ar, struct sk_buff *skb)
>                         return;
>                 }
>         }
> -       ath10k_htt_rx_msdu_buff_replenish(htt);
>  }
>
>
> -Rajkumar