Am Mittwoch, 24. September 2008 16:46:18 schrieb Johannes Berg:

On Wed, 2008-09-24 at 16:36 +0200, Helmut Schaa wrote:

quoted

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.

:(

quoted

+		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.

quoted

+				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?

quoted

-	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