On Mon, Feb 14, 2011 at 09:37:14AM +0100, Stanislaw Gruszka wrote:
> On Fri, Feb 11, 2011 at 02:00:37PM -0800, [email protected] wrote:
> > This is a note to let you know that I've just added the patch titled
> > 
> >     ath9k: fix race conditions when stop device
> > 
> 
> [snip]
> 
> > @@ -1901,6 +1904,12 @@ static int ath9k_set_key(struct ieee8021
> >             ret = -EINVAL;
> >     }
> >  
> > +   /* we can now sync irq and kill any running tasklets, since we already
> > +    * disabled interrupts and not holding a spin lock */
> > +   synchronize_irq(sc->irq);
> > +   tasklet_kill(&sc->intr_tq);
> > +   tasklet_kill(&sc->bcon_tasklet);
> > +
> >     ath9k_ps_restore(sc);
> >     mutex_unlock(&sc->mutex);
> 
> This hunk should go to ath9k_stop() not to ath9k_set_key(), backport is
> broken.
> 
> I thought that fix will be useful in stable. Now I think we don't need
> it. Please drop that patch.

Now dropped.

greg k-h

_______________________________________________
stable mailing list
[email protected]
http://linux.kernel.org/mailman/listinfo/stable

Reply via email to