ath9k: Fix bug in the way "bf_tx_aborted" of struct ath_buf is used
authorVasanthakumar Thiagarajan <vasanth@atheros.com>
Thu, 27 May 2010 02:06:53 +0000 (19:06 -0700)
committerJohn W. Linville <linville@tuxdriver.com>
Fri, 28 May 2010 17:40:22 +0000 (13:40 -0400)
commit7c9fd60f9764373414c0a64f500a78635b0a0a7b
tree5bf4d031d5c8796b15a7d39875365adc362405fe
parent9be8ab2ea81f84c1726d79613c342141f5d19a3b
ath9k: Fix bug in the way "bf_tx_aborted" of struct ath_buf is used

This bug was introduced by the following commit

Author: Vasanthakumar Thiagarajan <vasanth@atheros.com>
Date:   Thu Apr 15 17:38:46 2010 -0400

ath9k: Remove ATH9K_TX_SW_ABORTED and introduce a bool for this purpose

Wrong buffer is checked for bf_tx_aborted field in ath_tx_num_badfrms(),
this may result in a rate scaling with wrong feedback (number
of unacked frames in this case). It is the last one in the chain
of buffers for an aggregate frame that should be checked.

Also it misses the initialization of this field in the buffer,
this may lead to a situation where we stop the sw retransmission
of failed subframes associated to this buffer.

Signed-off-by: Vasanthakumar Thiagarajan <vasanth@atheros.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ath/ath9k/xmit.c