Skip to content

Commit 513c1d8

Browse files
P Praneeshgregkh
authored andcommitted
wifi: ath12k: Fix double budget decrement while reaping monitor ring
[ Upstream commit 54c3500 ] Currently, the budget for monitor ring is reduced during each ring entry reaping and again when the end reason is HAL_MON_END_OF_PPDU, leading to inefficient budget use. The below mentioned commit intended to decrement the budget only for HAL_MON_END_OF_PPDU but did not remove the other decrement. Fix this by eliminating the budget decrement for each ring entry reaping, ensuring the driver always reaps one full PPDU worth of entries from the monitor destination ring. Tested-on: QCN9274 hw2.0 PCI WLAN.WBE.1.4.1-00199-QCAHKSWPL_SILICONZ-1 Tested-on: WCN7850 hw2.0 PCI WLAN.HMT.1.0.c5-00481-QCAHMTSWPL_V1.0_V2.0_SILICONZ-3 Fixes: 394a3fa ("wifi: ath12k: Optimize NAPI budget by adjusting PPDU processing") Signed-off-by: P Praneesh <[email protected]> Reviewed-by: Vasanthakumar Thiagarajan <[email protected]> Link: https://patch.msgid.link/[email protected] Signed-off-by: Jeff Johnson <[email protected]> Signed-off-by: Sasha Levin <[email protected]>
1 parent 4e43457 commit 513c1d8

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

drivers/net/wireless/ath/ath12k/dp_mon.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3761,7 +3761,6 @@ int ath12k_dp_mon_srng_process(struct ath12k *ar, int *budget,
37613761
ath12k_hal_srng_access_begin(ab, srng);
37623762

37633763
while (likely(*budget)) {
3764-
*budget -= 1;
37653764
mon_dst_desc = ath12k_hal_srng_dst_peek(ab, srng);
37663765
if (unlikely(!mon_dst_desc))
37673766
break;

0 commit comments

Comments
 (0)