Johannes Berg wrote:
Seems we forgot to stop the queue while scanning. Better do that so we
don't transmit packets all the time during background scanning.
Signed-off-by: Johannes Berg <[EMAIL PROTECTED]>
--- a/net/ieee80211/softmac/ieee80211softmac_scan.c
+++ b/net/ieee80211/softmac/ieee80211softmac_scan.c
@@ -47,6 +47,7 @@ ieee80211softmac_start_scan(struct ieee8
sm->scanning = 1;
spin_unlock_irqrestore(&sm->lock, flags);
+ netif_stop_queue(sm->ieee->dev);
I think netif_stop_queue() is only supposed to be used directly from
inside a hard_start_xmit function. I think we should be using
netif_disable_tx() here.
This is what I was trying to point out when we were talking in person,
but I had confused myself with the start vs wake thing.
Daniel
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html