Return-path: Received: from wolverine02.qualcomm.com ([199.106.114.251]:39077 "EHLO wolverine02.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933085AbcLMNow (ORCPT ); Tue, 13 Dec 2016 08:44:52 -0500 From: "Valo, Kalle" To: Erik Stromdahl CC: "linux-wireless@vger.kernel.org" , "ath10k@lists.infradead.org" Subject: Re: [RFC v2 05/11] ath10k: htc: refactorization Date: Tue, 13 Dec 2016 13:44:44 +0000 Message-ID: <87inqoymd0.fsf@kamboji.qca.qualcomm.com> (sfid-20161213_144456_000465_7BEFAEFF) References: <1479496971-19174-1-git-send-email-erik.stromdahl@gmail.com> <1479496971-19174-6-git-send-email-erik.stromdahl@gmail.com> In-Reply-To: <1479496971-19174-6-git-send-email-erik.stromdahl@gmail.com> (Erik Stromdahl's message of "Fri, 18 Nov 2016 20:22:45 +0100") Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: Erik Stromdahl writes: > Code refactorization: > > Moved the code for ep 0 in ath10k_htc_rx_completion_handler > to ath10k_htc_control_rx_complete. > > This eases the implementation of SDIO/mbox significantly since > the ep_rx_complete cb is invoked directly from the SDIO/mbox > hif layer. > > Since the ath10k_htc_control_rx_complete already is present > (only containing a warning message) there is no reason for not > using it (instead of having a special case for ep 0 in > ath10k_htc_rx_completion_handler). > > Signed-off-by: Erik Stromdahl I tested this on QCA988X PCI board just to see if there are any regressions. It crashes immediately during module load, every time, and bisected that the crashing starts on this patch: [ 1239.715325] ath10k_pci 0000:02:00.0: pci irq msi oper_irq_mode 2 irq_mod= e 0 reset_mode 0 [ 1239.885125] ath10k_pci 0000:02:00.0: Direct firmware load for ath10k/pre= -cal-pci-0000:02:00.0.bin failed with error -2 [ 1239.885260] ath10k_pci 0000:02:00.0: Direct firmware load for ath10k/cal= -pci-0000:02:00.0.bin failed with error -2 [ 1239.885687] ath10k_pci 0000:02:00.0: qca988x hw2.0 target 0x4100016c chi= p_id 0x043202ff sub 0000:0000 [ 1239.885699] ath10k_pci 0000:02:00.0: kconfig debug 1 debugfs 1 tracing 1= dfs 1 testmode 1 [ 1239.885899] ath10k_pci 0000:02:00.0: firmware ver 10.2.4.70.59-2 api 5 f= eatures no-p2p,raw-mode,mfp,allows-mesh-bcast crc32 4159f498 [ 1239.941836] ath10k_pci 0000:02:00.0: Direct firmware load for ath10k/QCA= 988X/hw2.0/board-2.bin failed with error -2 [ 1239.941993] ath10k_pci 0000:02:00.0: board_file api 1 bmi_id N/A crc32 b= ebc7c08 [ 1241.136693] BUG: unable to handle kernel NULL pointer dereference at (= null) [ 1241.136738] IP: [< (null)>] (null) [ 1241.136759] *pdpt =3D 0000000000000000 *pde =3D f0002a55f0002a55 [ 1241.= 136781]=20 [ 1241.136793] Oops: 0010 [#1] SMP What's odd is that when I added some printks on my own and enabled both boot and htc debug levels it doesn't crash anymore. After everything works normally after that, I can start AP mode and connect to it. Is it a race somewhere? --=20 Kalle Valo=