Am Mittwoch, 24. September 2008 16:46:18 schrieb Johannes Berg:
> On Wed, 2008-09-24 at 16:36 +0200, Helmut Schaa wrote:
> > Could somebody please have a look at the TODO comments (I have no idea
> > how to wait until all null-func frames are ACKed)? Thanks.
>
> It's not really possible.
:(
> > + if (local->bg_scanning) {
> > + /*
> > + * background scan is in progress, notify all associated
> > + * access points about us leaving the channel and
> > + * update the filter flags
> > + */
> > + local->sw_scanning = 1;
>
> use true/false please
Sure, my fault.
> > + netif_tx_wake_all_queues(sdata->dev);
>
> This is worsening a problem we already have -- you can enable queues
> that the driver asked to be disabled. Until we fix that, I don't think
> we should tempt our luck even more.
I see! That's really problematic.
Do you have already an idea on how to fix it?
> > - if (local->sw_scanning || local->hw_scanning)
> > + if (local->sw_scanning || local->hw_scanning || local->bg_scanning)
>
> I don't really like that. I think these three bools should become an
> enum now.
Agreed.
> And why is sw_scanning false if bg_scanning is true anyway?
During a background scan the sw_scanning flag is set when a scan phase is
active and unset when an operation phase is active. Therefore I did not need
to adapt all checks for sw_scanning.
Thanks for your comments,
Helmut
--
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
