Add a debugfs file showing the rate at which
the last packet is received.
Signed-off-by: Saravana <[email protected]>
---
net/mac80211/debugfs_sta.c | 17 +++++++++++++++++
1 files changed, 17 insertions(+), 0 deletions(-)
diff --git a/net/mac80211/debugfs_sta.c b/net/mac80211/debugfs_sta.c
index 1090c24..3173fed 100644
--- a/net/mac80211/debugfs_sta.c
+++ b/net/mac80211/debugfs_sta.c
@@ -336,6 +336,22 @@ static ssize_t sta_current_tx_rate_read(struct file *file, char __user *userbuf,
}
STA_OPS(current_tx_rate);
+static ssize_t sta_last_rx_rate_read(struct file *file, char __user *userbuf,
+ size_t count, loff_t *ppos)
+{
+ struct sta_info *sta = file->private_data;
+ struct rate_info rinfo;
+ u16 rate;
+
+ sta_set_rate_info_rx(sta, &rinfo);
+
+ rate = cfg80211_calculate_bitrate(&rinfo);
+
+ return mac80211_format_buffer(userbuf, count, ppos, "%d.%d MBit/s\n",
+ rate/10, rate%10);
+}
+STA_OPS(last_rx_rate);
+
#define DEBUGFS_ADD(name) \
debugfs_create_file(#name, 0400, \
sta->debugfs.dir, sta, &sta_ ##name## _ops);
@@ -386,6 +402,7 @@ void ieee80211_sta_debugfs_add(struct sta_info *sta)
DEBUGFS_ADD(ht_capa);
DEBUGFS_ADD(last_ack_signal);
DEBUGFS_ADD(current_tx_rate);
+ DEBUGFS_ADD(last_rx_rate);
DEBUGFS_ADD_COUNTER(rx_packets, rx_packets);
DEBUGFS_ADD_COUNTER(tx_packets, tx_packets);
On Thu, 2012-11-29 at 19:54 +0530, Saravana wrote:
> Add a debugfs file showing the rate at which
> the last packet is received.
Applied, but:
> +static ssize_t sta_last_rx_rate_read(struct file *file, char __user *userbuf,
> + size_t count, loff_t *ppos)
fixed indentation here
> + return mac80211_format_buffer(userbuf, count, ppos, "%d.%d MBit/s\n",
> + rate/10, rate%10);
and here
(and also a bunch of whitespace changes in the other patches)
johannes
On 11/29/2012 08:04 PM, Johannes Berg wrote:
> On Thu, 2012-11-29 at 19:54 +0530, Saravana wrote:
>> Add a debugfs file showing the rate at which
>> the last packet is received.
>
> Applied, but:
>
>> +static ssize_t sta_last_rx_rate_read(struct file *file, char __user *userbuf,
>> + size_t count, loff_t *ppos)
>
> fixed indentation here
>
>
>> + return mac80211_format_buffer(userbuf, count, ppos, "%d.%d MBit/s\n",
>> + rate/10, rate%10);
>
> and here
>
> (and also a bunch of whitespace changes in the other patches)
>
Thanks Johannes for taking care of modifications. Will take care of indentation and the whitespace issues in future.
> johannes
>
>
On 2012-12-03 7:18 AM, Saravana wrote:
> On 12/02/2012 06:08 PM, Felix Fietkau wrote:
>> On 2012-11-29 3:24 PM, Saravana wrote:
>>> Add a debugfs file showing the rate at which
>>> the last packet is received.
>>>
>>> Signed-off-by: Saravana <[email protected]>
>> What's the point of adding debugfs files for data which is already
>> available via nl80211?
>>
> The debugfs is a repository of statistics. Many of the statistics like
> tx/rx bytes/packets, retries, signal,power etc which are available via nl80211(iw)
> are already available in debugfs. For the sake of completeness and to maintain
> uniformity the last rx rate is added to debugfs.
I think debugfs does not have (or need) completeness and uniformity, nor
is it meant as a repository of statistics. It is to be used for various
debugging purposes (as the name implies).
I think any additions of info to debugfs that are also available through
regular APIs should be considered unnecessary bloat and dropped.
- Felix
On 2012-11-29 3:24 PM, Saravana wrote:
> Add a debugfs file showing the rate at which
> the last packet is received.
>
> Signed-off-by: Saravana <[email protected]>
What's the point of adding debugfs files for data which is already
available via nl80211?
- Felix
On 12/02/2012 06:08 PM, Felix Fietkau wrote:
> On 2012-11-29 3:24 PM, Saravana wrote:
>> Add a debugfs file showing the rate at which
>> the last packet is received.
>>
>> Signed-off-by: Saravana <[email protected]>
> What's the point of adding debugfs files for data which is already
> available via nl80211?
>
The debugfs is a repository of statistics. Many of the statistics like
tx/rx bytes/packets, retries, signal,power etc which are available via nl80211(iw)
are already available in debugfs. For the sake of completeness and to maintain
uniformity the last rx rate is added to debugfs.
> - Felix
>