Re: [PATCH ath-next] wifi: ath12k: Pass the correct value of each TID during a stop AMPDU session

[Pablo MARTIN-GOMEZ <pmartin-gomez@freebox.fr>]

Hello,

On 25/02/2026 12:49, Reshma Immaculate Rajkumar wrote:
> With traffic ongoing for data TID [TID 0], an ADDBA request to
It's a DELBA that is going to be sent to stop a BA session.
> stop AMPDU for the BA session was received on management TID [TID 4].
> The corresponding TID number was incorrectly passed to stop the BA session,
> resulting in the BA session for data TIDs being stopped and the BA size
> being reduced to 1, causing an overall dip in TCP throughput.
I'm guessing you only tested TCP traffic, but UDP throughput is also going to dip.
> 
> Fix this issue by passing the correct argument from
> ath12k_dp_rx_ampdu_stop() to ath12k_dp_arch_peer_rx_tid_reo_update()
> during an AMPDU stop session. Instead of passing peer->dp_peer->rx_tid,
> which is the base address of the array, corresponding to TID 0, pass
> the value of &peer->dp_peer->rx_tid[params->tid]. With this, the
> different TID numbers are accounted for.
> 
> Tested-on: QCN9274 hw2.0 PCI WLAN.WBE.1.5-01651-QCAHKSWPL_SILICONZ-1
> 
> Fixes: d889913205cf ("wifi: ath12k: driver for Qualcomm Wi-Fi 7 devices")
> Signed-off-by: Reshma Immaculate Rajkumar <reshma.rajkumar@oss.qualcomm.com>
[...]

Pablo MG