On 8/27/2024 5:33 PM, Karthikeyan Periyasamy wrote:
> On 8/26/2024 7:19 AM, Baochen Qiang wrote:
...
>> @@ -883,7 +883,7 @@ void ath11k_mac_peer_cleanup_all(struct ath11k *ar)
>> mutex_lock(&ab->tbl_mtx_lock);
>> spin_lock_bh(&ab->base_lock);
>> list_for_each_entry_safe(peer, tmp, &ab->peers, list) {
>> - ath11k_peer_rx_tid_cleanup(ar, peer);
>> + ath11k_dp_peer_cleanup(ar, peer->vdev_id, peer->sta->addr);
>
> peer->tfm_mmic is allocated in ath11k_peer_rx_frag_setup() but its not
> cleanup in ath11k_dp_rx_frags_cleanup(), which is not symmetric now.
> Instead its freed in ath11k_dp_peer_cleanup(). can you refactor
> allocation/deallocation symmetric funcs ?
There was no action on this review comment.
Do you plan on submitting a v2?
/jeff
