Return-path: Received: from wolverine01.qualcomm.com ([199.106.114.254]:53296 "EHLO wolverine01.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753773Ab2ATPwk (ORCPT ); Fri, 20 Jan 2012 10:52:40 -0500 From: Vasanthakumar Thiagarajan To: CC: , Subject: [PATCH 1/6] ath6kl: Make sure to delete rx aggregation timer in aggr_reset_state() Date: Fri, 20 Jan 2012 21:21:57 +0530 Message-ID: <1327074722-3492-1-git-send-email-vthiagar@qca.qualcomm.com> (sfid-20120120_165242_558878_9CE036E7) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-wireless-owner@vger.kernel.org List-ID: The timer which is used to flush rx aggregation frames needs to be disabled when resetting the aggregation state. This is found in code review. Signed-off-by: Vasanthakumar Thiagarajan --- drivers/net/wireless/ath/ath6kl/txrx.c | 5 +++++ 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/drivers/net/wireless/ath/ath6kl/txrx.c b/drivers/net/wireless/ath/ath6kl/txrx.c index dd63371..cb7421a 100644 --- a/drivers/net/wireless/ath/ath6kl/txrx.c +++ b/drivers/net/wireless/ath/ath6kl/txrx.c @@ -1685,6 +1685,11 @@ void aggr_reset_state(struct aggr_info *aggr_info) { u8 tid; + if (aggr_info->timer_scheduled) { + del_timer(&aggr_info->timer); + aggr_info->timer_scheduled = false; + } + for (tid = 0; tid < NUM_OF_TIDS; tid++) aggr_delete_tid_state(aggr_info, tid); } -- 1.7.0.4