git.librecmc.org Git - librecmc/librecmc.git/blob

   1 From f9fff67d2d7ca6fa8066132003a3deef654c55b1 Mon Sep 17 00:00:00 2001
   2 From: Nagarajan Maran <quic_nmaran@quicinc.com>
   3 Date: Mon, 17 Apr 2023 13:35:02 +0300
   4 Subject: [PATCH] wifi: ath11k: Fix SKB corruption in REO destination ring
   5
   6 While running traffics for a long time, randomly an RX descriptor
   7 filled with value "0" from REO destination ring is received.
   8 This descriptor which is invalid causes the wrong SKB (SKB stored in
   9 the IDR lookup with buffer id "0") to be fetched which in turn
  10 causes SKB memory corruption issue and the same leads to crash
  11 after some time.
  12
  13 Changed the start id for idr allocation to "1" and the buffer id "0"
  14 is reserved for error validation. Introduced Sanity check to validate
  15 the descriptor, before processing the SKB.
  16
  17 Crash Signature :
  18
  19 Unable to handle kernel paging request at virtual address 3f004900
  20 PC points to "b15_dma_inv_range+0x30/0x50"
  21 LR points to "dma_cache_maint_page+0x8c/0x128".
  22 The Backtrace obtained is as follows:
  23 [<8031716c>] (b15_dma_inv_range) from [<80313a4c>] (dma_cache_maint_page+0x8c/0x128)
  24 [<80313a4c>] (dma_cache_maint_page) from [<80313b90>] (__dma_page_dev_to_cpu+0x28/0xcc)
  25 [<80313b90>] (__dma_page_dev_to_cpu) from [<7fb5dd68>] (ath11k_dp_process_rx+0x1e8/0x4a4 [ath11k])
  26 [<7fb5dd68>] (ath11k_dp_process_rx [ath11k]) from [<7fb53c20>] (ath11k_dp_service_srng+0xb0/0x2ac [ath11k])
  27 [<7fb53c20>] (ath11k_dp_service_srng [ath11k]) from [<7f67bba4>] (ath11k_pci_ext_grp_napi_poll+0x1c/0x78 [ath11k_pci])
  28 [<7f67bba4>] (ath11k_pci_ext_grp_napi_poll [ath11k_pci]) from [<807d5cf4>] (__napi_poll+0x28/0xb8)
  29 [<807d5cf4>] (__napi_poll) from [<807d5f28>] (net_rx_action+0xf0/0x280)
  30 [<807d5f28>] (net_rx_action) from [<80302148>] (__do_softirq+0xd0/0x280)
  31 [<80302148>] (__do_softirq) from [<80320408>] (irq_exit+0x74/0xd4)
  32 [<80320408>] (irq_exit) from [<803638a4>] (__handle_domain_irq+0x90/0xb4)
  33 [<803638a4>] (__handle_domain_irq) from [<805bedec>] (gic_handle_irq+0x58/0x90)
  34 [<805bedec>] (gic_handle_irq) from [<80301a78>] (__irq_svc+0x58/0x8c)
  35
  36 Tested-on: IPQ8074 hw2.0 AHB WLAN.HK.2.7.0.1-01744-QCAHKSWPL_SILICONZ-1
  37
  38 Signed-off-by: Nagarajan Maran <quic_nmaran@quicinc.com>
  39 Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
  40 Link: https://lore.kernel.org/r/20230403191533.28114-1-quic_nmaran@quicinc.com
  41 ---
  42  drivers/net/wireless/ath/ath11k/dp_rx.c | 9 ++++++---
  43  1 file changed, 6 insertions(+), 3 deletions(-)
  44
  45 --- a/drivers/net/wireless/ath/ath11k/dp_rx.c
  46 +++ b/drivers/net/wireless/ath/ath11k/dp_rx.c
  47 @@ -389,10 +389,10 @@ int ath11k_dp_rxbufs_replenish(struct at
  48                         goto fail_free_skb;
  49
  50                 spin_lock_bh(&rx_ring->idr_lock);
  51 -               buf_id = idr_alloc(&rx_ring->bufs_idr, skb, 0,
  52 -                                  rx_ring->bufs_max * 3, GFP_ATOMIC);
  53 +               buf_id = idr_alloc(&rx_ring->bufs_idr, skb, 1,
  54 +                                  (rx_ring->bufs_max * 3) + 1, GFP_ATOMIC);
  55                 spin_unlock_bh(&rx_ring->idr_lock);
  56 -               if (buf_id < 0)
  57 +               if (buf_id <= 0)
  58                         goto fail_dma_unmap;
  59
  60                 desc = ath11k_hal_srng_src_get_next_entry(ab, srng);
  61 @@ -2665,6 +2665,9 @@ try_again:
  62                                    cookie);
  63                 mac_id = FIELD_GET(DP_RXDMA_BUF_COOKIE_PDEV_ID, cookie);
  64
  65 +               if (unlikely(buf_id == 0))
  66 +                       continue;
  67 +
  68                 ar = ab->pdevs[mac_id].ar;
  69                 rx_ring = &ar->dp.rx_refill_buf_ring;
  70                 spin_lock_bh(&rx_ring->idr_lock);