Johannes Berg <johannes@xxxxxxxxxxxxxxxx> writes:
> On Mon, 2009-02-23 at 18:37 +0200, Kalle Valo wrote:
>
>> +void ieee80211_beacon_loss_work(struct work_struct *work)
>> +{
>> + struct ieee80211_sub_if_data *sdata =
>> + container_of(work, struct ieee80211_sub_if_data,
>> + u.mgd.beacon_loss_work);
>> +
>> + printk(KERN_DEBUG "%s: beacon loss from AP %pM "
>> + "- disassociating\n", sdata->dev->name, sdata->u.mgd.bssid);
>> +
>> + ieee80211_set_disassoc(sdata, true, true,
>> + WLAN_REASON_PREV_AUTH_NOT_VALID);
>> +}
>
> We used to go through a probe request cycle once to make sure, but I'm
> not sure there's a point in that. Just pointing out the change here.
Good catch, I'll fix this in v2.
I'm also not sure if it's good idea to send a probe request when we
have lost beacons. It slows down the AP lost case quite a lot. But
this can be considered in a separate patch, for now I want to have the
same functionality. Most probably I will revisit when I'll start
working with roaming improvements.
> The code looks pretty good, but this will lead to an interesting
> situation where "iwlist wlan1 scan last" ("iw dev wlan1 scan dump") will
> not show _any_ BSS, which will probably trip up NM; this happens because
> the BSS will not be updated and expire after 10 seconds. I think we need
> a way to "hold on" to the BSS.
I'll take a look at this and try to come up with something.
Thanks for reviewing the patches.
--
Kalle Valo
--
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
