mac80211: advance the state machine immediately if no delay is needed
authorHelmut Schaa <helmut.schaa@googlemail.com>
Thu, 23 Jul 2009 10:13:48 +0000 (12:13 +0200)
committerJohn W. Linville <linville@tuxdriver.com>
Mon, 27 Jul 2009 19:24:16 +0000 (15:24 -0400)
Instead of queueing the scan work again without delay just process the
next state immediately.

Signed-off-by: Helmut Schaa <helmut.schaa@googlemail.com>
Acked-by: Johannes Berg <johannes@sipsolutions.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
net/mac80211/scan.c

index 71500f1dddbc9320dca8757cf99f47deb23e7b17..db122e4e60e5935e74930b29d0370a2c834f0ed2 100644 (file)
@@ -587,15 +587,21 @@ void ieee80211_scan_work(struct work_struct *work)
                return;
        }
 
-       switch (local->scan_state) {
-       case SCAN_SET_CHANNEL:
-               if (ieee80211_scan_state_set_channel(local, &next_delay))
-                       return;
-               break;
-       case SCAN_SEND_PROBE:
-               ieee80211_scan_state_send_probe(local, &next_delay);
-               break;
-       }
+       /*
+        * as long as no delay is required advance immediately
+        * without scheduling a new work
+        */
+       do {
+               switch (local->scan_state) {
+               case SCAN_SET_CHANNEL:
+                       if (ieee80211_scan_state_set_channel(local, &next_delay))
+                               return;
+                       break;
+               case SCAN_SEND_PROBE:
+                       ieee80211_scan_state_send_probe(local, &next_delay);
+                       break;
+               }
+       } while (next_delay == 0);
 
        queue_delayed_work(local->hw.workqueue, &local->scan_work,
                           next_delay);