Search Linux Wireless

Re: [PATCH] cfg80211: increse bss expire time

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



I see that the patch was merged, but I'll comment anyway.

On Fri, 2012-04-13 at 16:38 +0530, Rajkumar Manoharan wrote:
> The background scan completion takes more time when the station is
> having heavy uplink traffic. The scan state machine decides to fall
> back to home channel on every off-channel visit when there are pending
> frames in tx queue. bgscan completion took ~30sec on dual band US
> regulatory card.
> 
> scan period = (20 active channels * probe timeout) +
>               (12 passive channels * passive probe timeout) +
>               (32 * timeout on home channel) +
>               (32 * flush timeout)


That formula seems a bit useless :-)

> Signed-off-by: Rajkumar Manoharan <rmanohar@xxxxxxxxxxxxxxxx>
> ---
>  net/wireless/scan.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/net/wireless/scan.c b/net/wireless/scan.c
> index fdbcfe6..1442bb6 100644
> --- a/net/wireless/scan.c
> +++ b/net/wireless/scan.c
> @@ -18,7 +18,7 @@
>  #include "nl80211.h"
>  #include "wext-compat.h"
>  
> -#define IEEE80211_SCAN_RESULT_EXPIRE	(15 * HZ)
> +#define IEEE80211_SCAN_RESULT_EXPIRE	(30 * HZ)

I'm not sure we'll ever manage to cover the entire scanning time unless
we bump this REALLY high. I'd much rather see us measuring the duration
of the scan, say per hardware, and not expire for that long. Of course
the maximum of all measurements, and take new ones every time, or
something like that. Also maybe avoid expiring while a scan is in
process.

johannes

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


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux