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.

Stanislaw 

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

Reply via email to