On 04/16/2013 09:51 AM, Dan Carpenter wrote: > This is called with spinlocks held so we have to use GFP_ATOMIC. It's > the sc_pcu_lock in ath9k_stop() that's the issue. The call tree looks > like this: > > ath9k_stop() > ath_prepare_reset() > ath_stoprecv() > ath_flushrecv() > ath_rx_tasklet() > ath9k_dfs_process_phyerr() > pd->add_pulse() => dpd_add_pulse() > channel_detector_get() > channel_detector_create() > pri_detector_init() > > channel_detector_create() uses GFP_ATOMIC as well. > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > --- > Static analysis stuff. I haven't tested this but it looks like a real > bug to me. > Thanks Dan, it is a bug that slipped through the related patch in [1]. Tested-by & Acked-by: Zefir Kurtisi <zefir.kurtisi@xxxxxxxxxxx> [1] http://article.gmane.org/gmane.linux.kernel.wireless.general/106349/ -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
