Tested-by: Sedat Dilek <sedat.dilek@xxxxxxxxx>
On Fri, Dec 3, 2010 at 5:05 AM, Nick Kossifidis <mickflemm@xxxxxxxxx> wrote:
> Â* Always free tx buffers before reset, since we also empty hw queues.
> ÂIf we don't and a queue gets stuck, we'll never decrease txq_len and sw
> Âwill keep thinking the queue is still stuck even after reset.
>
> ÂSigned-off-by: Nick Kossifidis <mickflemm@xxxxxxxxx>
>
> ---
> Âdrivers/net/wireless/ath/ath5k/base.c | Â Â6 ++++--
> Â1 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/wireless/ath/ath5k/base.c b/drivers/net/wireless/ath/ath5k/base.c
> index a8d380a..047b2a0 100644
> --- a/drivers/net/wireless/ath/ath5k/base.c
> +++ b/drivers/net/wireless/ath/ath5k/base.c
> @@ -2512,9 +2512,11 @@ ath5k_reset(struct ath5k_softc *sc, struct ieee80211_channel *chan,
> Â Â Â Âsynchronize_irq(sc->pdev->irq);
> Â Â Â Âstop_tasklets(sc);
>
> + Â Â Â /* We are going to empty hw queues
> + Â Â Â Â* so we should also free any remaining
> + Â Â Â Â* tx buffers */
> + Â Â Â ath5k_drain_tx_buffs(sc);
> Â Â Â Âif (chan) {
> - Â Â Â Â Â Â Â ath5k_drain_tx_buffs(sc);
> -
> Â Â Â Â Â Â Â Âsc->curchan = chan;
> Â Â Â Â Â Â Â Âsc->curband = &sc->sbands[chan->band];
> Â Â Â Â}
>
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
