On Thu, Jan 27, 2011 at 1:15 AM, Vivek Natarajan <vnatarajan@xxxxxxxxxxx> wrote: > Spin_lock has been tried to be acquired twice from ath9k_tasklet > to ath_reset which resulted in a machine freeze. > > Signed-off-by: Vivek Natarajan <vnatarajan@xxxxxxxxxxx> > --- Was this a stable fix? :( > Âdrivers/net/wireless/ath/ath9k/xmit.c | Â Â5 ++++- > Â1 files changed, 4 insertions(+), 1 deletions(-) > > diff --git a/drivers/net/wireless/ath/ath9k/xmit.c b/drivers/net/wireless/ath/ath9k/xmit.c > index c73452d..84673fb 100644 > --- a/drivers/net/wireless/ath/ath9k/xmit.c > +++ b/drivers/net/wireless/ath/ath9k/xmit.c > @@ -564,8 +564,11 @@ static void ath_tx_complete_aggr(struct ath_softc *sc, struct ath_txq *txq, > > Â Â Â Ârcu_read_unlock(); > > - Â Â Â if (needreset) > + Â Â Â if (needreset) { > + Â Â Â Â Â Â Â spin_unlock_bh(&sc->sc_pcu_lock); > Â Â Â Â Â Â Â Âath_reset(sc, false); > + Â Â Â Â Â Â Â spin_lock_bh(&sc->sc_pcu_lock); > + Â Â Â } > Â} > > Âstatic u32 ath_lookup_rate(struct ath_softc *sc, struct ath_buf *bf, > -- > 1.6.3.3 > > -- > 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 > -- 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