Search Linux Wireless

Re: [PATCH 3/4 V2] mac80211: add last beacon time in scan list

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

 



On Wed, 2008-06-25 at 15:17 +0300, Tomas Winkler wrote:
> From: Emmanuel Grumbach <emmanuel.grumbach@xxxxxxxxx>
> 
> This patch adds the interval between the scan results and the last time a
> beacon was received in the result of the scan.
> 
> Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@xxxxxxxxx>
> Signed-off-by: Tomas Winkler <tomas.winkler@xxxxxxxxx>
> ---
>  net/mac80211/mlme.c |   14 ++++++++++++++
>  1 files changed, 14 insertions(+), 0 deletions(-)
> 
> diff --git a/net/mac80211/mlme.c b/net/mac80211/mlme.c
> index 7b930d1..e76e62a 100644
> --- a/net/mac80211/mlme.c
> +++ b/net/mac80211/mlme.c
> @@ -4658,12 +4658,26 @@ ieee80211_sta_scan_result(struct net_device *dev,
>  		char *buf;
>  		buf = kmalloc(30, GFP_ATOMIC);
>  		if (buf) {
> +			unsigned long mid_range = (-1) / 2 + 1;

What is that?  I guess that's where Riemann's zeta function has its
non-trivial roots :-)

> +			time_diff = jiffies - bss->last_update > mid_range ?
> +				jiffies - bss->last_update :
> +				bss->last_update - jiffies;

That's pretty hairy.  Do we really lack a function to calculate time
difference?

> +			sprintf(buf, "Last beacon:%dms ago",
> +				jiffies_to_msecs(time_diff));

You lost space after the colon for some reason.

-- 
Regards,
Pavel Roskin
--
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 Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux