On Sat, Apr 5, 2008 at 2:11 AM, Chatre, Reinette <reinette.chatre@xxxxxxxxx> wrote: > > 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. > Haven't look at the code but IMHO this function should not be called from scanning, Only beacons from connected AP should affect the driver behavior. These beacons are received even without scanning. Tomas > > 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 > -- 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
