Rajkumar Manoharan <[email protected]> writes: > Decrement num_mpdus_ready only when rx amsdu is processed successfully. > Not doing so, will result in leak and impact stabilty under low memory > cases. > > Signed-off-by: Rajkumar Manoharan <[email protected]> > --- > drivers/net/wireless/ath/ath10k/htt_rx.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/net/wireless/ath/ath10k/htt_rx.c > b/drivers/net/wireless/ath/ath10k/htt_rx.c > index 96a7417..9696c2e 100644 > --- a/drivers/net/wireless/ath/ath10k/htt_rx.c > +++ b/drivers/net/wireless/ath/ath10k/htt_rx.c > @@ -2412,14 +2412,12 @@ static void ath10k_htt_txrx_compl_task(unsigned long > ptr) > struct ath10k_htt *htt = (struct ath10k_htt *)ptr; > struct ath10k *ar = htt->ar; > struct htt_tx_done tx_done = {}; > - struct sk_buff_head rx_q; > struct sk_buff_head rx_ind_q; > struct sk_buff_head tx_ind_q; > struct sk_buff *skb; > unsigned long flags; > int num_mpdus; > > - __skb_queue_head_init(&rx_q); > __skb_queue_head_init(&rx_ind_q); > __skb_queue_head_init(&tx_ind_q);
I guess you are removing the unused rx_q just as a cleanup? It's good practise to mention that in the commit log (or better yet in a separate patch). -- Kalle Valo _______________________________________________ ath10k mailing list [email protected] http://lists.infradead.org/mailman/listinfo/ath10k
