Re: mac80211: ieee80211_ops.bss_info_changed question

["Tomas Winkler" <tomasw@gmail.com>]

On Sat, Apr 5, 2008 at 2:11 AM, Chatre, Reinette
<reinette.chatre@intel.com> 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@vger.kernel.org
>  More majordomo info at  http://vger.kernel.org/majordomo-info.html
>