On Friday, April 04, 2008 7:43 AM, Johannes Berg wrote: > Reinette, > >> In the comments about bss_info_changed it is said that the callback >> has to be atomic. I was looking at it and could only find that it is >> called either from the workqueue or an ioctl. It thus seems that we >> may be able to sleep in there. I likely missed the spot where it >> needs to be atomic ... could you please direct me to it? > > I think you may be right, I can't find any spots either. I was likely > mislead when writing that by the fact that much of the code in > ieee80211_sta.c uses GFP_ATOMIC while it actually runs from a > workqueue, which is actually confusing me still. My mistake ... it can be called from tasklet during scanning (ieee80211_sta_rx_scan). The comment that requires it to be atomic is correct. Reinette -- 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