Hi, On Mon, Jul 20, 2020 at 6:32 PM Peter Oh <peter.oh@xxxxxxxx> wrote: > > I'll take my word back. > It's not this patch problem, but by others. > I have 2 extra patches before the 3 patches so my system looks like > > backports from ath.git 5.6-rc1 + linux kernel 4.4 (similar to OpenWrt) > On top of the working system, I cherry-picked these 5. > > #1. > ath10k: Avoid override CE5 configuration for QCA99X0 chipsets > ath.git commit 521fc37be3d879561ca5ab42d64719cf94116af0 > #2. > ath10k: Fix NULL pointer dereference in AHB device probe > wireless-drivers.git commit 1cfd3426ef989b83fa6176490a38777057e57f6c > #3. > ath10k: Add interrupt summary based CE processing > https://patchwork.kernel.org/patch/11628299/ > #4. > ath10k: Keep track of which interrupts fired, don't poll them > https://patchwork.kernel.org/patch/11654631/ > #5. > ath10k: Get rid of "per_ce_irq" hw param > https://patchwork.kernel.org/patch/11654633/ > > The error "[ 14.226184] ath10k_ahb a000000.wifi: failed to receive > initialized event from target: 80000000" is because of #1 and #2, > since this happens even after I reverted #3~#5. > Once I reverted all except #1 I got another crash. > > [ 11.179595] !#%&PageFault P<__ath10k_ce_rx_post_buf+0x14/0x98 > [ath10k_core]> L<0x4bc00> F<005> [0000000c] > [ 11.179643] Unable to handle kernel NULL pointer dereference at > virtual address 0000000c > [ 11.439207] [<7f15a69c>] (__ath10k_ce_rx_post_buf [ath10k_core]) > from [<7f15a874>] (ath10k_ce_rx_post_buf+0x3c/0x50 [ath10k_core]) > [ 11.447204] [<7f15a874>] (ath10k_ce_rx_post_buf [ath10k_core]) from > [<7f2889a4>] (ath10k_pci_diag_read_mem+0x104/0x2a8 [ath10k_pci]) > [ 11.458706] [<7f2889a4>] (ath10k_pci_diag_read_mem [ath10k_pci]) > from [<7f288b68>] (ath10k_pci_diag_read32+0x1c/0x2c [ath10k_pci]) > [ 11.470767] [<7f288b68>] (ath10k_pci_diag_read32 [ath10k_pci]) from > [<7f28abe8>] (ath10k_pci_init_config+0x2c/0x290 [ath10k_pci]) > [ 11.482314] [<7f28abe8>] (ath10k_pci_init_config [ath10k_pci]) from > [<7f28d160>] (ath10k_ahb_hif_power_up+0x7c/0xe8 [ath10k_pci]) > [ 11.494153] [<7f28d160>] (ath10k_ahb_hif_power_up [ath10k_pci]) > from [<7f135348>] (ath10k_core_register_work+0x84/0x8f8 [ath10k_core]) > [ 11.505766] [<7f135348>] (ath10k_core_register_work [ath10k_core]) > from [<8023b614>] (process_one_work+0x1c0/0x2f8) > [ 11.517594] [<8023b614>] (process_one_work) from [<8023c650>] > (worker_thread+0x280/0x3c0) > [ 11.527919] [<8023c650>] (worker_thread) from [<802408f8>] > (kthread+0xd8/0xe8) > [ 11.536247] [<802408f8>] (kthread) from [<80209ce8>] > (ret_from_fork+0x14/0x2c) > > When I revert #1 eventually, my system is back to working. > So I'm blaming the #1 and #2 could have potential bugs or require > ath.git branch up-to-date. You caught me just as I was signing off yesterday evening, but just to confirm that you are now fairly certain that none of the 3 patches I was involved with[*] are related to your problems. If that's wrong and there's an action I need to take on the patches then let me know! :-) [*] The three patches I was involved with: ath10k: Add interrupt summary based CE processing https://patchwork.kernel.org/patch/11628299/ ath10k: Keep track of which interrupts fired, don't poll them https://patchwork.kernel.org/patch/11654631/ ath10k: Get rid of "per_ce_irq" hw param https://patchwork.kernel.org/patch/11654633/ -Doug
