Re: [PATCH 1/4] Try 2: Add wireless statistics to d80211

2006-08-24 Thread Jiri Benc
On Wed, 23 Aug 2006 22:02:03 -0500, Larry Finger wrote:
 This patch modifies d80211 to add wireless statistics.
 
 Signed-Off-By: Larry [EMAIL PROTECTED]

Please fix the Signed-off-by line.

 
 ==
 diff --git a/include/net/d80211.h b/include/net/d80211.h
 index 42fdbf7..70655dc 100644
 --- a/include/net/d80211.h
 +++ b/include/net/d80211.h
 @@ -205,6 +205,8 @@ struct ieee80211_rx_status {
   int channel;
   int phymode;
   int ssi;
 + int signal;
 + int noise;
   int antenna;
   int rate;
   int flag;
 @@ -499,6 +501,9 @@ struct ieee80211_hw {
   /* This is the time in us to change channels
*/
   int channel_change_time;
 + /* This is maximum value of rssi reported by this interface
 +  */
 + int maxssi;

Device, not interface. And please do not put */ to a new line.

 [...]
 --- a/net/d80211/ieee80211_ioctl.c
 +++ b/net/d80211/ieee80211_ioctl.c
 [...]
 @@ -3019,7 +3062,6 @@ static int ieee80211_ioctl_giwauth(struc
   return ret;
   }
 
 -
   static int ieee80211_ioctl_siwencodeext(struct net_device *dev,
   struct iw_request_info *info,
   struct iw_point *erq, char *extra)

Please do not touch parts outside of the code you are
fixing/improving/etc. We'll need a code style patch later, but not now -
there are still too many patches floating around and I don't want to
break them more than is necessary.

Other then that, it looks good.

Thanks,

 Jiri

-- 
Jiri Benc
SUSE Labs
-
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


Re: [PATCH 1/4] Try 2: Add wireless statistics to d80211

2006-08-24 Thread Larry Finger

Jiri Benc wrote:

On Wed, 23 Aug 2006 22:02:03 -0500, Larry Finger wrote:

This patch modifies d80211 to add wireless statistics.
+   /* This is maximum value of rssi reported by this interface
+*/
+   int maxssi;


Device, not interface. And please do not put */ to a new line.


Noted on the first part, but I would like a clarification regarding style. I put the */ on the 
second line to match the rest of that section. Should a patch try to keep the local style in that 
part of the code, or change it to a some other standard?


Larry

-
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


Re: [PATCH 1/4] Try 2: Add wireless statistics to d80211

2006-08-24 Thread Jiri Benc
On Thu, 24 Aug 2006 09:39:52 -0500, Larry Finger wrote:
 Noted on the first part, but I would like a clarification regarding style. I 
 put the */ on the 
 second line to match the rest of that section. Should a patch try to keep the 
 local style in that 
 part of the code, or change it to a some other standard?

Patches should convert the code they are touching to coding style common
in the kernel. This way we will gradually convert at least part of
d80211 to the kernel coding style; the rest will be addressed by one big
patch just before merging into the vanilla (or -mm).

Thanks,

 Jiri

-- 
Jiri Benc
SUSE Labs
-
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


RE: [PATCH 1/4] Try 2: Add wireless statistics to d80211

2006-08-24 Thread Simon Barber
Why have both signal and rssi measures?

Simon 

-Original Message-
From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]
On Behalf Of Larry Finger
Sent: Wednesday, August 23, 2006 8:02 PM
To: Jiri Benc
Cc: John Linville; netdev@vger.kernel.org
Subject: [PATCH 1/4] Try 2: Add wireless statistics to d80211

This patch modifies d80211 to add wireless statistics.

Signed-Off-By: Larry [EMAIL PROTECTED]

==
diff --git a/include/net/d80211.h b/include/net/d80211.h index
42fdbf7..70655dc 100644
--- a/include/net/d80211.h
+++ b/include/net/d80211.h
@@ -205,6 +205,8 @@ struct ieee80211_rx_status {
  int channel;
  int phymode;
  int ssi;
+   int signal;
+   int noise;
  int antenna;
  int rate;
  int flag;
@@ -499,6 +501,9 @@ struct ieee80211_hw {
  /* This is the time in us to change channels
   */
  int channel_change_time;
+   /* This is maximum value of rssi reported by this interface
+*/
+   int maxssi;

int num_modes;
struct ieee80211_hw_modes *modes;
diff --git a/net/d80211/ieee80211.c b/net/d80211/ieee80211.c index
e72721f..2549484 100644
--- a/net/d80211/ieee80211.c
+++ b/net/d80211/ieee80211.c
@@ -3175,6 +3175,8 @@ ieee80211_rx_h_sta_process(struct ieee80
sta-rx_fragments++;
sta-rx_bytes += rx-skb-len;
sta-last_rssi = rx-u.rx.status-ssi;
+   sta-last_signal = rx-u.rx.status-signal;
+   sta-last_noise = rx-u.rx.status-noise;

if (!(rx-fc  IEEE80211_FCTL_MOREFRAGS)) {
/* Change STA power saving mode only in the end of a
frame diff --git a/net/d80211/ieee80211_i.h b/net/d80211/ieee80211_i.h
index 0d2d79d..1271513 100644
--- a/net/d80211/ieee80211_i.h
+++ b/net/d80211/ieee80211_i.h
@@ -336,6 +336,7 @@ struct ieee80211_local {
struct net_device *apdev; /* wlan#ap - management frames
(hostapd) */
int open_count;
int monitors;
+   struct iw_statistics wstats;
struct ieee80211_conf conf;

int dev_index;
diff --git a/net/d80211/ieee80211_ioctl.c b/net/d80211/ieee80211_ioctl.c
index 89a58e3..b121302 100644
--- a/net/d80211/ieee80211_ioctl.c
+++ b/net/d80211/ieee80211_ioctl.c
@@ -1581,6 +1581,16 @@ static int ieee80211_ioctl_giwrange(stru
range-min_frag = 256;
range-max_frag = 2346;

+   range-max_qual.qual = 100;
+   range-max_qual.level = 146;  /* set floor at -110 dBm (146 -
256) */
+   range-max_qual.noise = 146;
+   range-max_qual.updated = IW_QUAL_ALL_UPDATED;
+
+   range-avg_qual.qual = 50;
+   range-avg_qual.level = 0;
+   range-avg_qual.noise = 0;
+   range-avg_qual.updated = IW_QUAL_ALL_UPDATED;
+
return 0;
  }

@@ -2996,6 +3006,39 @@ static int ieee80211_ioctl_siwauth(struc
return ret;
  }

+/* Get wireless statistics.  Called by /proc/net/wireless and by 
+SIOCGIWSTATS */ static struct iw_statistics 
+*ieee80211_get_wireless_stats(struct net_device *net_dev) {
+   struct ieee80211_local *local = net_dev-ieee80211_ptr;
+   struct iw_statistics * wstats = local-wstats;
+   struct ieee80211_sub_if_data *sdata =
IEEE80211_DEV_TO_SUB_IF(net_dev);
+   struct sta_info *sta;
+   static int tmp_level = 0;
+   static int tmp_qual = 0;
+
+   sta = sta_info_get(local, sdata-u.sta.bssid);
+   if (!sta) {
+   wstats-discard.fragment = 0;
+   wstats-discard.misc = 0;
+   wstats-qual.qual = 0;
+   wstats-qual.level = 0;
+   wstats-qual.noise = 0;
+   wstats-qual.updated = IW_QUAL_ALL_INVALID;
+   } else {
+   if (!tmp_level) {   /* get initial values */
+   tmp_level = sta-last_signal;
+   tmp_qual = sta-last_rssi;
+   } else {/* smooth results */
+   tmp_level = (15 * tmp_level +
sta-last_signal)/16;
+   tmp_qual = (15 * tmp_qual + sta-last_rssi)/16;
+   }
+   wstats-qual.level = tmp_level;
+   wstats-qual.qual = 100*tmp_qual/local-hw-maxssi;
+   wstats-qual.noise = sta-last_noise;
+   wstats-qual.updated = IW_QUAL_ALL_UPDATED |
IW_QUAL_DBM;
+   }
+   return wstats;
+}

  static int ieee80211_ioctl_giwauth(struct net_device *dev,
   struct iw_request_info *info,
@@ -3019,7 +3062,6 @@ static int ieee80211_ioctl_giwauth(struc
return ret;
  }

-
  static int ieee80211_ioctl_siwencodeext(struct net_device *dev,
struct iw_request_info *info,
struct iw_point *erq, char
*extra) @@ -3184,6 +3226,7 @@ const struct iw_handler_def ieee80211_iw
.standard   = (iw_handler *) ieee80211_handler,
.private= (iw_handler *) ieee80211_private_handler,
.private_args   = 

Re: [PATCH 1/4] Try 2: Add wireless statistics to d80211

2006-08-24 Thread Larry Finger

Simon Barber wrote:

Why have both signal and rssi measures?


In the bcm43xx driver, the values are different. There is a routine that converts from rssi as a 
positive number into signal in a negative number that looks like dBm. Because of the reverse 
engineering, we have no idea what the logic behind this is, but I thought that we should keep it.


Larry

-
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