On Thu, 1 Mar 2007 00:43:10 -0500, Michael Wu wrote:
> On Wednesday 28 February 2007 16:44, Michael Wu wrote:
> > On Wednesday 28 February 2007 16:42, Michael Wu wrote:
> > > This makes scans switch STA interfaces into PS mode so the AP queues
> > > frames destined for us while we are scanning. This is achieved by sending
> > > a nullfunc data frame with the PS mode bit set before scanning commences,
> > > and a PS poll frame after scanning is completed.
> >
> > Er. I should update the description too..
> >
> And drop the ps_poll part since we don't use the ps poll frame now..
>
> [...]
> @@ -2541,10 +2572,12 @@ void ieee80211_scan_completed(struct iee
>
> spin_lock_bh(&local->sub_if_lock);
> list_for_each_entry(sdata, &local->sub_if_list, list) {
> - netif_wake_queue(sdata->dev);
> -
> - if (sdata->type == IEEE80211_IF_TYPE_STA)
> + if (sdata->type == IEEE80211_IF_TYPE_STA) {
> + if (sdata->u.sta.associated)
> + ieee80211_send_nullfunc(local, sdata, 0);
> ieee80211_sta_timer((unsigned long)&sdata->u.sta);
> + }
> + netif_wake_queue(sdata->dev);
Shouldn't the queue be woken before calling ieee80211_sta_timer?
Thanks,
Jiri
--
Jiri Benc
SUSE Labs
-
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
